Safari beach-balls loading large pages with HTML5 parser
Created attachment 58724 [details] Patch
Created attachment 58725 [details] Patch
Comment on attachment 58725 [details] Patch Minor nits. Don't know how to test this code... :( WebCore/html/HTML5DocumentParser.cpp:51 + // FIXME: We would this to be 0.2. Historically .2 caused slowness on old machines. 0.2 / .2 consistency. WebCore/html/HTML5DocumentParser.cpp:129 + bool HTML5DocumentParser::processingData() const WTF. Please add a FIXME to track this down and figure out what it does. WebCore/html/HTML5DocumentParser.cpp:174 + session.processedTokens++; prefix increment WebCore/html/HTML5DocumentParser.cpp:192 + return; This is supposed to be break, not return WebCore/html/HTML5DocumentParser.cpp:220 + // FIXME: This belongs on Document. Please add this to document. WebCore/html/HTML5DocumentParser.cpp:230 + // HACK: If a layout is scheduled, wait again to let the layout timer run first. Is this copied from the old document parser? WebCore/html/HTML5DocumentParser.cpp:278 + SynchronousMode mode = isFromNetwork ? AllowYield : ForceSynchronous; Please inline this into call site.
Created attachment 58730 [details] Patch for landing
Created attachment 58731 [details] Patch for landing
Comment on attachment 58731 [details] Patch for landing Clearing flags on attachment: 58731 Committed r61165: <http://trac.webkit.org/changeset/61165>
All reviewed patches have been landed. Closing bug.
http://trac.webkit.org/changeset/61165 might have broken Chromium Linux Release
Missing an include. Build a patch now.