Correct InspectorInstrumentation for background HTML parser
authorTony Gentilcore <tonyg@chromium.org>
Sat, 23 Feb 2013 03:14:36 +0000 (03:14 +0000)
committerTony Gentilcore <tonyg@chromium.org>
Sat, 23 Feb 2013 03:14:36 +0000 (03:14 +0000)
commit8487f82f55f4124eb467af8d18f4b68e6511d4f0
tree7d9926880d4d286a7ab9d0a89473728efd2afac0
parent79049af1cb4ab3ae856c3f6d39bc8ad51f0c8c29
Correct InspectorInstrumentation for background HTML parser
https://bugs.webkit.org/show_bug.cgi?id=110678

Reviewed by Adam Barth.

resumeParsingAfterScriptExecution potentially calls processParsedChunkFromBackgroundParser multiple times
without yielding. So it is not correct to count each of those as a separate ParseHTML event.

This patch moves the instrumentation outside of the loop.

No new tests because no new functionality.

* html/parser/HTMLDocumentParser.cpp:
(WebCore::HTMLDocumentParser::didReceiveParsedChunkFromBackgroundParser):
(WebCore::HTMLDocumentParser::processParsedChunkFromBackgroundParser):
(WebCore::HTMLDocumentParser::resumeParsingAfterScriptExecution):

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@143828 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Source/WebCore/ChangeLog
Source/WebCore/html/parser/HTMLDocumentParser.cpp