The security check in document.open should test for same-origin

The security check in document.open should test for same-origin

And not for same-origin domain, see
https://github.com/whatwg/html/issues/2282

R=mkwst@chromium.org
BUG=

Review-Url: https://codereview.chromium.org/2691083006
Cr-Commit-Position: refs/heads/master@{#450684}
Committed: https://chromium.googlesource.com/chromium/src/+/df05e7ac8ec9c43c03b8e066b3cf538c2c274cef

[jochen (gone - plz use gerrit), 2017-02-15 09:16:43]

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

[commit-bot: I haz the power, 2017-02-15 09:16:57]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Mike West, 2017-02-15 11:36:15]

https://codereview.chromium.org/2691083006/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/fast/parser/resources/document-open-in-unload-inner.html
(right):

https://codereview.chromium.org/2691083006/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/fast/parser/resources/document-open-in-unload-inner.html:2:
<iframe srcdoc="Hi"></iframe>
I don't understand how this tests the change. Did you consider pulling in the
test zcorpan@ suggested in
http://software.hixie.ch/utilities/js/live-dom-viewer/?saved=4829?

[commit-bot: I haz the power, 2017-02-15 12:02:26]

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

[commit-bot: I haz the power, 2017-02-15 12:02:27]

Dry run: This issue passed the CQ dry run.

[jochen (gone - plz use gerrit), 2017-02-15 12:13:50]

tests are here: https://github.com/w3c/web-platform-tests/pull/4642

[Mike West, 2017-02-15 12:57:23]

Ok. LGTM, assuming those tests pass after the patch.

[jochen (gone - plz use gerrit), 2017-02-15 13:16:05]

The CQ bit was checked by jochen@chromium.org

[commit-bot: I haz the power, 2017-02-15 13:16:30]

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-02-15 13:21:41]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1487164565447210, "parent_rev":
"3b2f56255b45fa7183f6048647b341db4b3fd50e", "commit_rev":
"df05e7ac8ec9c43c03b8e066b3cf538c2c274cef"}

[commit-bot: I haz the power, 2017-02-15 13:22:21]

Description was changed from

==========
The security check in document.open should test for same-origin

And not for same-origin domain, see
https://github.com/whatwg/html/issues/2282

R=mkwst@chromium.org
BUG=
==========

to

==========
The security check in document.open should test for same-origin

And not for same-origin domain, see
https://github.com/whatwg/html/issues/2282

R=mkwst@chromium.org
BUG=

Review-Url: https://codereview.chromium.org/2691083006
Cr-Commit-Position: refs/heads/master@{#450684}
Committed:
https://chromium.googlesource.com/chromium/src/+/df05e7ac8ec9c43c03b8e066b3cf...
==========

[commit-bot: I haz the power, 2017-02-15 13:22:22]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/df05e7ac8ec9c43c03b8e066b3cf...