EG test for sessionStorage sharing.

EG test for sessionStorage sharing.

Newly written test reproduces crbug.com/681434 and currently disabled
because of this bug.

BUG=681434
Review-Url: https://codereview.chromium.org/2672893003
Cr-Commit-Position: refs/heads/master@{#448042}
Committed: https://chromium.googlesource.com/chromium/src/+/31501962d5fe6c4e6729a6f6aa14c96170bdb6ce

[Eugene But (OOO till 7-30), 2017-02-02 22:26:50]

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

[Eugene But (OOO till 7-30), 2017-02-02 22:27:01]

eugenebut@chromium.org changed reviewers:
+ baxley@chromium.org

[commit-bot: I haz the power, 2017-02-02 22:27: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-02-02 23:41:19]

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

[commit-bot: I haz the power, 2017-02-02 23:41:20]

Dry run: This issue passed the CQ dry run.

[baxley, 2017-02-03 15:23:38]

LGTM

Thank you for writing an automated test case for a bug!

https://codereview.chromium.org/2672893003/diff/20001/ios/chrome/browser/web/...
File ios/chrome/browser/web/window_open_by_dom_egtest.mm (right):

https://codereview.chromium.org/2672893003/diff/20001/ios/chrome/browser/web/...
ios/chrome/browser/web/window_open_by_dom_egtest.mm:63: // Tests that
|sessionStorage| content is available for windows open by DOM via
nit: I don't quite understand "windows open by DOM", should it be "windows
opened by DOM"? Or are these technical terms that should be written that way?

[Eugene But (OOO till 7-30), 2017-02-03 19:04:23]

Thanks!

https://codereview.chromium.org/2672893003/diff/20001/ios/chrome/browser/web/...
File ios/chrome/browser/web/window_open_by_dom_egtest.mm (right):

https://codereview.chromium.org/2672893003/diff/20001/ios/chrome/browser/web/...
ios/chrome/browser/web/window_open_by_dom_egtest.mm:63: // Tests that
|sessionStorage| content is available for windows open by DOM via
On 2017/02/03 15:23:38, baxley wrote:
> nit: I don't quite understand "windows open by DOM", should it be "windows
> opened by DOM"? Or are these technical terms that should be written that way?
Yes, it should be "windows opened by DOM". Done.

[Eugene But (OOO till 7-30), 2017-02-03 19:04:26]

The CQ bit was checked by eugenebut@chromium.org

[Eugene But (OOO till 7-30), 2017-02-03 19:04:26]

The patchset sent to the CQ was uploaded after l-g-t-m from baxley@chromium.org
Link to the patchset: https://codereview.chromium.org/2672893003/#ps40001
(title: "Addressed review comment")

[commit-bot: I haz the power, 2017-02-03 19:05:25]

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-03 19:25:38]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1486148666226310,
"parent_rev": "7da96c48c351c6fd61c73a28ca94e15680dc24d1", "commit_rev":
"31501962d5fe6c4e6729a6f6aa14c96170bdb6ce"}

[commit-bot: I haz the power, 2017-02-03 19:26:08]

Description was changed from

==========
EG test for sessionStorage sharing.

Newly written test reproduces crbug.com/681434 and currently disabled
because of this bug.

BUG=681434
==========

to

==========
EG test for sessionStorage sharing.

Newly written test reproduces crbug.com/681434 and currently disabled
because of this bug.

BUG=681434

Review-Url: https://codereview.chromium.org/2672893003
Cr-Commit-Position: refs/heads/master@{#448042}
Committed:
https://chromium.googlesource.com/chromium/src/+/31501962d5fe6c4e6729a6f6aa14...
==========

[commit-bot: I haz the power, 2017-02-03 19:26:10]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/31501962d5fe6c4e6729a6f6aa14...