Changeset 209997 in webkit

Timestamp:

Dec 19, 2016 2:06:25 PM (7 years ago)

Author:

barraclough@apple.com

Message:

[CSS Parser] Enable CSS Deferred Parsing
​https://bugs.webkit.org/show_bug.cgi?id=165869

Unreviewed rollout – looks like this regressed performance.

Source/WebCore:

• css/parser/CSSParserMode.h:
• page/Settings.in:

Source/WebKit2:

• Shared/WebPreferencesDefinitions.h:

Location:

trunk/Source

Files:

• WebCore/ChangeLog (modified) (1 diff)
• WebCore/css/parser/CSSParserMode.h (modified) (1 diff)
• WebCore/page/Settings.in (modified) (1 diff)
• WebKit2/ChangeLog (modified) (1 diff)
• WebKit2/Shared/WebPreferencesDefinitions.h (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2016-12-19  Gavin Barraclough  <barraclough@apple.com>
+
+        [CSS Parser] Enable CSS Deferred Parsing
+        https://bugs.webkit.org/show_bug.cgi?id=165869
+
+        Unreviewed rollout – looks like this regressed performance.
+
+        * css/parser/CSSParserMode.h:
+        * page/Settings.in:
+
 2016-12-18  Brent Fulgham  <bfulgham@apple.com>
 

trunk/Source/WebCore/css/parser/CSSParserMode.h

@@ -108 +108 @@
     bool springTimingFunctionEnabled { false };
     
-    bool deferredCSSParserEnabled { true };
+    bool deferredCSSParserEnabled { false };
 
     URL completeURL(const String& url) const

trunk/Source/WebCore/page/Settings.in

@@ -250 +250 @@
 newBlockInsideInlineModelEnabled initial=false, setNeedsStyleRecalcInAllFrames=1
 
-deferredCSSParserEnabled initial=true
+deferredCSSParserEnabled initial=false
 
 httpEquivEnabled initial=true

trunk/Source/WebKit2/ChangeLog

@@ +1 @@
+2016-12-19  Gavin Barraclough  <barraclough@apple.com>
+
+        [CSS Parser] Enable CSS Deferred Parsing
+        https://bugs.webkit.org/show_bug.cgi?id=165869
+
+        Unreviewed rollout – looks like this regressed performance.
+
+        * Shared/WebPreferencesDefinitions.h:
+
 2016-12-19  Anders Carlsson  <andersca@apple.com>
 

trunk/Source/WebKit2/Shared/WebPreferencesDefinitions.h

@@ -229 +229 @@
     macro(ServiceControlsEnabled, serviceControlsEnabled, Bool, bool, false, "", "") \
     macro(NewBlockInsideInlineModelEnabled, newBlockInsideInlineModelEnabled, Bool, bool, false, "", "") \
-    macro(DeferredCSSParserEnabled, deferredCSSParserEnabled, Bool, bool, true, "", "") \
+    macro(DeferredCSSParserEnabled, deferredCSSParserEnabled, Bool, bool, false, "", "") \
     macro(HTTPEquivEnabled, httpEquivEnabled, Bool, bool, true, "", "") \
     macro(MockCaptureDevicesEnabled, mockCaptureDevicesEnabled, Bool, bool, false, "", "") \