Fix site-per-process layout tests timeouts due to document.write().

Fix site-per-process layout tests timeouts due to document.write().

See detailed analysis in the bug.  When the document.write() happened
after document load, they implicitly called document.open() and
dispatched another DidStartLoading, which might have prevented the
main page (and the test) from finishing loading if it wasn't yet
loaded at that point (due to the OOP subframe not yet loaded).  Add a
document.close() so that there's always going to be a matching
DidStopLoading().

BUG=622764
Committed: https://crrev.com/4add5ff9d0fc7baec422201850dbfa829f516fb4
Cr-Commit-Position: refs/heads/master@{#401743}

[alexmos, 2016-06-23 16:45:10]

alexmos@chromium.org changed reviewers:
+ dcheng@chromium.org

[alexmos, 2016-06-23 16:45:10]

Daniel, can you please take a look?

[dcheng, 2016-06-23 18:28:32]

The use of document.write here is questionable (don't we have a js-test.js
thingie with logging helpers?), but the fix lgtm

[alexmos, 2016-06-23 22:11:11]

The CQ bit was checked by alexmos@chromium.org

[commit-bot: I haz the power, 2016-06-23 22:11:37]

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

[commit-bot: I haz the power, 2016-06-23 22:16:53]

Committed patchset #1 (id:1)

[alexmos, 2016-06-23 22:17:19]

On 2016/06/23 18:28:32, dcheng wrote:
> The use of document.write here is questionable (don't we have a js-test.js
> thingie with logging helpers?), but the fix lgtm

Thanks.  Yeah, there are a bunch of other ways we could do this without
document.write (seems like testPassed() in js-test.js is one way), but I
actually don't mind keeping this as-is to keep some document.write coverage in
OOPIFs.

[commit-bot: I haz the power, 2016-06-23 22:18:50]

Description was changed from

==========
Fix site-per-process layout tests timeouts due to document.write().

See detailed analysis in the bug.  When the document.write() happened
after document load, they implicitly called document.open() and
dispatched another DidStartLoading, which might have prevented the
main page (and the test) from finishing loading if it wasn't yet
loaded at that point (due to the OOP subframe not yet loaded).  Add a
document.close() so that there's always going to be a matching
DidStopLoading().

BUG=622764
==========

to

==========
Fix site-per-process layout tests timeouts due to document.write().

See detailed analysis in the bug.  When the document.write() happened
after document load, they implicitly called document.open() and
dispatched another DidStartLoading, which might have prevented the
main page (and the test) from finishing loading if it wasn't yet
loaded at that point (due to the OOP subframe not yet loaded).  Add a
document.close() so that there's always going to be a matching
DidStopLoading().

BUG=622764

Committed: https://crrev.com/4add5ff9d0fc7baec422201850dbfa829f516fb4
Cr-Commit-Position: refs/heads/master@{#401743}
==========

[commit-bot: I haz the power, 2016-06-23 22:18:51]

Patchset 1 (id:??) landed as
https://crrev.com/4add5ff9d0fc7baec422201850dbfa829f516fb4
Cr-Commit-Position: refs/heads/master@{#401743}