Title: [140986] trunk/Source/WebCore
Revision
140986
Author
[email protected]
Date
2013-01-28 11:35:41 -0800 (Mon, 28 Jan 2013)

Log Message

Don't use the threaded HTML parser for _javascript_: URLs
https://bugs.webkit.org/show_bug.cgi?id=107975

Reviewed by Adam Barth.

Several layout tests depend on _javascript_: URL iframes loading synchronously including fast/loader/_javascript_-url-encoding.html.
This patch avoids using the threaded parser for those URLs so they will continue to be synchronous.

No new tests because covered by existing tests.

* html/parser/HTMLParserOptions.cpp:
(WebCore::HTMLParserOptions::HTMLParserOptions):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (140985 => 140986)


--- trunk/Source/WebCore/ChangeLog	2013-01-28 19:28:01 UTC (rev 140985)
+++ trunk/Source/WebCore/ChangeLog	2013-01-28 19:35:41 UTC (rev 140986)
@@ -1,3 +1,18 @@
+2013-01-28  Tony Gentilcore  <[email protected]>
+
+        Don't use the threaded HTML parser for _javascript_: URLs
+        https://bugs.webkit.org/show_bug.cgi?id=107975
+
+        Reviewed by Adam Barth.
+
+        Several layout tests depend on _javascript_: URL iframes loading synchronously including fast/loader/_javascript_-url-encoding.html.
+        This patch avoids using the threaded parser for those URLs so they will continue to be synchronous.
+
+        No new tests because covered by existing tests.
+
+        * html/parser/HTMLParserOptions.cpp:
+        (WebCore::HTMLParserOptions::HTMLParserOptions):
+
 2013-01-27  Sam Weinig  <[email protected]>
 
         SVGPathStringSource should not up-convert 8-bit strings to UTF-16

Modified: trunk/Source/WebCore/html/parser/HTMLParserOptions.cpp (140985 => 140986)


--- trunk/Source/WebCore/html/parser/HTMLParserOptions.cpp	2013-01-28 19:28:01 UTC (rev 140985)
+++ trunk/Source/WebCore/html/parser/HTMLParserOptions.cpp	2013-01-28 19:35:41 UTC (rev 140986)
@@ -41,7 +41,7 @@
     Settings* settings = document ? document->settings() : 0;
     usePreHTML5ParserQuirks = settings && settings->usePreHTML5ParserQuirks();
 #if ENABLE(THREADED_HTML_PARSER)
-    useThreading = settings && settings->threadedHTMLParser();
+    useThreading = settings && settings->threadedHTMLParser() && !document->url().isBlankURL();
 #else
     useThreading = false;
 #endif
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to