Add test for validating the proper size when we exit WebVR to VR browsing.

Add test for validating the proper size when we exit WebVR to VR browsing.

This is an automated test to validate the fix for crbug/718578.

BUG=718578
Review-Url: https://codereview.chromium.org/2918853002
Cr-Commit-Position: refs/heads/master@{#476564}
Committed: https://chromium.googlesource.com/chromium/src/+/f7b41941ac3c3e7a42d743e5bbae6fe5024edee8

[billorr, 2017-06-01 20:46:30]

billorr@chromium.org changed reviewers:
+ bsheedy@chromium.org, mthiesse@chromium.org

[bsheedy, 2017-06-01 20:52:31]

LGTM with comment.

https://codereview.chromium.org/2918853002/diff/1/chrome/android/javatests/sr...
File
chrome/android/javatests/src/org/chromium/chrome/browser/vr_shell/VrShellNavigationTest.java
(right):

https://codereview.chromium.org/2918853002/diff/1/chrome/android/javatests/sr...
chrome/android/javatests/src/org/chromium/chrome/browser/vr_shell/VrShellNavigationTest.java:273:
mVrTestRule.loadUrl(TEST_PAGE_WEBVR_URL, PAGE_LOAD_TIMEOUT_S);
Should be loadUrlAndAwaitInitialization to make sure that the getVRDisplays
promise resolves before trying to present.

https://codereview.chromium.org/2918853002/diff/1/chrome/android/javatests/sr...
chrome/android/javatests/src/org/chromium/chrome/browser/vr_shell/VrShellNavigationTest.java:285:
// Exit presentation through javascript.
Nit: JavaScript

[mthiesse, 2017-06-01 21:10:29]

lgtm

[billorr, 2017-06-02 01:07:55]

The CQ bit was checked by billorr@chromium.org

[billorr, 2017-06-02 01:07:55]

The patchset sent to the CQ was uploaded after l-g-t-m from
bsheedy@chromium.org, mthiesse@chromium.org
Link to the patchset: https://codereview.chromium.org/2918853002/#ps20001
(title: "CR feedback")

[commit-bot: I haz the power, 2017-06-02 01:08:43]

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-06-02 04:30:41]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1496365675520900,
"parent_rev": "854d102a9172d20548a8e8a1b5310e02dfeb9759", "commit_rev":
"5b1c98e8bbcaf9470a8e09db96ede9d233df7a3c"}

[commit-bot: I haz the power, 2017-06-02 04:30:58]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1496365675520900,
"parent_rev": "cf6afdd5f84d54d01df7f3145a0a8e2082b4df58", "commit_rev":
"f7b41941ac3c3e7a42d743e5bbae6fe5024edee8"}

[commit-bot: I haz the power, 2017-06-02 04:31:06]

Description was changed from

==========
Add test for validating the proper size when we exit WebVR to VR browsing.

This is an automated test to validate the fix for crbug/718578.

BUG=718578
==========

to

==========
Add test for validating the proper size when we exit WebVR to VR browsing.

This is an automated test to validate the fix for crbug/718578.

BUG=718578

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

[commit-bot: I haz the power, 2017-06-02 04:31:07]

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

[tiborg, 2017-06-02 15:36:51]

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

[tiborg, 2017-06-02 15:36:52]

https://codereview.chromium.org/2918853002/diff/20001/chrome/android/javatest...
File
chrome/android/javatests/src/org/chromium/chrome/browser/vr_shell/VrShellNavigationTest.java
(right):

https://codereview.chromium.org/2918853002/diff/20001/chrome/android/javatest...
chrome/android/javatests/src/org/chromium/chrome/browser/vr_shell/VrShellNavigationTest.java:271:
public void testExitPresentationWebVrToVrShell()
This test should probably not be here. VrShellTest would be better. Or a new
test where we test presentation <-> non-presentation, fullscreen <->
non-fullscreen kind of tests.