Handle correctly start and commit of non-latest navigations.

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 158 matching lines @@
   [session_controller() commitPendingItem];
   navigation_manager()->AddPendingItem(
       GURL("http://www.url.com/1"), Referrer(), ui::PAGE_TRANSITION_TYPED,
       web::NavigationInitiationType::USER_INITIATED,
       web::NavigationManager::UserAgentOverrideOption::INHERIT);
   [session_controller() commitPendingItem];
 
   EXPECT_TRUE(navigation_manager()->CanGoBack());
   EXPECT_TRUE(navigation_manager()->CanGoToOffset(-1));
 
-  [session_controller() goToItemAtIndex:1];
+  [session_controller() goToItemAtIndex:1 discardNonCommittedItems:NO];
   EXPECT_TRUE(navigation_manager()->CanGoBack());
   EXPECT_TRUE(navigation_manager()->CanGoToOffset(-1));
 
-  [session_controller() goToItemAtIndex:0];
+  [session_controller() goToItemAtIndex:0 discardNonCommittedItems:NO];
   EXPECT_FALSE(navigation_manager()->CanGoBack());
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(-1));
 
-  [session_controller() goToItemAtIndex:1];
+  [session_controller() goToItemAtIndex:1 discardNonCommittedItems:NO];
   EXPECT_TRUE(navigation_manager()->CanGoBack());
   EXPECT_TRUE(navigation_manager()->CanGoToOffset(-1));
 }
 
 // Tests that going forward or positive offset is not possible without a
 // committed item.
 TEST_F(NavigationManagerTest, CanGoForwardWithoutCommitedItem) {
   EXPECT_FALSE(navigation_manager()->CanGoForward());
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(1));
 }
@@ skipping 25 matching lines @@
   [session_controller() commitPendingItem];
   navigation_manager()->AddPendingItem(
       GURL("http://www.url.com/1"), Referrer(), ui::PAGE_TRANSITION_TYPED,
       web::NavigationInitiationType::USER_INITIATED,
       web::NavigationManager::UserAgentOverrideOption::INHERIT);
   [session_controller() commitPendingItem];
 
   EXPECT_FALSE(navigation_manager()->CanGoForward());
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(1));
 
-  [session_controller() goToItemAtIndex:1];
+  [session_controller() goToItemAtIndex:1 discardNonCommittedItems:NO];
   EXPECT_TRUE(navigation_manager()->CanGoForward());
   EXPECT_TRUE(navigation_manager()->CanGoToOffset(1));
 
-  [session_controller() goToItemAtIndex:0];
+  [session_controller() goToItemAtIndex:0 discardNonCommittedItems:NO];
   EXPECT_TRUE(navigation_manager()->CanGoForward());
   EXPECT_TRUE(navigation_manager()->CanGoToOffset(1));
 
-  [session_controller() goToItemAtIndex:1];
+  [session_controller() goToItemAtIndex:1 discardNonCommittedItems:NO];
   EXPECT_TRUE(navigation_manager()->CanGoForward());
   EXPECT_TRUE(navigation_manager()->CanGoToOffset(1));
 
-  [session_controller() goToItemAtIndex:2];
+  [session_controller() goToItemAtIndex:2 discardNonCommittedItems:NO];
   EXPECT_FALSE(navigation_manager()->CanGoForward());
   EXPECT_FALSE(navigation_manager()->CanGoToOffset(1));
 }
 
 // Tests CanGoToOffset API for positive, negative and zero delta. Tested
 // navigation manager will have redirect entries to make sure they are
 // appropriately skipped.
 TEST_F(NavigationManagerTest, OffsetsWithoutPendingIndex) {
   navigation_manager()->AddPendingItem(
       GURL("http://www.url.com/0"), Referrer(), ui::PAGE_TRANSITION_LINK,
@@ skipping 19 matching lines @@
   navigation_manager()->AddPendingItem(
       GURL("http://www.url.com/redirect"), Referrer(),
       ui::PAGE_TRANSITION_CLIENT_REDIRECT,
       web::NavigationInitiationType::USER_INITIATED,
       web::NavigationManager::UserAgentOverrideOption::INHERIT);
   [session_controller() commitPendingItem];
   ASSERT_EQ(5, navigation_manager()->GetItemCount());
   ASSERT_EQ(4, navigation_manager()->GetLastCommittedItemIndex());
 
   // Go to entry at index 1 and test API from that state.
-  [session_controller() goToItemAtIndex:1];
+  [session_controller() goToItemAtIndex:1 discardNonCommittedItems:NO];
   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];
+  [session_controller() goToItemAtIndex:2 discardNonCommittedItems:NO];
   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];
