Remove the concept of currentItemIndex

ios/web/navigation/navigation_manager_impl_unittest.mm

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #import "ios/web/navigation/navigation_manager_impl.h"
 
 #include "base/logging.h"
 #import "base/mac/scoped_nsobject.h"
 #import "ios/web/navigation/crw_session_controller+private_constructors.h"
 #import "ios/web/navigation/navigation_manager_delegate.h"
@@ skipping 33 matching lines @@
  private:
   TestBrowserState browser_state_;
   TestNavigationManagerDelegate delegate_;
   std::unique_ptr<NavigationManagerImpl> manager_;
   base::scoped_nsobject<CRWSessionController> controller_;
 };
 
 // Tests state of an empty navigation manager.
 TEST_F(NavigationManagerTest, EmptyManager) {
   EXPECT_EQ(0, navigation_manager()->GetItemCount());
-  EXPECT_EQ(-1, navigation_manager()->GetCurrentItemIndex());
-  EXPECT_EQ(-1, navigation_manager()->GetCurrentItemIndex());
+  EXPECT_EQ(-1, navigation_manager()->GetLastCommittedItemIndex());
+  EXPECT_EQ(-1, navigation_manager()->GetLastCommittedItemIndex());
   EXPECT_FALSE(navigation_manager()->GetPendingItem());
   EXPECT_EQ(-1, navigation_manager()->GetPendingItemIndex());
   EXPECT_EQ(-1, navigation_manager()->GetIndexForOffset(0));
 }
 
 // Tests that GetPendingItemIndex() returns -1 if there is no pending entry.
 TEST_F(NavigationManagerTest, GetPendingItemIndexWithoutPendingEntry) {
   navigation_manager()->AddPendingItem(
       GURL("http://www.url.com"), Referrer(), ui::PAGE_TRANSITION_TYPED,
       web::NavigationInitiationType::USER_INITIATED);
@@ skipping 198 matching lines @@
   navigation_manager()->AddPendingItem(
       GURL("http://www.url.com/2"), Referrer(), ui::PAGE_TRANSITION_LINK,
       web::NavigationInitiationType::USER_INITIATED);
   [session_controller() commitPendingItem];
   navigation_manager()->AddPendingItem(
       GURL("http://www.url.com/redirect"), Referrer(),
       ui::PAGE_TRANSITION_IS_REDIRECT_MASK,
       web::NavigationInitiationType::USER_INITIATED);
   [session_controller() commitPendingItem];
   ASSERT_EQ(5, navigation_manager()->GetItemCount());
-  ASSERT_EQ(4, navigation_manager()->GetCurrentItemIndex());
+  ASSERT_EQ(4, navigation_manager()->GetLastCommittedItemIndex());
 
   // Go to entry at index 1 and test API from that state.
   [session_controller() goToItemAtIndex:1];
-  ASSERT_EQ(1, navigation_manager()->GetCurrentItemIndex());
+  ASSERT_EQ(1, navigation_manager()->GetLastCommittedItemIndex());
   ASSERT_EQ(-1, navigation_manager()->GetPendingItemIndex());
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(-1));
   EXPECT_EQ(-1, navigation_manager()->GetIndexForOffset(-1));
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(-2));
   EXPECT_EQ(-2, navigation_manager()->GetIndexForOffset(-2));
   EXPECT_TRUE(navigation_manager()->CanGoToOffset(1));
   EXPECT_EQ(2, navigation_manager()->GetIndexForOffset(1));
   EXPECT_TRUE(navigation_manager()->CanGoToOffset(2));
   EXPECT_EQ(4, navigation_manager()->GetIndexForOffset(2));
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(3));
   EXPECT_EQ(5, navigation_manager()->GetIndexForOffset(3));
   // Test with large values.
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(INT_MAX));
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(INT_MIN));
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(-1000000000));
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(1000000000));
   EXPECT_EQ(INT_MAX, navigation_manager()->GetIndexForOffset(INT_MAX));
   EXPECT_EQ(INT_MIN, navigation_manager()->GetIndexForOffset(INT_MIN));
   EXPECT_EQ(-1000000000, navigation_manager()->GetIndexForOffset(-1000000000));
   EXPECT_EQ(1000000002, navigation_manager()->GetIndexForOffset(1000000000));
 
   // Go to entry at index 2 and test API from that state.
   [session_controller() goToItemAtIndex:2];
