Remove DumpWithoutCrashing() from navigation_handle_impl

Remove DumpWithoutCrashing() from navigation_handle_impl

More tidy up of DumpWithoutCrashing() calls. Per
https://goto.google.com/uoflm, this doesn't seem to have been hit post
M56, so let me know if you'd like to remove the whole `if` instead.

R=clamy@chromium.org
BUG=653344, 694688
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

Review-Url: https://codereview.chromium.org/2714823002
Cr-Commit-Position: refs/heads/master@{#453298}
Committed: https://chromium.googlesource.com/chromium/src/+/9efb69199efad988c5088925d9c992ceeb92de59

[scottmg, 2017-02-23 22:10:01]

Description was changed from

==========
Remove DumpWithoutCrashing() from navigation_handle_impl

More tidy up of DumpWithoutCrashing() calls.

R=clamy@chromium.org
BUG=653344,694688
==========

to

==========
Remove DumpWithoutCrashing() from navigation_handle_impl

More tidy up of DumpWithoutCrashing() calls.

R=clamy@chromium.org
BUG=653344,694688
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

[scottmg, 2017-02-23 22:11:19]

Description was changed from

==========
Remove DumpWithoutCrashing() from navigation_handle_impl

More tidy up of DumpWithoutCrashing() calls.

R=clamy@chromium.org
BUG=653344,694688
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Remove DumpWithoutCrashing() from navigation_handle_impl

More tidy up of DumpWithoutCrashing() calls. Per goto.google.com//uoflm,
this doesn't seem to have been hit since M56, so let me know if you'd
like to remove the whole `if` instead.

R=clamy@chromium.org
BUG=653344,694688
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

[scottmg, 2017-02-23 22:11:26]

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

[commit-bot: I haz the power, 2017-02-23 22:11:59]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[scottmg, 2017-02-23 22:12:06]

Description was changed from

==========
Remove DumpWithoutCrashing() from navigation_handle_impl

More tidy up of DumpWithoutCrashing() calls. Per goto.google.com//uoflm,
this doesn't seem to have been hit since M56, so let me know if you'd
like to remove the whole `if` instead.

R=clamy@chromium.org
BUG=653344,694688
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Remove DumpWithoutCrashing() from navigation_handle_impl

More tidy up of DumpWithoutCrashing() calls. Per
https://goto.google.com/uoflm, this doesn't seem to have been hit since
M56, so let me know if you'd like to remove the whole `if` instead.

R=clamy@chromium.org
BUG=653344,694688
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

[scottmg, 2017-02-23 22:12:51]

Description was changed from

==========
Remove DumpWithoutCrashing() from navigation_handle_impl

More tidy up of DumpWithoutCrashing() calls. Per
https://goto.google.com/uoflm, this doesn't seem to have been hit since
M56, so let me know if you'd like to remove the whole `if` instead.

R=clamy@chromium.org
BUG=653344,694688
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Remove DumpWithoutCrashing() from navigation_handle_impl

More tidy up of DumpWithoutCrashing() calls. Per
https://goto.google.com/uoflm, this doesn't seem to have been hit post
M56, so let me know if you'd like to remove the whole `if` instead.

R=clamy@chromium.org
BUG=653344,694688
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

[commit-bot: I haz the power, 2017-02-24 01:04:36]

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

[commit-bot: I haz the power, 2017-02-24 01:04:37]

Dry run: This issue passed the CQ dry run.

[clamy, 2017-02-27 15:20:11]

lgtm.

[scottmg, 2017-02-27 18:41:32]

The CQ bit was checked by scottmg@chromium.org

[commit-bot: I haz the power, 2017-02-27 18:42:10]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-27 19:36:26]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1488220892814780, "parent_rev":
"4c12e06b9f81f8845fa504213ebc3077773ca444", "commit_rev":
"9efb69199efad988c5088925d9c992ceeb92de59"}

[commit-bot: I haz the power, 2017-02-27 19:37:04]

Description was changed from

==========
Remove DumpWithoutCrashing() from navigation_handle_impl

More tidy up of DumpWithoutCrashing() calls. Per
https://goto.google.com/uoflm, this doesn't seem to have been hit post
M56, so let me know if you'd like to remove the whole `if` instead.

R=clamy@chromium.org
BUG=653344,694688
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Remove DumpWithoutCrashing() from navigation_handle_impl

More tidy up of DumpWithoutCrashing() calls. Per
https://goto.google.com/uoflm, this doesn't seem to have been hit post
M56, so let me know if you'd like to remove the whole `if` instead.

R=clamy@chromium.org
BUG=653344,694688
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

Review-Url: https://codereview.chromium.org/2714823002
Cr-Commit-Position: refs/heads/master@{#453298}
Committed:
https://chromium.googlesource.com/chromium/src/+/9efb69199efad988c5088925d9c9...
==========

[commit-bot: I haz the power, 2017-02-27 19:37:06]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/9efb69199efad988c5088925d9c9...