Do not create proxy hosts in the subtree of navigating frame.

Do not create proxy hosts in the subtree of navigating frame.

This is an another attempt at landing https://codereview.chromium.org/536143002.

When a frame is navigating cross-process, its existing document will be completely destroyed and all child frames will be gone. This means that we don't need to create proxy objects for the new SiteInstance in the subtree of the navigating frame.

BUG=357747, 417030
Committed: https://crrev.com/e6edde315c6cb57cae5f67056191dceab3f7403b
Cr-Commit-Position: refs/heads/master@{#300112}

[nasko, 2014-09-24 21:34:44]

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

[nasko, 2014-09-24 21:34:44]

Hey Charlie,
This is an attempt to reland the change to avoid creating proxies in the subtree
that is about to navigate. It seems the asan bots passed and we are close to
fixing the UaF, so this should be able to go in fine.
Thanks in advance!
Nasko

[Charlie Reis, 2014-09-24 21:59:36]

LGTM

https://codereview.chromium.org/600483003/diff/1/content/browser/site_per_pro...
File content/browser/site_per_process_browsertest.cc (right):

https://codereview.chromium.org/600483003/diff/1/content/browser/site_per_pro...
content/browser/site_per_process_browsertest.cc:632: SiteInstance* site;
nit: Please initialize this to null.

[nasko, 2014-09-25 16:20:27]

https://codereview.chromium.org/600483003/diff/1/content/browser/site_per_pro...
File content/browser/site_per_process_browsertest.cc (right):

https://codereview.chromium.org/600483003/diff/1/content/browser/site_per_pro...
content/browser/site_per_process_browsertest.cc:632: SiteInstance* site;
On 2014/09/24 21:59:36, Charlie Reis wrote:
> nit: Please initialize this to null.

Done.

[nasko, 2014-10-17 15:09:49]

The CQ bit was checked by nasko@chromium.org

[commit-bot: I haz the power, 2014-10-17 15:10:50]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/600483003/100001

[commit-bot: I haz the power, 2014-10-17 15:37:07]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2014-10-17 15:38:02]

Patchset 6 (id:??) landed as
https://crrev.com/e6edde315c6cb57cae5f67056191dceab3f7403b
Cr-Commit-Position: refs/heads/master@{#300112}