-  ASSERT_EQ(2, navigation_manager()->GetCurrentItemIndex());
+  ASSERT_EQ(2, navigation_manager()->GetLastCommittedItemIndex());
   ASSERT_EQ(-1, navigation_manager()->GetPendingItemIndex());
   EXPECT_TRUE(navigation_manager()->CanGoToOffset(-1));
   EXPECT_EQ(1, navigation_manager()->GetIndexForOffset(-1));
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(-2));
   EXPECT_EQ(-1, navigation_manager()->GetIndexForOffset(-2));
   EXPECT_TRUE(navigation_manager()->CanGoToOffset(1));
   EXPECT_EQ(4, navigation_manager()->GetIndexForOffset(1));
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(2));
   EXPECT_EQ(5, navigation_manager()->GetIndexForOffset(2));
   // Test with large values.
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(INT_MAX));
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(INT_MIN));
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(-1000000000));
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(1000000000));
   EXPECT_EQ(INT_MAX, navigation_manager()->GetIndexForOffset(INT_MAX));
   EXPECT_EQ(-2147483647, navigation_manager()->GetIndexForOffset(INT_MIN));
   EXPECT_EQ(-999999999, navigation_manager()->GetIndexForOffset(-1000000000));
   EXPECT_EQ(1000000003, navigation_manager()->GetIndexForOffset(1000000000));
 
   // Go to entry at index 4 and test API from that state.
   [session_controller() goToItemAtIndex:4];
-  ASSERT_EQ(4, navigation_manager()->GetCurrentItemIndex());
+  ASSERT_EQ(4, navigation_manager()->GetLastCommittedItemIndex());
   ASSERT_EQ(-1, navigation_manager()->GetPendingItemIndex());
   EXPECT_TRUE(navigation_manager()->CanGoToOffset(-1));
   EXPECT_EQ(2, navigation_manager()->GetIndexForOffset(-1));
   EXPECT_TRUE(navigation_manager()->CanGoToOffset(-2));
   EXPECT_EQ(1, navigation_manager()->GetIndexForOffset(-2));
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(1));
   EXPECT_EQ(5, navigation_manager()->GetIndexForOffset(1));
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(2));
   EXPECT_EQ(6, navigation_manager()->GetIndexForOffset(2));
   // Test with large values.
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(INT_MAX));
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(INT_MIN));
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(-1000000000));
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(1000000000));
   EXPECT_EQ(INT_MAX, navigation_manager()->GetIndexForOffset(INT_MAX));
   EXPECT_EQ(-2147483646, navigation_manager()->GetIndexForOffset(INT_MIN));
   EXPECT_EQ(-999999998, navigation_manager()->GetIndexForOffset(-1000000000));
   EXPECT_EQ(1000000004, navigation_manager()->GetIndexForOffset(1000000000));
 
   // Test with existing transient entry.
   [session_controller() addTransientItemWithURL:GURL("http://www.url.com")];
   ASSERT_EQ(5, navigation_manager()->GetItemCount());
-  ASSERT_EQ(4, navigation_manager()->GetCurrentItemIndex());
+  ASSERT_EQ(4, navigation_manager()->GetLastCommittedItemIndex());
   ASSERT_EQ(-1, navigation_manager()->GetPendingItemIndex());
   EXPECT_TRUE(navigation_manager()->CanGoToOffset(-1));
   EXPECT_EQ(4, navigation_manager()->GetIndexForOffset(-1));
   EXPECT_TRUE(navigation_manager()->CanGoToOffset(-2));
   EXPECT_EQ(2, navigation_manager()->GetIndexForOffset(-2));
   EXPECT_TRUE(navigation_manager()->CanGoToOffset(-3));
   EXPECT_EQ(1, navigation_manager()->GetIndexForOffset(-3));
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(1));
   EXPECT_EQ(5, navigation_manager()->GetIndexForOffset(1));
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(2));
   EXPECT_EQ(6, navigation_manager()->GetIndexForOffset(2));
   // Test with large values.
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(INT_MAX));
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(INT_MIN));
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(-1000000000));
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(1000000000));
   EXPECT_EQ(INT_MAX, navigation_manager()->GetIndexForOffset(INT_MAX));
   EXPECT_EQ(-2147483645, navigation_manager()->GetIndexForOffset(INT_MIN));
   EXPECT_EQ(-999999997, navigation_manager()->GetIndexForOffset(-1000000000));
   EXPECT_EQ(1000000004, navigation_manager()->GetIndexForOffset(1000000000));
 
   // Now test with pending item index.
   [session_controller() discardNonCommittedItems];
 
   // Set pending index to 1 and test API from that state.
   [session_controller() setPendingItemIndex:1];
