Bug 107071 - Introduce HTMLParserThread to be able to parse on a background thread
Summary: Introduce HTMLParserThread to be able to parse on a background thread
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Adam Barth
URL:
Keywords:
Depends on:
Blocks: 106127
  Show dependency treegraph
 
Reported: 2013-01-16 17:43 PST by Adam Barth
Modified: 2013-01-16 18:06 PST (History)
3 users (show)

See Also:


Attachments
Patch (22.96 KB, patch)
2013-01-16 17:46 PST, Adam Barth
no flags Details | Formatted Diff | Diff
Patch for landing (25.31 KB, patch)
2013-01-16 18:01 PST, Adam Barth
no flags Details | Formatted Diff | Diff
Patch for landing (23.11 KB, patch)
2013-01-16 18:02 PST, Adam Barth
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Adam Barth 2013-01-16 17:43:46 PST
#include <wtf/Functional.h>
Comment 1 Adam Barth 2013-01-16 17:46:26 PST
Created attachment 183072 [details]
Patch
Comment 2 Tony Gentilcore 2013-01-16 17:51:48 PST
Comment on attachment 183072 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=183072&action=review

lg with nits

> Source/WebCore/html/parser/HTMLParserThread.cpp:100
> +#endif // ENABLE(BLOB)

Copy/paste error here

> Source/WebCore/html/parser/HTMLParserThread.h:36
> +#include <stdio.h>

I suspect this is no longer necessary and was around for an old printf or something.
Comment 3 Adam Barth 2013-01-16 18:01:09 PST
Created attachment 183077 [details]
Patch for landing
Comment 4 Adam Barth 2013-01-16 18:02:03 PST
Created attachment 183078 [details]
Patch for landing
Comment 5 Adam Barth 2013-01-16 18:06:49 PST
Committed r139944: <http://trac.webkit.org/changeset/139944>