Title: [139367] trunk/Source
Revision
139367
Author
aba...@webkit.org
Date
2013-01-10 14:06:20 -0800 (Thu, 10 Jan 2013)

Log Message

Add an ENABLE macro and a WebCore::Setting for the threaded parser
https://bugs.webkit.org/show_bug.cgi?id=106595

Reviewed by Benjamin Poulain.

Source/WebCore: 

This patch adds both a WebCore::Setting and an ENABLE macro for the
threaded parser. We plan to keep the ENABLE macro off by default for a
while. The runtime setting will let us more easily conduct A/B
experiments when the time arises.

* page/Settings.in:

Source/WebKit/chromium: 

Explicitly disable THREADED_HTML_PARSER for Chromium so that we can
more easily enable it locally.

* features.gypi:

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (139366 => 139367)


--- trunk/Source/WebCore/ChangeLog	2013-01-10 21:58:39 UTC (rev 139366)
+++ trunk/Source/WebCore/ChangeLog	2013-01-10 22:06:20 UTC (rev 139367)
@@ -1,3 +1,17 @@
+2013-01-10  Adam Barth  <aba...@webkit.org>
+
+        Add an ENABLE macro and a WebCore::Setting for the threaded parser
+        https://bugs.webkit.org/show_bug.cgi?id=106595
+
+        Reviewed by Benjamin Poulain.
+
+        This patch adds both a WebCore::Setting and an ENABLE macro for the
+        threaded parser. We plan to keep the ENABLE macro off by default for a
+        while. The runtime setting will let us more easily conduct A/B
+        experiments when the time arises.
+
+        * page/Settings.in:
+
 2013-01-10  Xianzhu Wang  <wangxian...@chromium.org>
 
         Regression(r129944): Heap-use-after-free in WebCore::computeNonFastScrollableRegion

Modified: trunk/Source/WebCore/page/Settings.in (139366 => 139367)


--- trunk/Source/WebCore/page/Settings.in	2013-01-10 21:58:39 UTC (rev 139366)
+++ trunk/Source/WebCore/page/Settings.in	2013-01-10 22:06:20 UTC (rev 139367)
@@ -154,6 +154,8 @@
 syncXHRInDocumentsEnabled initial=true
 cookieEnabled initial=true
 
+threadedHTMLParser initial=false, conditional=THREADED_HTML_PARSER
+
 # When enabled, window.blur() does not change focus, and
 # window.focus() only changes focus when invoked from the context that
 # created the window.

Modified: trunk/Source/WebKit/chromium/ChangeLog (139366 => 139367)


--- trunk/Source/WebKit/chromium/ChangeLog	2013-01-10 21:58:39 UTC (rev 139366)
+++ trunk/Source/WebKit/chromium/ChangeLog	2013-01-10 22:06:20 UTC (rev 139367)
@@ -1,3 +1,15 @@
+2013-01-10  Adam Barth  <aba...@webkit.org>
+
+        Add an ENABLE macro and a WebCore::Setting for the threaded parser
+        https://bugs.webkit.org/show_bug.cgi?id=106595
+
+        Reviewed by Benjamin Poulain.
+
+        Explicitly disable THREADED_HTML_PARSER for Chromium so that we can
+        more easily enable it locally.
+
+        * features.gypi:
+
 2013-01-10  John Mellor  <joh...@chromium.org>
 
         Fix scale of screen.width, window.outerWidth and @media device-width when page scale not applied in compositor.

Modified: trunk/Source/WebKit/chromium/features.gypi (139366 => 139367)


--- trunk/Source/WebKit/chromium/features.gypi	2013-01-10 21:58:39 UTC (rev 139366)
+++ trunk/Source/WebKit/chromium/features.gypi	2013-01-10 22:06:20 UTC (rev 139367)
@@ -115,10 +115,11 @@
       'ENABLE_SVG_FONTS=<(enable_svg)',
       'ENABLE_TEMPLATE_ELEMENT=1',
       'ENABLE_TEXT_AUTOSIZING=1',
+      'ENABLE_THREADED_HTML_PARSER=0',
       'ENABLE_TOUCH_ADJUSTMENT=1',
       'ENABLE_TOUCH_EVENTS=<(enable_touch_events)',
+      'ENABLE_TOUCH_EVENT_TRACKING=<(enable_touch_events)',
       'ENABLE_TOUCH_ICON_LOADING=<(enable_touch_icon_loading)',
-      'ENABLE_TOUCH_EVENT_TRACKING=<(enable_touch_events)',
       'ENABLE_TOUCH_SLIDER=1',
       'ENABLE_USER_TIMING=1',
       'ENABLE_V8_SCRIPT_DEBUG_SERVER=1',
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to