Changeset 228982 in webkit

Timestamp:

Feb 25, 2018 6:13:09 PM (6 years ago)

Author:

ap@apple.com

Message:

Font smoothing doesn't get disabled if the preference is set before launching WebContent process
​https://bugs.webkit.org/show_bug.cgi?id=183108

Reviewed by Tim Horton.

• WebProcess/WebProcess.cpp: (WebKit::WebProcess::initializeWebProcess):

Fix the bug. While at it, also avoid the anti-pattern in setAlwaysUsesComplexTextCodePath.

Location:

trunk/Source/WebKit

Files:

• ChangeLog (modified) (1 diff)
• WebProcess/WebProcess.cpp (modified) (1 diff)

trunk/Source/WebKit/ChangeLog

@@ +1 @@
+2018-02-25  Alexey Proskuryakov  <ap@apple.com>
+
+        Font smoothing doesn't get disabled if the preference is set before launching WebContent process
+        https://bugs.webkit.org/show_bug.cgi?id=183108
+
+        Reviewed by Tim Horton.
+
+        * WebProcess/WebProcess.cpp: (WebKit::WebProcess::initializeWebProcess):
+        Fix the bug. While at it, also avoid the anti-pattern in setAlwaysUsesComplexTextCodePath.
+
 2018-02-25  Philippe Normand  <pnormand@igalia.com>
 

trunk/Source/WebKit/WebProcess/WebProcess.cpp

@@ -375 +375 @@
     setResourceLoadStatisticsEnabled(parameters.resourceLoadStatisticsEnabled);
 
-    if (parameters.shouldAlwaysUseComplexTextCodePath)
-        setAlwaysUsesComplexTextCodePath(true);
-
-    if (parameters.shouldUseFontSmoothing)
-        setShouldUseFontSmoothing(true);
+    setAlwaysUsesComplexTextCodePath(parameters.shouldAlwaysUseComplexTextCodePath);
+
+    setShouldUseFontSmoothing(parameters.shouldUseFontSmoothing);
 
     if (parameters.shouldUseTestingNetworkSession)