RESOLVED FIXED 109760
HTMLTokenScanner should be (mostly!) thread-safe
https://bugs.webkit.org/show_bug.cgi?id=109760
Summary HTMLTokenScanner should be (mostly!) thread-safe
Adam Barth
Reported 2013-02-13 15:45:36 PST
HTMLTokenScanner should be (mostly!) thread-safe
Attachments
Patch (9.30 KB, patch)
2013-02-13 15:47 PST, Adam Barth
no flags
Patch for landing (8.81 KB, patch)
2013-02-13 18:22 PST, Adam Barth
no flags
Adam Barth
Comment 1 2013-02-13 15:47:50 PST
Adam Barth
Comment 2 2013-02-13 15:55:51 PST
This patch will need to be re-based slightly since one of the early patches changed some of these names.
Eric Seidel (no email)
Comment 3 2013-02-13 16:14:32 PST
Comment on attachment 188201 [details] Patch OK.
WebKit Review Bot
Comment 4 2013-02-13 16:40:41 PST
Comment on attachment 188201 [details] Patch Rejecting attachment 188201 [details] from commit-queue. Failed to run "['/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '--bot-id=gce-cq-04', 'apply-attachment', '--no-update', '--non-interactive', 188201, '--port=chromium-xvfb']" exit_code: 2 cwd: /mnt/git/webkit-commit-queue Last 500 characters of output: eeded at 218 with fuzz 2 (offset -131 lines). 10 out of 11 hunks FAILED -- saving rejects to file Source/WebCore/html/parser/HTMLPreloadScanner.cpp.rej patching file Source/WebCore/html/parser/HTMLPreloadScanner.h Hunk #2 FAILED at 48. 1 out of 2 hunks FAILED -- saving rejects to file Source/WebCore/html/parser/HTMLPreloadScanner.h.rej Failed to run "[u'/mnt/git/webkit-commit-queue/Tools/Scripts/svn-apply', '--force', '--reviewer', 'Eric Seidel']" exit_code: 1 cwd: /mnt/git/webkit-commit-queue Full output: http://queues.webkit.org/results/16530379
Adam Barth
Comment 5 2013-02-13 18:22:27 PST
Created attachment 188246 [details] Patch for landing
WebKit Review Bot
Comment 6 2013-02-13 19:33:38 PST
Comment on attachment 188246 [details] Patch for landing Clearing flags on attachment: 188246 Committed r142843: <http://trac.webkit.org/changeset/142843>
WebKit Review Bot
Comment 7 2013-02-13 19:33:41 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.