Force synchronized parser for substitute main resource load marked forceSynchronousLoad.

Force synchronized parser for substitute main resource load marked forceSynchronousLoad.

Before this CL, synchronized parsing for forceSynchronousLoad
substituteData was not enforced. The flag was simply ignored in parsing
code path.

This bug has not surfaced because all substituteData which required
synchronized loading were used against about:blank URL, and
HTMLDocumentParserOptions had a special casing that about:blank urls
required synchronous parsing.

This CL explicitly enforces synchronous parse for forceSynchronousLoad
substituteData, so that flag is actually used.
This is also a step toward removing special casing against about:black
URL in HTMLDocumentParser.

BUG=421289
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=183570

[kouhei (in TOK), 2014-10-10 05:23:51]

kouhei@chromium.org changed reviewers:
+ eseidel@chromium.org, haraken@chromium.org

[kouhei (in TOK), 2014-10-10 05:25:20]

This is to address test failures on https://codereview.chromium.org/644573002/ ,
which removed special casing for about:blank urls. Would you take a look?

[kouhei (in TOK), 2014-10-10 07:07:32]

kouhei@chromium.org changed reviewers:
+ japhet@chromium.org

[Nate Chapin, 2014-10-10 23:41:48]

lgtm

[kouhei (in TOK), 2014-10-11 03:04:32]

Thanks for review!

[kouhei (in TOK), 2014-10-11 03:04:48]

The CQ bit was checked by kouhei@chromium.org

[commit-bot: I haz the power, 2014-10-11 03:05:19]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/646913002/1

[commit-bot: I haz the power, 2014-10-11 03:29:38]

Committed patchset #1 (id:1) as 183570