Avoid renderer kills for missing NavigationHandle in DidFail.

Avoid renderer kills for missing NavigationHandle in DidFail.

TBR=clamy
BUG=605289
TEST=Fewer renderer kills in the stability data.
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation

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

[Charlie Reis, 2016-04-20 21:54:22]

Description was changed from

==========
Avoid renderer kills for missing NavigationHandle in DidFail.

BUG=605289
TEST=Fewer renderer kills in the stability data.
==========

to

==========
Avoid renderer kills for missing NavigationHandle in DidFail.

BUG=605289
TEST=Fewer renderer kills in the stability data.
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
==========

[Charlie Reis, 2016-04-20 21:56:34]

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

[Charlie Reis, 2016-04-20 21:56:34]

Nasko, can you take a look?  This undoes one of the kills from
https://codereview.chromium.org/1880423002/, and makes the logic closer to what
it was before (e.g., no early return if the handle is null).

TBR'ing clamy@.

[Charlie Reis, 2016-04-20 21:56:44]

Description was changed from

==========
Avoid renderer kills for missing NavigationHandle in DidFail.

BUG=605289
TEST=Fewer renderer kills in the stability data.
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Avoid renderer kills for missing NavigationHandle in DidFail.

TBR=clamy
BUG=605289
TEST=Fewer renderer kills in the stability data.
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
==========

[nasko, 2016-04-20 22:03:54]

LGTM

[Charlie Reis, 2016-04-20 22:20:38]

The CQ bit was checked by creis@chromium.org

[commit-bot: I haz the power, 2016-04-20 22:21:03]

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

[commit-bot: I haz the power, 2016-04-20 23:39:24]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-04-20 23:39:25]

Try jobs failed on following builders:
  win_clang on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_clang/builds/...)

[Charlie Reis, 2016-04-21 00:04:50]

The CQ bit was checked by creis@chromium.org

[commit-bot: I haz the power, 2016-04-21 00:05:48]

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

[commit-bot: I haz the power, 2016-04-21 03:12:12]

Description was changed from

==========
Avoid renderer kills for missing NavigationHandle in DidFail.

TBR=clamy
BUG=605289
TEST=Fewer renderer kills in the stability data.
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Avoid renderer kills for missing NavigationHandle in DidFail.

TBR=clamy
BUG=605289
TEST=Fewer renderer kills in the stability data.
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
==========

[commit-bot: I haz the power, 2016-04-21 03:12:13]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-22 19:31:05]

Description was changed from

==========
Avoid renderer kills for missing NavigationHandle in DidFail.

TBR=clamy
BUG=605289
TEST=Fewer renderer kills in the stability data.
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Avoid renderer kills for missing NavigationHandle in DidFail.

TBR=clamy
BUG=605289
TEST=Fewer renderer kills in the stability data.
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation

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

[commit-bot: I haz the power, 2016-04-22 19:31:06]

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