-  ASSERT_EQ(4, navigation_manager()->GetCurrentItemIndex());
+  ASSERT_EQ(4, navigation_manager()->GetLastCommittedItemIndex());
   ASSERT_EQ(1, navigation_manager()->GetPendingItemIndex());
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(-1));
   EXPECT_EQ(-1, navigation_manager()->GetIndexForOffset(-1));
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(-2));
   EXPECT_EQ(-2, navigation_manager()->GetIndexForOffset(-2));
   EXPECT_TRUE(navigation_manager()->CanGoToOffset(1));
   EXPECT_EQ(2, navigation_manager()->GetIndexForOffset(1));
   EXPECT_TRUE(navigation_manager()->CanGoToOffset(2));
   EXPECT_EQ(4, navigation_manager()->GetIndexForOffset(2));
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(3));
   EXPECT_EQ(5, navigation_manager()->GetIndexForOffset(3));
   // Test with large values.
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(INT_MAX));
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(INT_MIN));
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(-1000000000));
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(1000000000));
   EXPECT_EQ(INT_MAX, navigation_manager()->GetIndexForOffset(INT_MAX));
   EXPECT_EQ(INT_MIN, navigation_manager()->GetIndexForOffset(INT_MIN));
   EXPECT_EQ(-1000000000, navigation_manager()->GetIndexForOffset(-1000000000));
   EXPECT_EQ(1000000002, navigation_manager()->GetIndexForOffset(1000000000));
 
   // Set pending index to 2 and test API from that state.
   [session_controller() setPendingItemIndex:2];
-  ASSERT_EQ(4, navigation_manager()->GetCurrentItemIndex());
+  ASSERT_EQ(4, navigation_manager()->GetLastCommittedItemIndex());
   ASSERT_EQ(2, navigation_manager()->GetPendingItemIndex());
   EXPECT_TRUE(navigation_manager()->CanGoToOffset(-1));
   EXPECT_EQ(1, navigation_manager()->GetIndexForOffset(-1));
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(-2));
   EXPECT_EQ(-1, navigation_manager()->GetIndexForOffset(-2));
   EXPECT_TRUE(navigation_manager()->CanGoToOffset(1));
   EXPECT_EQ(4, navigation_manager()->GetIndexForOffset(1));
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(2));
   EXPECT_EQ(5, navigation_manager()->GetIndexForOffset(2));
   // Test with large values.
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(INT_MAX));
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(INT_MIN));
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(-1000000000));
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(1000000000));
   EXPECT_EQ(INT_MAX, navigation_manager()->GetIndexForOffset(INT_MAX));
   EXPECT_EQ(-2147483647, navigation_manager()->GetIndexForOffset(INT_MIN));
   EXPECT_EQ(-999999999, navigation_manager()->GetIndexForOffset(-1000000000));
   EXPECT_EQ(1000000003, navigation_manager()->GetIndexForOffset(1000000000));
 
   // Set pending index to 4 and committed entry to 1 and test.
   [session_controller() goToItemAtIndex:1];
   [session_controller() setPendingItemIndex:4];
