Bug 108096

Summary:

Don't use threaded HTML parser for data: URLs

Product:

WebKit

Reporter:

Tony Gentilcore <tonyg>

Component:

New Bugs

Assignee:

Tony Gentilcore <tonyg>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

ojan.autocc, webkit.review.bot

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Bug Depends on:

Bug Blocks:

106127

Attachments:

Description	Flags
Patch	none
Patch for landing	none

Description Tony Gentilcore 2013-01-28 11:40:19 PST

Don't use threaded HTML parser for data: URLs

Comment 1 Tony Gentilcore 2013-01-28 11:41:53 PST

Created attachment 185023 [details]
Patch

Comment 2 Eric Seidel (no email) 2013-01-28 11:44:30 PST

Comment on attachment 185023 [details]
Patch

You should add a comment as to why, as this if is getting complicated enough to cause future confusion.

// We force the main-thread parser for about:blank, javascript: and data: urls for compatibility with historical synchronous loading/parsing behavior of those schemes.

Comment 3 Tony Gentilcore 2013-01-28 11:47:17 PST

Created attachment 185024 [details]
Patch for landing

Comment 4 WebKit Review Bot 2013-01-28 12:37:44 PST

Comment on attachment 185024 [details]
Patch for landing

Clearing flags on attachment: 185024

Committed r140992: <http://trac.webkit.org/changeset/140992>

Comment 5 WebKit Review Bot 2013-01-28 12:37:47 PST

All reviewed patches have been landed.  Closing bug.