+  [session_controller() goToItemAtIndex:4 discardNonCommittedItems:NO];
   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));
@@ skipping 75 matching lines @@
   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() goToItemAtIndex:1 discardNonCommittedItems:NO];
   [session_controller() setPendingItemIndex:4];
   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() goToItemAtIndex:4 discardNonCommittedItems:NO];
   [session_controller() setPendingItemIndex:-1];
   [session_controller() addTransientItemWithURL:GURL("http://www.url.com")];
   ASSERT_EQ(5, navigation_manager()->GetItemCount());
   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));
@@ skipping 38 matching lines @@
   [session_controller() setPendingItemIndex:1];
 
   ASSERT_EQ(3, navigation_manager()->GetItemCount());
   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() goToItemAtIndex:0 discardNonCommittedItems:NO];
   [session_controller() setPendingItemIndex:1];
   ASSERT_EQ(3, navigation_manager()->GetItemCount());
   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 when given a pending item, adding a new pending item replaces the
 // existing pending item if their URLs are different.
@@ skipping 632 matching lines @@
       web::NavigationInitiationType::USER_INITIATED,
       web::NavigationManager::UserAgentOverrideOption::INHERIT);
   [session_controller() commitPendingItem];
 
   navigation_manager()->AddPendingItem(
       GURL("http://www.url.com/2"), Referrer(), ui::PAGE_TRANSITION_TYPED,
       web::NavigationInitiationType::USER_INITIATED,
       web::NavigationManager::UserAgentOverrideOption::INHERIT);
   [session_controller() commitPendingItem];
 
-  [session_controller() goToItemAtIndex:1];
+  [session_controller() goToItemAtIndex:1 discardNonCommittedItems:NO];
   EXPECT_EQ(1, navigation_manager()->GetLastCommittedItemIndex());
 
   navigation_manager()->Reload(web::ReloadType::NORMAL,
                                false /* check_for_repost */);
   EXPECT_TRUE(navigation_manager_delegate().reload_called());
 
   ASSERT_TRUE(navigation_manager()->GetLastCommittedItem());
   EXPECT_EQ(url_before_reload,
             navigation_manager()->GetLastCommittedItem()->GetURL());
 }
@@ skipping 107 matching lines @@
   navigation_manager()->GetPendingItem()->SetOriginalRequestURL(
       expected_original_url);
   [session_controller() commitPendingItem];
 
   navigation_manager()->AddPendingItem(
       GURL("http://www.url.com/2"), Referrer(), ui::PAGE_TRANSITION_TYPED,
       web::NavigationInitiationType::USER_INITIATED,
       web::NavigationManager::UserAgentOverrideOption::INHERIT);
   [session_controller() commitPendingItem];
 
-  [session_controller() goToItemAtIndex:1];
+  [session_controller() goToItemAtIndex:1 discardNonCommittedItems:NO];
   EXPECT_EQ(1, navigation_manager()->GetLastCommittedItemIndex());
 
   navigation_manager()->Reload(web::ReloadType::ORIGINAL_REQUEST_URL,
                                false /* check_for_repost */);
   EXPECT_TRUE(navigation_manager_delegate().reload_called());
 
   ASSERT_TRUE(navigation_manager()->GetLastCommittedItem());
   EXPECT_EQ(expected_original_url,
             navigation_manager()->GetLastCommittedItem()->GetURL());
 }
 
 }  // namespace web