Remove CRWSessionEntry from NavigationManagerImpl.

Remove CRWSessionEntry from NavigationManagerImpl.

BUG=454984
Review-Url: https://codereview.chromium.org/2704663004
Cr-Commit-Position: refs/heads/master@{#452413}
Committed: https://chromium.googlesource.com/chromium/src/+/2cae20f73af243e449f333646837cb7a98cdbf84

[kkhorimoto, 2017-02-17 23:41:00]

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

[Eugene But (OOO till 7-30), 2017-02-18 01:21:38]

lgtm

https://codereview.chromium.org/2704663004/diff/1/ios/web/navigation/navigati...
File ios/web/navigation/navigation_manager_impl.mm (right):

https://codereview.chromium.org/2704663004/diff/1/ios/web/navigation/navigati...
ios/web/navigation/navigation_manager_impl.mm:269: if (idx >= GetItems().size())
Do you want to file a bug and add a TODO to change RemoveItemAtIndex and accept
size_t?

[kkhorimoto, 2017-02-23 05:21:57]

The CQ bit was checked by kkhorimoto@chromium.org

[kkhorimoto, 2017-02-23 05:21:57]

The patchset sent to the CQ was uploaded after l-g-t-m from
eugenebut@chromium.org
Link to the patchset: https://codereview.chromium.org/2704663004/#ps20001
(title: "Added size_t todos")

[commit-bot: I haz the power, 2017-02-23 05:22:14]

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

[kkhorimoto, 2017-02-23 05:22:31]

https://codereview.chromium.org/2704663004/diff/1/ios/web/navigation/navigati...
File ios/web/navigation/navigation_manager_impl.mm (right):

https://codereview.chromium.org/2704663004/diff/1/ios/web/navigation/navigati...
ios/web/navigation/navigation_manager_impl.mm:269: if (idx >= GetItems().size())
On 2017/02/18 01:21:38, Eugene But wrote:
> Do you want to file a bug and add a TODO to change RemoveItemAtIndex and
accept
> size_t?

One was already created; I've added TODOs in navigation_manager.h

[commit-bot: I haz the power, 2017-02-23 05:54:51]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1487827317234600,
"parent_rev": "064e872bec2dc2bc381a493c6be6c0f24d672c4d", "commit_rev":
"2cae20f73af243e449f333646837cb7a98cdbf84"}

[commit-bot: I haz the power, 2017-02-23 05:55:24]

Description was changed from

==========
Remove CRWSessionEntry from NavigationManagerImpl.

BUG=454984
==========

to

==========
Remove CRWSessionEntry from NavigationManagerImpl.

BUG=454984

Review-Url: https://codereview.chromium.org/2704663004
Cr-Commit-Position: refs/heads/master@{#452413}
Committed:
https://chromium.googlesource.com/chromium/src/+/2cae20f73af243e449f333646837...
==========

[commit-bot: I haz the power, 2017-02-23 05:55:25]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/2cae20f73af243e449f333646837...