-  ASSERT_EQ(1, navigation_manager()->GetCurrentItemIndex());
+  ASSERT_EQ(1, navigation_manager()->GetLastCommittedItemIndex());
   ASSERT_EQ(4, navigation_manager()->GetPendingItemIndex());
   EXPECT_TRUE(navigation_manager()->CanGoToOffset(-1));
   EXPECT_EQ(2, navigation_manager()->GetIndexForOffset(-1));
   EXPECT_TRUE(navigation_manager()->CanGoToOffset(-2));
   EXPECT_EQ(1, navigation_manager()->GetIndexForOffset(-2));
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(1));
   EXPECT_EQ(5, navigation_manager()->GetIndexForOffset(1));
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(2));
   EXPECT_EQ(6, navigation_manager()->GetIndexForOffset(2));
   // Test with large values.
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(INT_MAX));
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(INT_MIN));
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(-1000000000));
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(1000000000));
   EXPECT_EQ(INT_MAX, navigation_manager()->GetIndexForOffset(INT_MAX));
   EXPECT_EQ(-2147483646, navigation_manager()->GetIndexForOffset(INT_MIN));
   EXPECT_EQ(-999999998, navigation_manager()->GetIndexForOffset(-1000000000));
   EXPECT_EQ(1000000004, navigation_manager()->GetIndexForOffset(1000000000));
 
   // Test with existing transient entry in the end of the stack.
   [session_controller() goToItemAtIndex:4];
   [session_controller() setPendingItemIndex:-1];
   [session_controller() addTransientItemWithURL:GURL("http://www.url.com")];
   ASSERT_EQ(5, navigation_manager()->GetItemCount());
-  ASSERT_EQ(4, navigation_manager()->GetCurrentItemIndex());
+  ASSERT_EQ(4, navigation_manager()->GetLastCommittedItemIndex());
   ASSERT_EQ(-1, navigation_manager()->GetPendingItemIndex());
   EXPECT_TRUE(navigation_manager()->CanGoToOffset(-1));
   EXPECT_EQ(4, navigation_manager()->GetIndexForOffset(-1));
   EXPECT_TRUE(navigation_manager()->CanGoToOffset(-2));
   EXPECT_EQ(2, navigation_manager()->GetIndexForOffset(-2));
   EXPECT_TRUE(navigation_manager()->CanGoToOffset(-3));
   EXPECT_EQ(1, navigation_manager()->GetIndexForOffset(-3));
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(1));
   EXPECT_EQ(5, navigation_manager()->GetIndexForOffset(1));
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(2));
@@ skipping 24 matching lines @@
       web::NavigationInitiationType::USER_INITIATED);
   [session_controller() commitPendingItem];
   navigation_manager()->AddPendingItem(
       GURL("http://www.url.com/2"), Referrer(), ui::PAGE_TRANSITION_LINK,
       web::NavigationInitiationType::USER_INITIATED);
   [session_controller() commitPendingItem];
   [session_controller() addTransientItemWithURL:GURL("http://www.url.com/1")];
   [session_controller() setPendingItemIndex:1];
 
   ASSERT_EQ(3, navigation_manager()->GetItemCount());
-  ASSERT_EQ(2, navigation_manager()->GetCurrentItemIndex());
+  ASSERT_EQ(2, navigation_manager()->GetLastCommittedItemIndex());
   ASSERT_EQ(1, navigation_manager()->GetPendingItemIndex());
   EXPECT_EQ(2, navigation_manager()->GetIndexForOffset(1));
   EXPECT_EQ(0, navigation_manager()->GetIndexForOffset(-1));
 
   // Now go forward to that middle transient item (pending index is 1,
   // current index is 0).
   [session_controller() goToItemAtIndex:0];
   [session_controller() setPendingItemIndex:1];
   ASSERT_EQ(3, navigation_manager()->GetItemCount());
-  ASSERT_EQ(0, navigation_manager()->GetCurrentItemIndex());
+  ASSERT_EQ(0, navigation_manager()->GetLastCommittedItemIndex());
   ASSERT_EQ(1, navigation_manager()->GetPendingItemIndex());
   EXPECT_EQ(2, navigation_manager()->GetIndexForOffset(1));
   EXPECT_EQ(0, navigation_manager()->GetIndexForOffset(-1));
 }
 
 // Tests that desktop user agent can be enforced to use for next pending item.
 TEST_F(NavigationManagerTest, OverrideDesktopUserAgent) {
   navigation_manager()->OverrideDesktopUserAgentForNextPendingItem();
   navigation_manager()->AddPendingItem(
       GURL("http://www.url.com"), Referrer(), ui::PAGE_TRANSITION_TYPED,
@@ skipping 96 matching lines @@
 
   GURL url_before_reload = navigation_manager()->GetVisibleItem()->GetURL();
   navigation_manager()->Reload(web::ReloadType::NORMAL,
                                false /* check_for_repost */);
 
   EXPECT_EQ(url_before_reload,
             navigation_manager()->GetVisibleItem()->GetURL());
 }
 
 }  // namespace web