| Index: ios/web/navigation/navigation_manager_impl.mm
|
| diff --git a/ios/web/navigation/navigation_manager_impl.mm b/ios/web/navigation/navigation_manager_impl.mm
|
| index c5a317671b36df3782e80c180f913f6207412321..4dbefc46f0899d58a752ad8d436da9ab0b7fcf16 100644
|
| --- a/ios/web/navigation/navigation_manager_impl.mm
|
| +++ b/ios/web/navigation/navigation_manager_impl.mm
|
| @@ -139,8 +139,8 @@ void NavigationManagerImpl::OnNavigationItemCommitted() {
|
| DCHECK(details.item);
|
| details.previous_item_index = [session_controller_ previousNavigationIndex];
|
| if (details.previous_item_index >= 0) {
|
| - DCHECK(GetPreviousItem());
|
| - details.previous_url = GetPreviousItem()->GetURL();
|
| + DCHECK([session_controller_ previousItem]);
|
| + details.previous_url = [session_controller_ previousItem]->GetURL();
|
| details.is_in_page =
|
| AreURLsInPageNavigation(details.previous_url, details.item->GetURL());
|
| } else {
|
| @@ -193,10 +193,6 @@ NavigationItem* NavigationManagerImpl::GetLastUserItem() const {
|
| return [session_controller_ lastUserItem];
|
| }
|
|
|
| -NavigationItem* NavigationManagerImpl::GetPreviousItem() const {
|
| - return [session_controller_ previousItem];
|
| -}
|
| -
|
| NavigationItemList NavigationManagerImpl::GetItems() const {
|
| return [session_controller_ items];
|
| }
|
| @@ -249,12 +245,11 @@ void NavigationManagerImpl::AddTransientURLRewriter(
|
| }
|
|
|
| int NavigationManagerImpl::GetItemCount() const {
|
| - return GetItems().size();
|
| + return [session_controller_ itemCount];
|
| }
|
|
|
| NavigationItem* NavigationManagerImpl::GetItemAtIndex(size_t index) const {
|
| - NavigationItemList items = GetItems();
|
| - return index < items.size() ? items[index] : nullptr;
|
| + return [session_controller_ itemAtIndex:index];
|
| }
|
|
|
| int NavigationManagerImpl::GetCurrentItemIndex() const {
|
| @@ -274,7 +269,7 @@ int NavigationManagerImpl::GetPendingItemIndex() const {
|
| }
|
|
|
| int NavigationManagerImpl::GetLastCommittedItemIndex() const {
|
| - if (GetItems().empty())
|
| + if (GetItemCount() == 0)
|
| return -1;
|
| return [session_controller_ currentNavigationIndex];
|
| }
|
| @@ -283,8 +278,7 @@ bool NavigationManagerImpl::RemoveItemAtIndex(int index) {
|
| if (index == GetLastCommittedItemIndex() || index == GetPendingItemIndex())
|
| return false;
|
|
|
| - size_t idx = static_cast<size_t>(index);
|
| - if (idx >= GetItems().size())
|
| + if (index < 0 || index >= GetItemCount())
|
| return false;
|
|
|
| [session_controller_ removeItemAtIndex:index];
|
|
|