Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1172)

Unified Diff: Source/core/html/parser/HTMLParserOptions.cpp

Issue 644573002: Disable async main thread HTML parsing (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: WebFrameTest should wait for background thread tasks Created 6 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « Source/core/html/parser/HTMLDocumentParser.cpp ('k') | Source/web/tests/FrameTestHelpers.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/html/parser/HTMLParserOptions.cpp
diff --git a/Source/core/html/parser/HTMLParserOptions.cpp b/Source/core/html/parser/HTMLParserOptions.cpp
index 1eca29df0c505d79d25da3ec729f4e43be9e95b8..494909b500a1afd66a2eaafd60d0b4a55ce3323c 100644
--- a/Source/core/html/parser/HTMLParserOptions.cpp
+++ b/Source/core/html/parser/HTMLParserOptions.cpp
@@ -39,15 +39,7 @@ HTMLParserOptions::HTMLParserOptions(Document* document)
LocalFrame* frame = document ? document->frame() : 0;
scriptEnabled = frame && frame->script().canExecuteScripts(NotAboutToExecuteScript);
pluginsEnabled = frame && frame->loader().allowPlugins(NotAboutToInstantiatePlugin);
-
- // We force the main-thread parser for three cases:
- // - about:blank and javascript (which uses about:blank) for compatibility
- // with historical synchronous loading/parsing behavior.
- // - inspector/DOMPatchSupport replacing the whole document.
- // (DOMPatchSupport calls in |DocumentParser::pinToMainThread()| for this case)
- // FIXME: Gecko does not load javascript: urls synchronously, why do we?
- // See LayoutTests/loader/iframe-sync-loads.html
- useThreading = document && !document->url().isAboutBlankURL();
+ useThreading = true;
}
}
« no previous file with comments | « Source/core/html/parser/HTMLDocumentParser.cpp ('k') | Source/web/tests/FrameTestHelpers.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698