Don't lower priority for scripts inserted by doc.write

Don't lower priority for scripts inserted by doc.write

HTML Chunks inserted by document.write() are run through a separate
instance of the preload scanner from the main document.  If the main
document scanner has already discovered an image then all scripts
discovered by any preload scanner regardless of where they came from
would be considered late-body.

This behavior causes a regression where a document.write() in the head
that includes multiple script tags will discover them but treat them as
late-body scripts and only load them one at a time.

reland of http://crbug.com/2826213003

Only change from the original was to fix the layout test.  Specifically,
the URL for the preload image was not re-used for the actual image
which caused the test to issue a console warning about the preload
not being used but only if the overall test took long enough for the
warning to trigger.

Patch set #1 is the original CL untouched

BUG=713727, 712338
Review-Url: https://codereview.chromium.org/2826213003
Cr-Commit-Position: refs/heads/master@{#467029}
Committed: https://chromium.googlesource.com/chromium/src/+/1e0d3cbbf60c67ba9856eadcb9e829b3c4bec6c7

patch from issue 2826213003 at patchset 70001 (http://crrev.com/2826213003#ps70001)

Review-Url: https://codereview.chromium.org/2841363002
Cr-Commit-Position: refs/heads/master@{#467672}
Committed: https://chromium.googlesource.com/chromium/src/+/591cbec71ef07912c8c7d2b0acc1efb25196d7f1

[Pat Meenan, 2017-04-27 01:40:33]

The CQ bit was checked by pmeenan@chromium.org to run a CQ dry run

[Pat Meenan, 2017-04-27 01:41:26]

pmeenan@chromium.org changed reviewers:
+ kinuko@chromium.org

[Pat Meenan, 2017-04-27 01:41:27]

kinuko@ - here is an updated CL that has both the original and changes (sorry
about not doing it this way the first time).

[commit-bot: I haz the power, 2017-04-27 01:41:41]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-27 03:23:33]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-04-27 03:23:34]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[kinuko, 2017-04-27 06:52:02]

lgtm

[Pat Meenan, 2017-04-27 12:35:47]

The CQ bit was checked by pmeenan@chromium.org

[commit-bot: I haz the power, 2017-04-27 12:36:05]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-27 14:41:32]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1493296547504300,
"parent_rev": "7af20094a387dfa6a3f0eb5fb5b15066bdf90f19", "commit_rev":
"591cbec71ef07912c8c7d2b0acc1efb25196d7f1"}

[commit-bot: I haz the power, 2017-04-27 14:41:43]

Description was changed from

==========
Don't lower priority for scripts inserted by doc.write

HTML Chunks inserted by document.write() are run through a separate
instance of the preload scanner from the main document.  If the main
document scanner has already discovered an image then all scripts
discovered by any preload scanner regardless of where they came from
would be considered late-body.

This behavior causes a regression where a document.write() in the head
that includes multiple script tags will discover them but treat them as
late-body scripts and only load them one at a time.

reland of http://crbug.com/2826213003

Only change from the original was to fix the layout test.  Specifically,
the URL for the preload image was not re-used for the actual image
which caused the test to issue a console warning about the preload
not being used but only if the overall test took long enough for the
warning to trigger.

Patch set #1 is the original CL untouched

BUG=713727, 712338
Review-Url: https://codereview.chromium.org/2826213003
Cr-Commit-Position: refs/heads/master@{#467029}
Committed:
https://chromium.googlesource.com/chromium/src/+/1e0d3cbbf60c67ba9856eadcb9e8...

patch from issue 2826213003 at patchset 70001
(http://crrev.com/2826213003#ps70001)
==========

to

==========
Don't lower priority for scripts inserted by doc.write

HTML Chunks inserted by document.write() are run through a separate
instance of the preload scanner from the main document.  If the main
document scanner has already discovered an image then all scripts
discovered by any preload scanner regardless of where they came from
would be considered late-body.

This behavior causes a regression where a document.write() in the head
that includes multiple script tags will discover them but treat them as
late-body scripts and only load them one at a time.

reland of http://crbug.com/2826213003

Only change from the original was to fix the layout test.  Specifically,
the URL for the preload image was not re-used for the actual image
which caused the test to issue a console warning about the preload
not being used but only if the overall test took long enough for the
warning to trigger.

Patch set #1 is the original CL untouched

BUG=713727, 712338
Review-Url: https://codereview.chromium.org/2826213003
Cr-Commit-Position: refs/heads/master@{#467029}
Committed:
https://chromium.googlesource.com/chromium/src/+/1e0d3cbbf60c67ba9856eadcb9e8...

patch from issue 2826213003 at patchset 70001
(http://crrev.com/2826213003#ps70001)

Review-Url: https://codereview.chromium.org/2841363002
Cr-Commit-Position: refs/heads/master@{#467672}
Committed:
https://chromium.googlesource.com/chromium/src/+/591cbec71ef07912c8c7d2b0acc1...
==========

[commit-bot: I haz the power, 2017-04-27 14:41:44]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/591cbec71ef07912c8c7d2b0acc1...