v8bindings: Reverts crrev.com/2606723002 with minimum changes.

v8bindings: Reverts crrev.com/2606723002 with minimum changes.

Allows named access on window regardless of same/cross origin
nor mismatch between browsing context name and browsing context
container's name.

Since crrev.com/2606723002 caused crbug.com/701489, we revert
the CL with minimum changes because the entire revert is hard.

BUG=538562, 701489
Review-Url: https://codereview.chromium.org/2753773003
Cr-Commit-Position: refs/heads/master@{#457138}
Committed: https://chromium.googlesource.com/chromium/src/+/8633d64a9677342d7d704e24068f4838a2d2c88a

[Yuki, 2017-03-15 10:16:43]

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

[commit-bot: I haz the power, 2017-03-15 10:16:54]

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-03-15 10:41:30]

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

[commit-bot: I haz the power, 2017-03-15 10:41:31]

Dry run: Try jobs failed on following builders:
  linux_chromium_tsan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Yuki, 2017-03-15 12:40:16]

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

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

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-03-15 14:04:50]

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

[commit-bot: I haz the power, 2017-03-15 14:04:51]

Dry run: This issue passed the CQ dry run.

[domenic, 2017-03-15 17:51:00]

domenic@chromium.org changed reviewers:
+ domenic@chromium.org

[domenic, 2017-03-15 17:51:01]

LGTM, seems like a minimal change that even increases our pass rate in WPT
expectations, and clearly documents the path forward.

Hopefully implementing the full spec/passing all web platform tests is still web
compatible. But if not we're happy to revise the spec, just let us know.

[alexmos, 2017-03-15 17:58:47]

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

[alexmos, 2017-03-15 17:58:48]

content/ LGTM

[dcheng, 2017-03-15 18:00:36]

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

[dcheng, 2017-03-15 18:00:36]

CQing this on yukishiino's behalf so we can merge this.

[dcheng, 2017-03-15 18:00:42]

The CQ bit was checked by dcheng@chromium.org

[commit-bot: I haz the power, 2017-03-15 18:01:19]

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-03-15 18:06:39]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1489600842135920,
"parent_rev": "ebf283a6e622fdd02d0ab9ee2386dfa50ebcc4e3", "commit_rev":
"8633d64a9677342d7d704e24068f4838a2d2c88a"}

[commit-bot: I haz the power, 2017-03-15 18:07:27]

Description was changed from

==========
v8bindings: Reverts crrev.com/2606723002 with minimum changes.

Allows named access on window regardless of same/cross origin
nor mismatch between browsing context name and browsing context
container's name.

Since crrev.com/2606723002 caused crbug.com/701489, we revert
the CL with minimum changes because the entire revert is hard.

BUG=538562,701489
==========

to

==========
v8bindings: Reverts crrev.com/2606723002 with minimum changes.

Allows named access on window regardless of same/cross origin
nor mismatch between browsing context name and browsing context
container's name.

Since crrev.com/2606723002 caused crbug.com/701489, we revert
the CL with minimum changes because the entire revert is hard.

BUG=538562,701489

Review-Url: https://codereview.chromium.org/2753773003
Cr-Commit-Position: refs/heads/master@{#457138}
Committed:
https://chromium.googlesource.com/chromium/src/+/8633d64a9677342d7d704e24068f...
==========

[commit-bot: I haz the power, 2017-03-15 18:07:28]

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

[haraken, 2017-03-15 20:08:14]

LGTM