107519 – Turn on ENABLE_THREADED_HTML_PARSER for Chromium (it's still disabled at runtime)

Bug 107519 - Turn on ENABLE_THREADED_HTML_PARSER for Chromium (it's still disabled at runtime)

Summary: Turn on ENABLE_THREADED_HTML_PARSER for Chromium (it's still disabled at runt...

Attachments
Patch (1.49 KB, patch) 2013-01-22 01:28 PST, Eric Seidel (no email)	no flags	Details \| Formatted Diff \| Diff
Patch (3.81 KB, patch) 2013-01-22 01:40 PST, Eric Seidel (no email)	no flags	Details \| Formatted Diff \| Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Eric Seidel (no email) 2013-01-22 01:26:20 PST

Turn on ENABLE_THREADED_HTML_PARSER for Chromium (it's still disabled at runtime)

Comment 1 Eric Seidel (no email) 2013-01-22 01:28:19 PST

Created attachment 183924 [details]
Patch

Comment 2 Adam Barth 2013-01-22 01:30:14 PST

Comment on attachment 183924 [details]
Patch

I presume we don't start the thread if the runtime switch is off.

Comment 3 Eric Seidel (no email) 2013-01-22 01:37:00 PST

I'll add an ASSERT to be double-sure.

Comment 4 Eric Seidel (no email) 2013-01-22 01:40:15 PST

Created attachment 183928 [details]
Patch

Comment 5 Eric Seidel (no email) 2013-01-22 01:50:06 PST

Adding the gardeners as an FYI. I do not expect this to cause any change in behavior.

Comment 6 WebKit Review Bot 2013-01-22 02:25:58 PST

Comment on attachment 183928 [details]
Patch

Clearing flags on attachment: 183928

Committed r140402: <http://trac.webkit.org/changeset/140402>

Comment 7 WebKit Review Bot 2013-01-22 02:26:03 PST

All reviewed patches have been landed.  Closing bug.