Test theory about didCommitProvisionalLoad crash.

Test theory about didCommitProvisionalLoad crash.

It appears that we're creating a RenderView with both a main frame
routing ID and a proxy routing ID, which shouldn't be possible after
the switch to disable swappedout://.

BUG=575245
TEST=Crash in RenderViewImpl::Initialize instead.

Committed: https://crrev.com/d03d22651ca68719b472a6489a0ea9fa3170ab4c
Cr-Commit-Position: refs/heads/master@{#371009}

[Charlie Reis, 2016-01-22 18:00:51]

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

[Charlie Reis, 2016-01-22 18:00:51]

https://codereview.chromium.org/1618063003/diff/1/content/renderer/render_vie...
File content/renderer/render_view_impl.cc (right):

https://codereview.chromium.org/1618063003/diff/1/content/renderer/render_vie...
content/renderer/render_view_impl.cc:694:
CHECK(!SiteIsolationPolicy::IsSwappedOutStateForbidden());
I suspect that we'll fail this check rather than crash in
didCommitProvisionalLoad, based on the params I saw in crash reports.  It looks
like the RenderView has *both* a main frame routing ID and a proxy ID, which (I
think) shouldn't be possible anymore.

[Charlie Reis, 2016-01-22 18:20:28]

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

[commit-bot: I haz the power, 2016-01-22 18:20:56]

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

[nasko, 2016-01-22 18:39:38]

LGTM

[nasko, 2016-01-22 18:40:50]

The CQ bit was unchecked by nasko@chromium.org

[nasko, 2016-01-22 18:41:06]

The CQ bit was checked by nasko@chromium.org

[commit-bot: I haz the power, 2016-01-22 18:41:23]

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

[commit-bot: I haz the power, 2016-01-22 19:36:11]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-01-22 19:37:19]

Description was changed from

==========
Test theory about didCommitProvisionalLoad crash.

It appears that we're creating a RenderView with both a main frame
routing ID and a proxy routing ID, which shouldn't be possible after
the switch to disable swappedout://.

BUG=575245
TEST=Crash in RenderViewImpl::Initialize instead.
==========

to

==========
Test theory about didCommitProvisionalLoad crash.

It appears that we're creating a RenderView with both a main frame
routing ID and a proxy routing ID, which shouldn't be possible after
the switch to disable swappedout://.

BUG=575245
TEST=Crash in RenderViewImpl::Initialize instead.

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

[commit-bot: I haz the power, 2016-01-22 19:37:20]

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