Bug 110678

Summary: Correct InspectorInstrumentation for background HTML parser
Product: WebKit Reporter: Tony Gentilcore <tonyg>
Component: New BugsAssignee: Tony Gentilcore <tonyg>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, eric, esprehn+autocc, ojan.autocc, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 106127    
Attachments:
Description Flags
Patch none

Tony Gentilcore
Reported 2013-02-22 18:28:07 PST
Correct InspectorInstrumentation for background HTML parser
Attachments
Patch (4.36 KB, patch)
2013-02-22 18:31 PST, Tony Gentilcore 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Tony Gentilcore
Comment 1 2013-02-22 18:31:40 PST
Created attachment 189889 [details] Patch
WebKit Review Bot
Comment 2 2013-02-22 19:16:25 PST
Comment on attachment 189889 [details] Patch Clearing flags on attachment: 189889 Committed r143828: <http://trac.webkit.org/changeset/143828>
WebKit Review Bot
Comment 3 2013-02-22 19:16:28 PST
All reviewed patches have been landed. Closing bug.
Eric Seidel (no email)
Comment 4 2013-02-22 20:05:00 PST
Comment on attachment 189889 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=189889&action=review > Source/WebCore/html/parser/HTMLDocumentParser.cpp:316 > + InspectorInstrumentationCookie cookie = InspectorInstrumentation::willWriteHTML(document(), 0, lineNumber().zeroBasedInt()); This should be trivial to fix, if we add it to the chuck struct passed from the background. :)
Eric Seidel (no email)
Comment 5 2013-02-22 20:05:11 PST
Thanks for doing this. :)
Note You need to log in before you can comment on or make changes to this bug.