Changeset 140402 in webkit
- Timestamp:
- Jan 22, 2013 2:24:30 AM (11 years ago)
- Location:
- trunk/Source
- Files:
-
- 4 edited
Legend:
- Unmodified
- Added
- Removed
-
trunk/Source/WebCore/ChangeLog
r140401 r140402 1 2013-01-22 Eric Seidel <eric@webkit.org> 2 3 Turn on ENABLE_THREADED_HTML_PARSER for Chromium (it's still disabled at runtime) 4 https://bugs.webkit.org/show_bug.cgi?id=107519 5 6 Reviewed by Adam Barth. 7 8 Add some ASSERTs to be double-dog-sure that we're never using 9 any of the threading code when threading is not enabled. 10 11 I also removed an if-branch in the !ENABLE_THREADED_HTML_PARSER case 12 as it's not needed. 13 14 * html/parser/HTMLDocumentParser.cpp: 15 (WebCore::HTMLDocumentParser::prepareToStopParsing): 16 (WebCore::HTMLDocumentParser::startBackgroundParser): 17 (WebCore::HTMLDocumentParser::stopBackgroundParser): 18 1 19 2013-01-22 Kentaro Hara <haraken@chromium.org> 2 20 -
trunk/Source/WebCore/html/parser/HTMLDocumentParser.cpp
r140209 r140402 147 147 RefPtr<HTMLDocumentParser> protect(this); 148 148 149 #if ENABLE(THREADED_HTML_PARSER) 149 150 // NOTE: This pump should only ever emit buffered character tokens, 150 151 // so ForceSynchronous vs. AllowYield should be meaningless. 151 152 if (!shouldUseThreading()) 153 #endif 152 154 pumpTokenizerIfPossible(ForceSynchronous); 153 155 154 156 if (isStopped()) 155 157 return; … … 455 457 void HTMLDocumentParser::startBackgroundParser() 456 458 { 459 ASSERT(shouldUseThreading()); 457 460 ASSERT(!m_haveBackgroundParser); 458 461 m_haveBackgroundParser = true; … … 466 469 void HTMLDocumentParser::stopBackgroundParser() 467 470 { 471 ASSERT(shouldUseThreading()); 468 472 ASSERT(m_haveBackgroundParser); 469 473 m_haveBackgroundParser = false; -
trunk/Source/WebKit/chromium/ChangeLog
r140401 r140402 1 2013-01-22 Eric Seidel <eric@webkit.org> 2 3 Turn on ENABLE_THREADED_HTML_PARSER for Chromium (it's still disabled at runtime) 4 https://bugs.webkit.org/show_bug.cgi?id=107519 5 6 Reviewed by Adam Barth. 7 8 This makes our development lives easier, and makes it possible for the bots 9 to run threaded-parser-only tests by toggling the runtime enable 10 via window.internals.settings. 11 12 * features.gypi: 13 1 14 2013-01-22 Kentaro Hara <haraken@chromium.org> 2 15 -
trunk/Source/WebKit/chromium/features.gypi
r140377 r140402 118 118 'ENABLE_TEMPLATE_ELEMENT=1', 119 119 'ENABLE_TEXT_AUTOSIZING=1', 120 'ENABLE_THREADED_HTML_PARSER= 0',120 'ENABLE_THREADED_HTML_PARSER=1', 121 121 'ENABLE_TOUCH_ADJUSTMENT=1', 122 122 'ENABLE_TOUCH_EVENTS=<(enable_touch_events)',
Note: See TracChangeset
for help on using the changeset viewer.