Prevent GuestView from swapping process on cross-site navigations in --site-per-process.

Prevent GuestView from swapping process on cross-site navigations in --site-per-process.

BUG=330264
Committed: https://crrev.com/9fbf63023255c931d2903316dc5b645575015f36
Cr-Commit-Position: refs/heads/master@{#340470}

[lfg, 2015-07-24 19:00:04]

lfg@chromium.org changed reviewers:
+ creis@chromium.org

[lfg, 2015-07-24 19:00:04]

Hi Charlie, please take a look.

[Charlie Reis, 2015-07-24 19:20:32]

In the CL description, s/to swap process/from swapping processes/.

LGTM with nits.  Is there a test that this causes to pass on the FYI bots?  (It
only affects webview in --site-per-process, right?)

https://codereview.chromium.org/1259573006/diff/1/content/browser/loader/cros...
File content/browser/loader/cross_site_resource_handler.cc (right):

https://codereview.chromium.org/1259573006/diff/1/content/browser/loader/cros...
content/browser/loader/cross_site_resource_handler.cc:103: // For now, GuestView
never transfer on cross-site navigations.
nit: GuestViews

[lfg, 2015-07-24 22:05:45]

On 2015/07/24 19:20:32, Charlie Reis wrote:
> In the CL description, s/to swap process/from swapping processes/.
> 
> LGTM with nits.  Is there a test that this causes to pass on the FYI bots? 
(It
> only affects webview in --site-per-process, right?)

Yes, that function is only called in --site-per-process mode. We can't enable
the other tests yet -- The newwindow API still needs more work done in order to
enable the tests.

>
https://codereview.chromium.org/1259573006/diff/1/content/browser/loader/cros...
> File content/browser/loader/cross_site_resource_handler.cc (right):
> 
>
https://codereview.chromium.org/1259573006/diff/1/content/browser/loader/cros...
> content/browser/loader/cross_site_resource_handler.cc:103: // For now,
GuestView
> never transfer on cross-site navigations.
> nit: GuestViews

Done.

[lfg, 2015-07-27 14:50:38]

The CQ bit was checked by lfg@chromium.org

[lfg, 2015-07-27 14:50:38]

The patchset sent to the CQ was uploaded after l-g-t-m from creis@chromium.org
Link to the patchset: https://codereview.chromium.org/1259573006/#ps20001
(title: " ")

[commit-bot: I haz the power, 2015-07-27 14:51:02]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1259573006/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1259573006/20001

[commit-bot: I haz the power, 2015-07-27 15:41:50]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-07-27 15:43:22]

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