Allow webview guests to skip WAR checks in ShouldAllowOpenURL (Merge to M58)

Allow webview guests to skip WAR checks in ShouldAllowOpenURL (Merge to M58)

Prior to r429532, the web-accessible resource checks in
ShouldAllowOpenURL were skipped for all schemes other than
HTTP/HTTPS/chrome-extension.  After r429532, we've started enforcing
them for most other schemes, but this breaks navigations from webview
guests to webview-accessible app resources whenever those go through
the OpenURL path, since the webview-accessible resource check is done
separately later, in AllowCrossRendererResourceLoadHelper, and
ShouldAllowOpenURL was killing the navigation before it ever got
there.

This CL will return webview guests to the previous behavior.

BUG=691941
Review-Url: https://codereview.chromium.org/2766313002
Cr-Commit-Position: refs/heads/master@{#458959}
(cherry picked from commit f89e04046aaac44ecad27056fb856c3d6e37370f)

Review-Url: https://codereview.chromium.org/2781783002 .
Cr-Commit-Position: refs/branch-heads/3029@{#447}
Cr-Branched-From: 939b32ee5ba05c396eef3fd992822fcca9a2e262-refs/heads/master@{#454471}
Committed: https://chromium.googlesource.com/chromium/src/+/32ec47289bec2eb6327723f491182e9f1c9a6103

[alexmos, 2017-03-28 00:34:51]

Description was changed from

==========
Allow webview guests to skip WAR checks in ShouldAllowOpenURL (Merge to M58)

Prior to r429532, the web-accessible resource checks in
ShouldAllowOpenURL were skipped for all schemes other than
HTTP/HTTPS/chrome-extension.  After r429532, we've started enforcing
them for most other schemes, but this breaks navigations from webview
guests to webview-accessible app resources whenever those go through
the OpenURL path, since the webview-accessible resource check is done
separately later, in AllowCrossRendererResourceLoadHelper, and
ShouldAllowOpenURL was killing the navigation before it ever got
there.

This CL will return webview guests to the previous behavior.

BUG=691941

Review-Url: https://codereview.chromium.org/2766313002
Cr-Commit-Position: refs/heads/master@{#458959}
(cherry picked from commit f89e04046aaac44ecad27056fb856c3d6e37370f)
==========

to

==========
Allow webview guests to skip WAR checks in ShouldAllowOpenURL (Merge to M58)

Prior to r429532, the web-accessible resource checks in
ShouldAllowOpenURL were skipped for all schemes other than
HTTP/HTTPS/chrome-extension.  After r429532, we've started enforcing
them for most other schemes, but this breaks navigations from webview
guests to webview-accessible app resources whenever those go through
the OpenURL path, since the webview-accessible resource check is done
separately later, in AllowCrossRendererResourceLoadHelper, and
ShouldAllowOpenURL was killing the navigation before it ever got
there.

This CL will return webview guests to the previous behavior.

BUG=691941

Review-Url: https://codereview.chromium.org/2766313002
Cr-Commit-Position: refs/heads/master@{#458959}
(cherry picked from commit f89e04046aaac44ecad27056fb856c3d6e37370f)

Review-Url: https://codereview.chromium.org/2781783002 .
Cr-Commit-Position: refs/branch-heads/3029@{#447}
Cr-Branched-From:
939b32ee5ba05c396eef3fd992822fcca9a2e262-refs/heads/master@{#454471}
Committed:
https://chromium.googlesource.com/chromium/src/+/32ec47289bec2eb6327723f49118...
==========

[alexmos, 2017-03-28 00:34:52]

Committed patchset #2 (id:20001) manually as
32ec47289bec2eb6327723f491182e9f1c9a6103.