Bug 107159

Summary: Stop the background parser when canceling parsing to avoid crashing on many layout tests
Product: WebKit Reporter: Eric Seidel (no email) <eric>
Component: New BugsAssignee: Eric Seidel (no email) <eric>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, tonyg, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 106127    
Attachments:
Description Flags
Patch
none
Patch for landing none

Description Eric Seidel (no email) 2013-01-17 12:32:19 PST
Stop the background parser when canceling parsing to avoid crashing on many layout tests:w
Comment 1 Eric Seidel (no email) 2013-01-17 12:33:20 PST
Created attachment 183246 [details]
Patch
Comment 2 WebKit Review Bot 2013-01-17 12:59:57 PST
Comment on attachment 183246 [details]
Patch

Rejecting attachment 183246 [details] from commit-queue.

Failed to run "['/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '-..." exit_code: 2 cwd: /mnt/git/webkit-commit-queue

Last 500 characters of output:
o-rtti -fno-threadsafe-statics -fvisibility-inlines-hidden -Wsign-compare  -c ../../Source/WebCore/html/parser/HTMLDocumentParser.cpp -o obj/Source/WebCore/html/parser/webcore_html.HTMLDocumentParser.o
../../Source/WebCore/html/parser/HTMLDocumentParser.cpp: In member function 'virtual void WebCore::HTMLDocumentParser::stopParsing()':
../../Source/WebCore/html/parser/HTMLDocumentParser.cpp:134: error: 'stopBackgroundParser' was not declared in this scope
ninja: build stopped: subcommand failed.

Full output: http://queues.webkit.org/results/15943107
Comment 3 Eric Seidel (no email) 2013-01-17 13:00:20 PST
I guess the rest hasnt' been upstreamed yet?
Comment 4 Eric Seidel (no email) 2013-01-17 13:01:44 PST
Oh, this needs enable wraps.
Comment 5 Eric Seidel (no email) 2013-01-17 13:06:16 PST
Created attachment 183251 [details]
Patch for landing
Comment 6 WebKit Review Bot 2013-01-17 14:54:17 PST
Comment on attachment 183251 [details]
Patch for landing

Clearing flags on attachment: 183251

Committed r140050: <http://trac.webkit.org/changeset/140050>
Comment 7 WebKit Review Bot 2013-01-17 14:54:20 PST
All reviewed patches have been landed.  Closing bug.