Make WebContentsImpl::UpdateState navigation enties DCHECK conditional

Make WebContentsImpl::UpdateState navigation enties DCHECK conditional

When swapped out state is not used, a RenderViewHost is not guaranteed
to have a valid RenderFrameHost for the main frame. It is useful to
keep the DCHECK, but this CL makes it conditional on the presence of
a main RenderFrameHost.

BUG=357747
Committed: https://crrev.com/346969918900aa4b42f9836e1cdc29ff53c43b7c
Cr-Commit-Position: refs/heads/master@{#365642}

[nasko, 2015-12-16 21:26:07]

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

[nasko, 2015-12-16 21:26:08]

Hey Charlie,
This is split out from the disable swapped out CL mainly to keep it only to a
boolean flip. Can you review it?
Thanks in advance!
Nasko

[nasko, 2015-12-16 21:27:52]

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

[commit-bot: I haz the power, 2015-12-16 21:28:46]

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

[Charlie Reis, 2015-12-16 22:15:03]

LGTM

[nasko, 2015-12-16 22:19:37]

The CQ bit was unchecked by nasko@chromium.org

[nasko, 2015-12-16 22:19:45]

The CQ bit was checked by nasko@chromium.org

[commit-bot: I haz the power, 2015-12-16 22:21:52]

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

[commit-bot: I haz the power, 2015-12-16 22:36:54]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-12-16 22:37:59]

Description was changed from

==========
Make WebContentsImpl::UpdateState navigation enties DCHECK conditional

When swapped out state is not used, a RenderViewHost is not guaranteed
to have a valid RenderFrameHost for the main frame. It is useful to
keep the DCHECK, but this CL makes it conditional on the presence of
a main RenderFrameHost.

BUG=357747
==========

to

==========
Make WebContentsImpl::UpdateState navigation enties DCHECK conditional

When swapped out state is not used, a RenderViewHost is not guaranteed
to have a valid RenderFrameHost for the main frame. It is useful to
keep the DCHECK, but this CL makes it conditional on the presence of
a main RenderFrameHost.

BUG=357747

Committed: https://crrev.com/346969918900aa4b42f9836e1cdc29ff53c43b7c
Cr-Commit-Position: refs/heads/master@{#365642}
==========

[commit-bot: I haz the power, 2015-12-16 22:38:01]

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