[ios] Added a CHECK to didCommitNavigation: to debug crbug.com/676340.

[ios] Added a CHECK to didCommitNavigation: to debug crbug.com/676340.

didCommitNavigation: crashes if NavigationManager is empty. This new
CHECK crashes if MN is empty and |didStartProvisionalNavigation| was
called.

If this CHECK is hit it means that |didStartProvisionalNavigation| was
called as expected but it did not call |registerLoadRequest|. If that's
the case then |didStartProvisionalNavigation| needs to be changed to
always call |registerLoadRequest| if MN is empty.

BUG=676458
Committed: https://crrev.com/272901feb7b3ed31f1aa50e50b799512f1152917
Cr-Commit-Position: refs/heads/master@{#440796}

[Eugene But (OOO till 7-30), 2016-12-22 22:13:09]

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

[commit-bot: I haz the power, 2016-12-22 22:13:29]

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

[Eugene But (OOO till 7-30), 2016-12-22 22:40:22]

eugenebut@chromium.org changed reviewers:
+ kkhorimoto@chromium.org

[commit-bot: I haz the power, 2016-12-22 23:15:45]

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

[commit-bot: I haz the power, 2016-12-22 23:15:46]

Dry run: This issue passed the CQ dry run.

[kkhorimoto, 2016-12-27 21:52:29]

lgtm

[Eugene But (OOO till 7-30), 2016-12-27 22:05:44]

The CQ bit was checked by eugenebut@chromium.org

[commit-bot: I haz the power, 2016-12-27 22:05:57]

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, 2016-12-27 22:14:54]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1482876344717800, "parent_rev":
"754d3b8aa7f55e34ac60158048ed969ff362a2a6", "commit_rev":
"f3c21b29b0596c47c2f55da7196a842dae875509"}

[commit-bot: I haz the power, 2016-12-27 22:15:21]

Description was changed from

==========
[ios] Added a CHECK to didCommitNavigation: to debug crbug.com/676340.

didCommitNavigation: crashes if NavigationManager is empty. This new
CHECK crashes if MN is empty and |didStartProvisionalNavigation| was
called.

If this CHECK is hit it means that |didStartProvisionalNavigation| was
called as expected but it did not call |registerLoadRequest|. If that's
the case then |didStartProvisionalNavigation| needs to be changed to
always call |registerLoadRequest| if MN is empty.

BUG=676458
==========

to

==========
[ios] Added a CHECK to didCommitNavigation: to debug crbug.com/676340.

didCommitNavigation: crashes if NavigationManager is empty. This new
CHECK crashes if MN is empty and |didStartProvisionalNavigation| was
called.

If this CHECK is hit it means that |didStartProvisionalNavigation| was
called as expected but it did not call |registerLoadRequest|. If that's
the case then |didStartProvisionalNavigation| needs to be changed to
always call |registerLoadRequest| if MN is empty.

BUG=676458

Review-Url: https://codereview.chromium.org/2596243003
==========

[commit-bot: I haz the power, 2016-12-27 22:15:22]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2017-01-02 15:47:30]

Description was changed from

==========
[ios] Added a CHECK to didCommitNavigation: to debug crbug.com/676340.

didCommitNavigation: crashes if NavigationManager is empty. This new
CHECK crashes if MN is empty and |didStartProvisionalNavigation| was
called.

If this CHECK is hit it means that |didStartProvisionalNavigation| was
called as expected but it did not call |registerLoadRequest|. If that's
the case then |didStartProvisionalNavigation| needs to be changed to
always call |registerLoadRequest| if MN is empty.

BUG=676458

Review-Url: https://codereview.chromium.org/2596243003
==========

to

==========
[ios] Added a CHECK to didCommitNavigation: to debug crbug.com/676340.

didCommitNavigation: crashes if NavigationManager is empty. This new
CHECK crashes if MN is empty and |didStartProvisionalNavigation| was
called.

If this CHECK is hit it means that |didStartProvisionalNavigation| was
called as expected but it did not call |registerLoadRequest|. If that's
the case then |didStartProvisionalNavigation| needs to be changed to
always call |registerLoadRequest| if MN is empty.

BUG=676458

Committed: https://crrev.com/272901feb7b3ed31f1aa50e50b799512f1152917
Cr-Commit-Position: refs/heads/master@{#440796}
==========

[commit-bot: I haz the power, 2017-01-02 15:47:30]

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