Add an extra check in NavigateRemoteFrameToBlankAndDataURLs.

Add an extra check in NavigateRemoteFrameToBlankAndDataURLs.

This CL adds an explicit navigation from within an iframe to about:blank.
It verifies that the iframe stays in the same process after navigation
has completed as the initiator of the navigation is the iframe itself.

BUG=591509
Committed: https://crrev.com/d948aa86b4ab06738c7d8c67fb13bf42d95a1ff3
Cr-Commit-Position: refs/heads/master@{#389862}

[nasko, 2016-04-26 16:23:14]

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

[nasko, 2016-04-26 16:23:15]

Hey Nick,
Can you review this CL for me? It adds an explicit test for an iframe navigating
itself to about:blank, which should stay in the same process.
Thanks in advance!
Nasko

[ncarter (slow), 2016-04-26 18:40:27]

lgtm

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

https://codereview.chromium.org/1917223002/diff/1/content/browser/site_per_pr...
content/browser/site_per_process_browsertest.cc:1385: DepictFrameTree(root));
lgtm

If you are working on improving the resolution of "about:blank", I think you may
wish to take a look at DISABLED_NavigateGrandchildToAboutBlank, in
FrameTreeBrowserTest. That's a case where we do ostensibly the wrong thing, see
the bug it mentions for details.

[nasko, 2016-04-26 19:53:30]

The CQ bit was checked by nasko@chromium.org

[commit-bot: I haz the power, 2016-04-26 19:53:53]

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

[commit-bot: I haz the power, 2016-04-26 19:59:01]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-26 20:00:04]

Description was changed from

==========
Add an extra check in NavigateRemoteFrameToBlankAndDataURLs.

This CL adds an explicit navigation from within an iframe to about:blank.
It verifies that the iframe stays in the same process after navigation
has completed as the initiator of the navigation is the iframe itself.

BUG=591509
==========

to

==========
Add an extra check in NavigateRemoteFrameToBlankAndDataURLs.

This CL adds an explicit navigation from within an iframe to about:blank.
It verifies that the iframe stays in the same process after navigation
has completed as the initiator of the navigation is the iframe itself.

BUG=591509

Committed: https://crrev.com/d948aa86b4ab06738c7d8c67fb13bf42d95a1ff3
Cr-Commit-Position: refs/heads/master@{#389862}
==========

[commit-bot: I haz the power, 2016-04-26 20:00:05]

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