Fix MediaGalleriesPlatformAppBrowserTest tests with PlzNavigate.

Fix MediaGalleriesPlatformAppBrowserTest tests with PlzNavigate.

r217066 added a dummy app window to keep the app alive during the test. This gives errors in the chrome apps JS because the file doesn't actually exist, so the chrome apps JS objects aren't injected. This works without PlzNavigate because the JS objects are injected based on the provisional load URL, which is valid (this is before the renderer finds out the file doesn't exist). With PlzNavigate, the frame is instead navigated to an error page URL, which we don't inject chrome apps bindings in.

Fix this by increasing the apps timeout value in C++ in the test harness.

This fixes
MediaGalleriesPlatformAppBrowserTest.MediaGalleriesRead
MediaGalleriesPlatformAppBrowserTest.Scan
MediaGalleriesPlatformAppBrowserTest.ToURL
with PlzNavigate.

BUG=504347
Committed: https://crrev.com/9f396cd685f05bf2d4d9f86c7e3d0e0ed85790f8
Cr-Commit-Position: refs/heads/master@{#429477}

[jam, 2016-11-02 19:31:10]

Description was changed from

==========
Fix MediaGalleriesPlatformAppBrowserTest tests with PlzNavigate.

r217066 added a dummy app window to keep the app alive during the test. This
gives errors in the chrome apps JS because the file doesn't actually exist, so
the chrome apps JS objects aren't injected. This works without PlzNavigate
because the JS objects are injected based on the provisional load URL, which is
valid (this is before the renderer finds out the file doesn't exist). With
PlzNavigate, the frame is instead navigated to an error page URL, which we don't
inject chrome apps bindings in.

This fixes
MediaGalleriesPlatformAppBrowserTest.MediaGalleriesRead
MediaGalleriesPlatformAppBrowserTest.Scan
MediaGalleriesPlatformAppBrowserTest.ToURL
with PlzNavigate.

BUG=504347
==========

to

==========
Fix MediaGalleriesPlatformAppBrowserTest tests with PlzNavigate.

r217066 added a dummy app window to keep the app alive during the test. This
gives errors in the chrome apps JS because the file doesn't actually exist, so
the chrome apps JS objects aren't injected. This works without PlzNavigate
because the JS objects are injected based on the provisional load URL, which is
valid (this is before the renderer finds out the file doesn't exist). With
PlzNavigate, the frame is instead navigated to an error page URL, which we don't
inject chrome apps bindings in.

This fixes
MediaGalleriesPlatformAppBrowserTest.MediaGalleriesRead
MediaGalleriesPlatformAppBrowserTest.Scan
MediaGalleriesPlatformAppBrowserTest.ToURL
with PlzNavigate.

BUG=504347
==========

[jam, 2016-11-02 19:31:10]

jam@chromium.org changed reviewers:
+ tommycli@chromium.org

[jam, 2016-11-02 19:41:57]

Patchset #1 (id:1) has been deleted

[jam, 2016-11-02 19:43:14]

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

[jam, 2016-11-02 19:43:34]

Description was changed from

==========
Fix MediaGalleriesPlatformAppBrowserTest tests with PlzNavigate.

r217066 added a dummy app window to keep the app alive during the test. This
gives errors in the chrome apps JS because the file doesn't actually exist, so
the chrome apps JS objects aren't injected. This works without PlzNavigate
because the JS objects are injected based on the provisional load URL, which is
valid (this is before the renderer finds out the file doesn't exist). With
PlzNavigate, the frame is instead navigated to an error page URL, which we don't
inject chrome apps bindings in.

This fixes
MediaGalleriesPlatformAppBrowserTest.MediaGalleriesRead
MediaGalleriesPlatformAppBrowserTest.Scan
MediaGalleriesPlatformAppBrowserTest.ToURL
with PlzNavigate.

BUG=504347
==========

to

==========
Fix MediaGalleriesPlatformAppBrowserTest tests with PlzNavigate.

r217066 added a dummy app window to keep the app alive during the test. This
gives errors in the chrome apps JS because the file doesn't actually exist, so
the chrome apps JS objects aren't injected. This works without PlzNavigate
because the JS objects are injected based on the provisional load URL, which is
valid (this is before the renderer finds out the file doesn't exist). With
PlzNavigate, the frame is instead navigated to an error page URL, which we don't
inject chrome apps bindings in.

Fix this by increasing the apps timeout value in C++ in the test harness.

This fixes
MediaGalleriesPlatformAppBrowserTest.MediaGalleriesRead
MediaGalleriesPlatformAppBrowserTest.Scan
MediaGalleriesPlatformAppBrowserTest.ToURL
with PlzNavigate.

BUG=504347
==========

[commit-bot: I haz the power, 2016-11-02 19:43:55]

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, 2016-11-02 20:59:04]

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

[commit-bot: I haz the power, 2016-11-02 20:59:04]

Dry run: This issue passed the CQ dry run.

[tommycli, 2016-11-03 00:22:06]

if indeed we can get rid of the dummy window, then great! lgtm

[jam, 2016-11-03 00:24:14]

The CQ bit was checked by jam@chromium.org

[commit-bot: I haz the power, 2016-11-03 00:24:52]

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, 2016-11-03 00:34:25]

Description was changed from

==========
Fix MediaGalleriesPlatformAppBrowserTest tests with PlzNavigate.

r217066 added a dummy app window to keep the app alive during the test. This
gives errors in the chrome apps JS because the file doesn't actually exist, so
the chrome apps JS objects aren't injected. This works without PlzNavigate
because the JS objects are injected based on the provisional load URL, which is
valid (this is before the renderer finds out the file doesn't exist). With
PlzNavigate, the frame is instead navigated to an error page URL, which we don't
inject chrome apps bindings in.

Fix this by increasing the apps timeout value in C++ in the test harness.

This fixes
MediaGalleriesPlatformAppBrowserTest.MediaGalleriesRead
MediaGalleriesPlatformAppBrowserTest.Scan
MediaGalleriesPlatformAppBrowserTest.ToURL
with PlzNavigate.

BUG=504347
==========

to

==========
Fix MediaGalleriesPlatformAppBrowserTest tests with PlzNavigate.

r217066 added a dummy app window to keep the app alive during the test. This
gives errors in the chrome apps JS because the file doesn't actually exist, so
the chrome apps JS objects aren't injected. This works without PlzNavigate
because the JS objects are injected based on the provisional load URL, which is
valid (this is before the renderer finds out the file doesn't exist). With
PlzNavigate, the frame is instead navigated to an error page URL, which we don't
inject chrome apps bindings in.

Fix this by increasing the apps timeout value in C++ in the test harness.

This fixes
MediaGalleriesPlatformAppBrowserTest.MediaGalleriesRead
MediaGalleriesPlatformAppBrowserTest.Scan
MediaGalleriesPlatformAppBrowserTest.ToURL
with PlzNavigate.

BUG=504347
==========

[commit-bot: I haz the power, 2016-11-03 00:34:26]

Committed patchset #1 (id:20001)

[commit-bot: I haz the power, 2016-11-03 00:36:58]

Description was changed from

==========
Fix MediaGalleriesPlatformAppBrowserTest tests with PlzNavigate.

r217066 added a dummy app window to keep the app alive during the test. This
gives errors in the chrome apps JS because the file doesn't actually exist, so
the chrome apps JS objects aren't injected. This works without PlzNavigate
because the JS objects are injected based on the provisional load URL, which is
valid (this is before the renderer finds out the file doesn't exist). With
PlzNavigate, the frame is instead navigated to an error page URL, which we don't
inject chrome apps bindings in.

Fix this by increasing the apps timeout value in C++ in the test harness.

This fixes
MediaGalleriesPlatformAppBrowserTest.MediaGalleriesRead
MediaGalleriesPlatformAppBrowserTest.Scan
MediaGalleriesPlatformAppBrowserTest.ToURL
with PlzNavigate.

BUG=504347
==========

to

==========
Fix MediaGalleriesPlatformAppBrowserTest tests with PlzNavigate.

r217066 added a dummy app window to keep the app alive during the test. This
gives errors in the chrome apps JS because the file doesn't actually exist, so
the chrome apps JS objects aren't injected. This works without PlzNavigate
because the JS objects are injected based on the provisional load URL, which is
valid (this is before the renderer finds out the file doesn't exist). With
PlzNavigate, the frame is instead navigated to an error page URL, which we don't
inject chrome apps bindings in.

Fix this by increasing the apps timeout value in C++ in the test harness.

This fixes
MediaGalleriesPlatformAppBrowserTest.MediaGalleriesRead
MediaGalleriesPlatformAppBrowserTest.Scan
MediaGalleriesPlatformAppBrowserTest.ToURL
with PlzNavigate.

BUG=504347

Committed: https://crrev.com/9f396cd685f05bf2d4d9f86c7e3d0e0ed85790f8
Cr-Commit-Position: refs/heads/master@{#429477}
==========

[commit-bot: I haz the power, 2016-11-03 00:36:59]

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