Bug 1383817 - clamp gamma/contrast for ScaledFontDWrite when creating SkTypeface. r=mchang

MozReview-Commit-ID: LSSffVooDCI

gfx/2d/ScaledFontDWrite.cpp

@@ -156,7 +156,19 @@ ScaledFontDWrite::GetSkTypeface()
       return nullptr;
     }
 
-    mTypeface = SkCreateTypefaceFromDWriteFont(factory, mFontFace, mStyle, mForceGDIMode, mGamma, mContrast);
+    Float gamma = mGamma;
+    // Skia doesn't support a gamma value outside of 0-4, so default to 2.2
+    if (gamma < 0.0f || gamma > 4.0f) {
+      gamma = 2.2f;
+    }
+
+    Float contrast = mContrast;
+    // Skia doesn't support a contrast value outside of 0-1, so default to 1.0
+    if (contrast < 0.0f || contrast > 1.0f) {
+      contrast = 1.0f;
+    }
+
+    mTypeface = SkCreateTypefaceFromDWriteFont(factory, mFontFace, mStyle, mForceGDIMode, gamma, contrast);
   }
   return mTypeface;
 }

gfx/thebes/gfxWindowsPlatform.cpp

@@ -1193,14 +1193,6 @@ gfxWindowsPlatform::SetupClearTypeParams()
             }
         }
 
-        if (GetDefaultContentBackend() == BackendType::SKIA) {
-          // Skia doesn't support a contrast value outside of 0-1, so default to 1.0
-          if (contrast < 0.0 || contrast > 1.0) {
-            NS_WARNING("Custom dwrite contrast not supported in Skia. Defaulting to 1.0.");
-            contrast = 1.0;
-          }
-        }
-
         // For parameters that have not been explicitly set,
         // we copy values from default params (or our overridden value for contrast)
         if (gamma < 1.0 || gamma > 2.2) {