Create new pending item if UserAgentOverrideOption is not INHERIT.

ios/web/navigation/crw_session_controller_unittest.mm

 // Copyright 2012 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/crw_session_controller.h"
 
 #import <Foundation/Foundation.h>
 
 #include <utility>
 
@@ skipping 48 matching lines @@
 };
 
 TEST_F(CRWSessionControllerTest, Init) {
   EXPECT_TRUE([session_controller_ items].empty());
   EXPECT_FALSE([session_controller_ currentItem]);
 }
 
 // Tests session controller state after setting a pending index.
 TEST_F(CRWSessionControllerTest, SetPendingIndex) {
   [session_controller_
-      addPendingItem:GURL("http://www.example.com")
-            referrer:web::Referrer()
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.example.com")
+                     referrer:web::Referrer()
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::

[Eugene But (OOO till 7-30), 2017/04/06 22:53:17]

Do you want to use "using web::NavigationManager::UserAgentOverrideOption"
statement to improve the formatting?

[liaoyuke, 2017/04/07 15:49:08]

Done.

+                                  INHERIT];
   [session_controller_ commitPendingItem];
 
   EXPECT_EQ(-1, [session_controller_ pendingItemIndex]);
   [session_controller_ setPendingItemIndex:0];
   EXPECT_EQ(0, [session_controller_ pendingItemIndex]);
   EXPECT_EQ([session_controller_ items].back().get(),
             [session_controller_ pendingItem]);
 }
 
 TEST_F(CRWSessionControllerTest, addPendingItem) {
   [session_controller_
-      addPendingItem:GURL("http://www.url.com")
-            referrer:MakeReferrer("http://www.referer.com")
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.url.com")
+                     referrer:MakeReferrer("http://www.referer.com")
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
 
   EXPECT_TRUE([session_controller_ items].empty());
   EXPECT_EQ(
       GURL("http://www.url.com/"),
       [session_controller_ currentURL]);
 }
 
 TEST_F(CRWSessionControllerTest, addPendingItemWithCommittedItems) {
   [session_controller_
-      addPendingItem:GURL("http://www.committed.url.com")
-            referrer:MakeReferrer("http://www.referer.com")
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.committed.url.com")
+                     referrer:MakeReferrer("http://www.referer.com")
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ commitPendingItem];
 
   [session_controller_
-      addPendingItem:GURL("http://www.url.com")
-            referrer:MakeReferrer("http://www.referer.com")
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.url.com")
+                     referrer:MakeReferrer("http://www.referer.com")
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
 
   EXPECT_EQ(1U, [session_controller_ items].size());
   EXPECT_EQ(GURL("http://www.committed.url.com/"),
             [session_controller_ URLForItemAtIndex:0U]);
   EXPECT_EQ(
       GURL("http://www.url.com/"),
       [session_controller_ currentURL]);
 }
 
 // Tests that adding a pending item resets pending item index.
 TEST_F(CRWSessionControllerTest, addPendingItemWithExistingPendingItemIndex) {
   [session_controller_
-      addPendingItem:GURL("http://www.example.com")
-            referrer:web::Referrer()
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.example.com")
+                     referrer:web::Referrer()
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ commitPendingItem];
   [session_controller_
-      addPendingItem:GURL("http://www.example.com/0")
-            referrer:web::Referrer()
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.example.com/0")
+                     referrer:web::Referrer()
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ commitPendingItem];
 
   // Set 0 as pending item index.
   [session_controller_ setPendingItemIndex:0];
   EXPECT_EQ(GURL("http://www.example.com/"),
             [session_controller_ pendingItem]->GetURL());
   EXPECT_EQ(0, [session_controller_ pendingItemIndex]);
 
   // Add a pending item, which should drop pending navigation index.
   [session_controller_
-      addPendingItem:GURL("http://www.example.com/1")
-            referrer:web::Referrer()
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.example.com/1")
+                     referrer:web::Referrer()
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   EXPECT_EQ(GURL("http://www.example.com/1"),
             [session_controller_ pendingItem]->GetURL());
   EXPECT_EQ(-1, [session_controller_ pendingItemIndex]);
 }
 
 TEST_F(CRWSessionControllerTest, addPendingItemOverriding) {
   [session_controller_
-      addPendingItem:GURL("http://www.url.com")
-            referrer:MakeReferrer("http://www.referer.com")
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.url.com")
+                     referrer:MakeReferrer("http://www.referer.com")
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_
-      addPendingItem:GURL("http://www.another.url.com")
-            referrer:MakeReferrer("http://www.another.referer.com")
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.another.url.com")
+                     referrer:MakeReferrer("http://www.another.referer.com")
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
 
   EXPECT_TRUE([session_controller_ items].empty());
   EXPECT_EQ(
       GURL("http://www.another.url.com/"),
       [session_controller_ currentURL]);
 }
 
 TEST_F(CRWSessionControllerTest, addPendingItemAndCommit) {
   [session_controller_
-      addPendingItem:GURL("http://www.url.com")
-            referrer:MakeReferrer("http://www.referer.com")
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.url.com")
+                     referrer:MakeReferrer("http://www.referer.com")
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ commitPendingItem];
 
   EXPECT_EQ(1U, [session_controller_ items].size());
   EXPECT_EQ(GURL("http://www.url.com/"),
             [session_controller_ URLForItemAtIndex:0U]);
   EXPECT_EQ([session_controller_ items].front().get(),
             [session_controller_ currentItem]);
 }
 
 TEST_F(CRWSessionControllerTest, addPendingItemOverridingAndCommit) {
   [session_controller_
-      addPendingItem:GURL("http://www.url.com")
-            referrer:MakeReferrer("http://www.referer.com")
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.url.com")
+                     referrer:MakeReferrer("http://www.referer.com")
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_
-      addPendingItem:GURL("http://www.another.url.com")
-            referrer:MakeReferrer("http://www.another.referer.com")
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.another.url.com")
+                     referrer:MakeReferrer("http://www.another.referer.com")
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ commitPendingItem];
 
   EXPECT_EQ(1U, [session_controller_ items].size());
   EXPECT_EQ(GURL("http://www.another.url.com/"),
             [session_controller_ URLForItemAtIndex:0U]);
   EXPECT_EQ([session_controller_ items].front().get(),
             [session_controller_ currentItem]);
 }
 
 TEST_F(CRWSessionControllerTest, addPendingItemAndCommitMultiple) {
   [session_controller_
-      addPendingItem:GURL("http://www.url.com")
-            referrer:MakeReferrer("http://www.referer.com")
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.url.com")
+                     referrer:MakeReferrer("http://www.referer.com")
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ commitPendingItem];
 
   [session_controller_
-      addPendingItem:GURL("http://www.another.url.com")
-            referrer:MakeReferrer("http://www.another.referer.com")
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.another.url.com")
+                     referrer:MakeReferrer("http://www.another.referer.com")
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ commitPendingItem];
 
   EXPECT_EQ(2U, [session_controller_ items].size());
   EXPECT_EQ(GURL("http://www.url.com/"),
             [session_controller_ URLForItemAtIndex:0U]);
   EXPECT_EQ(GURL("http://www.another.url.com/"),
             [session_controller_ URLForItemAtIndex:1U]);
   EXPECT_EQ([session_controller_ items][1U].get(),
             [session_controller_ currentItem]);
 }
 
 TEST_F(CRWSessionControllerTest, addPendingItemAndDiscard) {
   [session_controller_
-      addPendingItem:GURL("http://www.url.com")
-            referrer:MakeReferrer("http://www.referer.com")
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.url.com")
+                     referrer:MakeReferrer("http://www.referer.com")
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ discardNonCommittedItems];
 
   EXPECT_TRUE([session_controller_ items].empty());
   EXPECT_FALSE([session_controller_ currentItem]);
 }
 
 // Tests discarding pending item added via |setPendingItemIndex:| call.
 TEST_F(CRWSessionControllerTest, setPendingItemIndexAndDiscard) {
   [session_controller_
-      addPendingItem:GURL("http://www.example.com")
-            referrer:web::Referrer()
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.example.com")
+                     referrer:web::Referrer()
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ commitPendingItem];
 
   [session_controller_ setPendingItemIndex:0];
   EXPECT_TRUE([session_controller_ pendingItem]);
   EXPECT_EQ(0, [session_controller_ pendingItemIndex]);
 
   [session_controller_ discardNonCommittedItems];
   EXPECT_FALSE([session_controller_ pendingItem]);
   EXPECT_EQ(-1, [session_controller_ pendingItemIndex]);
 }
 
 TEST_F(CRWSessionControllerTest, addPendingItemAndDiscardAndAddAndCommit) {
   [session_controller_
-      addPendingItem:GURL("http://www.url.com")
-            referrer:MakeReferrer("http://www.referer.com")
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.url.com")
+                     referrer:MakeReferrer("http://www.referer.com")
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ discardNonCommittedItems];
 
   [session_controller_
-      addPendingItem:GURL("http://www.another.url.com")
-            referrer:MakeReferrer("http://www.referer.com")
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.another.url.com")
+                     referrer:MakeReferrer("http://www.referer.com")
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ commitPendingItem];
 
   EXPECT_EQ(1U, [session_controller_ items].size());
   EXPECT_EQ(GURL("http://www.another.url.com/"),
             [session_controller_ URLForItemAtIndex:0U]);
   EXPECT_EQ([session_controller_ items].front().get(),
             [session_controller_ currentItem]);
 }
 
 TEST_F(CRWSessionControllerTest, addPendingItemAndCommitAndAddAndDiscard) {
   [session_controller_
-      addPendingItem:GURL("http://www.url.com")
-            referrer:MakeReferrer("http://www.referer.com")
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.url.com")
+                     referrer:MakeReferrer("http://www.referer.com")
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ commitPendingItem];
 
   [session_controller_
-      addPendingItem:GURL("http://www.another.url.com")
-            referrer:MakeReferrer("http://www.referer.com")
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.another.url.com")
+                     referrer:MakeReferrer("http://www.referer.com")
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ discardNonCommittedItems];
 
   EXPECT_EQ(1U, [session_controller_ items].size());
   EXPECT_EQ(GURL("http://www.url.com/"),
             [session_controller_ URLForItemAtIndex:0U]);
   EXPECT_EQ([session_controller_ items].front().get(),
             [session_controller_ currentItem]);
 }
 
 TEST_F(CRWSessionControllerTest,
        commitPendingItemWithoutPendingOrCommittedItem) {
   [session_controller_ commitPendingItem];
 
   EXPECT_TRUE([session_controller_ items].empty());
   EXPECT_FALSE([session_controller_ currentItem]);
 }
 
 TEST_F(CRWSessionControllerTest,
        commitPendingItemWithoutPendingItemWithCommittedItem) {
   // Setup committed item.
   [session_controller_
-      addPendingItem:GURL("http://www.url.com/")
-            referrer:MakeReferrer("http://www.referrer.com/")
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.url.com/")
+                     referrer:MakeReferrer("http://www.referrer.com/")
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ commitPendingItem];
 
   // Commit pending item when there is no such one
   [session_controller_ commitPendingItem];
 
   EXPECT_EQ(1U, [session_controller_ items].size());
   EXPECT_EQ([session_controller_ items].front().get(),
             [session_controller_ currentItem]);
 }
 
 // Tests that forward items are discarded after navigation item is committed.
 TEST_F(CRWSessionControllerTest, commitPendingItemWithExistingForwardItems) {
   // Make 3 items.
   [session_controller_
-      addPendingItem:GURL("http://www.example.com/0")
-            referrer:MakeReferrer("http://www.example.com/a")
-          transition:ui::PAGE_TRANSITION_LINK
-      initiationType:web::NavigationInitiationType::RENDERER_INITIATED];
+               addPendingItem:GURL("http://www.example.com/0")
+                     referrer:MakeReferrer("http://www.example.com/a")
+                   transition:ui::PAGE_TRANSITION_LINK
+               initiationType:web::NavigationInitiationType::RENDERER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ commitPendingItem];
   [session_controller_
-      addPendingItem:GURL("http://www.example.com/1")
-            referrer:MakeReferrer("http://www.example.com/b")
-          transition:ui::PAGE_TRANSITION_LINK
-      initiationType:web::NavigationInitiationType::RENDERER_INITIATED];
+               addPendingItem:GURL("http://www.example.com/1")
+                     referrer:MakeReferrer("http://www.example.com/b")
+                   transition:ui::PAGE_TRANSITION_LINK
+               initiationType:web::NavigationInitiationType::RENDERER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ commitPendingItem];
   [session_controller_
-      addPendingItem:GURL("http://www.example.com/2")
-            referrer:MakeReferrer("http://www.example.com/c")
-          transition:ui::PAGE_TRANSITION_LINK
-      initiationType:web::NavigationInitiationType::RENDERER_INITIATED];
+               addPendingItem:GURL("http://www.example.com/2")
+                     referrer:MakeReferrer("http://www.example.com/c")
+                   transition:ui::PAGE_TRANSITION_LINK
+               initiationType:web::NavigationInitiationType::RENDERER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ commitPendingItem];
 
   // Go back to the first item.
   [session_controller_ goToItemAtIndex:0];
 
   // Create and commit a new pending item.
   [session_controller_
-      addPendingItem:GURL("http://www.example.com/2")
-            referrer:MakeReferrer("http://www.example.com/c")
-          transition:ui::PAGE_TRANSITION_LINK
-      initiationType:web::NavigationInitiationType::RENDERER_INITIATED];
+               addPendingItem:GURL("http://www.example.com/2")
+                     referrer:MakeReferrer("http://www.example.com/c")
+                   transition:ui::PAGE_TRANSITION_LINK
+               initiationType:web::NavigationInitiationType::RENDERER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ commitPendingItem];
 
   // All forward items should go away.
   EXPECT_EQ(2U, [session_controller_ items].size());
   EXPECT_EQ(0U, [session_controller_ forwardItems].size());
   ASSERT_EQ(1, [session_controller_ lastCommittedItemIndex]);
   ASSERT_EQ(0, [session_controller_ previousItemIndex]);
 }
 
 // Tests committing pending item index from the middle.
 TEST_F(CRWSessionControllerTest, commitPendingItemIndex) {
   [session_controller_
-      addPendingItem:GURL("http://www.example.com/0")
-            referrer:web::Referrer()
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.example.com/0")
+                     referrer:web::Referrer()
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ commitPendingItem];
   [session_controller_
-      addPendingItem:GURL("http://www.example.com/1")
-            referrer:web::Referrer()
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.example.com/1")
+                     referrer:web::Referrer()
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ commitPendingItem];
   [session_controller_
-      addPendingItem:GURL("http://www.example.com/2")
-            referrer:web::Referrer()
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.example.com/2")
+                     referrer:web::Referrer()
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ commitPendingItem];
   ASSERT_EQ(3U, [session_controller_ items].size());
 
   // Go to the middle, and commit first pending item index.
   [session_controller_ goToItemAtIndex:1];
   [session_controller_ setPendingItemIndex:0];
   ASSERT_EQ(0, [session_controller_ pendingItemIndex]);
   web::NavigationItem* pending_item = [session_controller_ pendingItem];
   ASSERT_TRUE(pending_item);
   ASSERT_EQ(1, [session_controller_ lastCommittedItemIndex]);
@@ skipping 15 matching lines @@
   [session_controller_ discardNonCommittedItems];
 
   EXPECT_TRUE([session_controller_ items].empty());
   EXPECT_FALSE([session_controller_ currentItem]);
 }
 
 TEST_F(CRWSessionControllerTest,
        DiscardPendingItemWithoutPendingItemWithCommittedItem) {
   // Setup committed item
   [session_controller_
-      addPendingItem:GURL("http://www.url.com")
-            referrer:MakeReferrer("http://www.referer.com")
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.url.com")
+                     referrer:MakeReferrer("http://www.referer.com")
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ commitPendingItem];
 
   // Discard noncommitted items when there is no such one
   [session_controller_ discardNonCommittedItems];
 
   EXPECT_EQ(1U, [session_controller_ items].size());
   EXPECT_EQ([session_controller_ items].front().get(),
             [session_controller_ currentItem]);
 }
 
 TEST_F(CRWSessionControllerTest, updatePendingItemWithoutPendingItem) {
   [session_controller_ updatePendingItem:GURL("http://www.another.url.com")];
   [session_controller_ commitPendingItem];
 
   EXPECT_TRUE([session_controller_ items].empty());
   EXPECT_FALSE([session_controller_ currentItem]);
 }
 
 TEST_F(CRWSessionControllerTest, updatePendingItemWithPendingItem) {
   [session_controller_
-      addPendingItem:GURL("http://www.url.com")
-            referrer:MakeReferrer("http://www.referer.com")
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.url.com")
+                     referrer:MakeReferrer("http://www.referer.com")
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ updatePendingItem:GURL("http://www.another.url.com")];
 
   EXPECT_EQ(
       GURL("http://www.another.url.com/"),
       [session_controller_ currentURL]);
 }
 
 TEST_F(CRWSessionControllerTest,
        updatePendingItemWithPendingItemAlreadyCommited) {
   [session_controller_
-      addPendingItem:GURL("http://www.url.com")
-            referrer:MakeReferrer("http://www.referer.com")
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.url.com")
+                     referrer:MakeReferrer("http://www.referer.com")
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ commitPendingItem];
   [session_controller_ updatePendingItem:GURL("http://www.another.url.com")];
   [session_controller_ commitPendingItem];
 
   EXPECT_EQ(1U, [session_controller_ items].size());
   EXPECT_EQ(GURL("http://www.url.com/"),
             [session_controller_ URLForItemAtIndex:0U]);
   EXPECT_EQ([session_controller_ items].front().get(),
             [session_controller_ currentItem]);
 }
 
 // Tests inserting session controller state.
 TEST_F(CRWSessionControllerTest, CopyState) {
   // Add 1 committed and 1 pending item to target controller.
   [session_controller_
-      addPendingItem:GURL("http://www.url.com/2")
-            referrer:web::Referrer()
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.url.com/2")
+                     referrer:web::Referrer()
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ commitPendingItem];
   [session_controller_
-      addPendingItem:GURL("http://www.url.com/3")
-            referrer:web::Referrer()
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.url.com/3")
+                     referrer:web::Referrer()
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
 
   // Create source session controller with 1 committed item.
   base::scoped_nsobject<CRWSessionController> other_session_controller(
       [[CRWSessionController alloc] initWithBrowserState:&browser_state_]);
   [other_session_controller
-      addPendingItem:GURL("http://www.url.com/0")
-            referrer:web::Referrer()
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.url.com/0")
+                     referrer:web::Referrer()
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [other_session_controller commitPendingItem];
   [other_session_controller
-      addPendingItem:GURL("http://www.url.com/1")
-            referrer:web::Referrer()
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.url.com/1")
+                     referrer:web::Referrer()
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
 
   // Insert and verify the state of target session controller.
   EXPECT_TRUE([session_controller_ canPruneAllButLastCommittedItem]);
   [session_controller_
       copyStateFromSessionControllerAndPrune:other_session_controller.get()];
 
   EXPECT_EQ(2U, [session_controller_ items].size());
   EXPECT_EQ(1, [session_controller_ lastCommittedItemIndex]);
   EXPECT_EQ(-1, [session_controller_ previousItemIndex]);
   EXPECT_EQ(-1, [session_controller_ pendingItemIndex]);
 
   EXPECT_EQ(GURL("http://www.url.com/0"),
             [session_controller_ URLForItemAtIndex:0]);
   EXPECT_EQ(GURL("http://www.url.com/2"),
             [session_controller_ URLForItemAtIndex:1]);
   EXPECT_EQ(GURL("http://www.url.com/3"),
             [session_controller_ pendingItem]->GetURL());
 }
 
 // Tests inserting session controller state from empty session controller.
 TEST_F(CRWSessionControllerTest, CopyStateFromEmptySessionController) {
   // Add 2 committed items to target controller.
   [session_controller_
-      addPendingItem:GURL("http://www.url.com/0")
-            referrer:web::Referrer()
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.url.com/0")
+                     referrer:web::Referrer()
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ commitPendingItem];
   [session_controller_
-      addPendingItem:GURL("http://www.url.com/1")
-            referrer:web::Referrer()
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.url.com/1")
+                     referrer:web::Referrer()
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ commitPendingItem];
 
   // Create empty source session controller.
   base::scoped_nsobject<CRWSessionController> other_session_controller(
       [[CRWSessionController alloc] initWithBrowserState:&browser_state_]);
 
   // Insert and verify the state of target session controller.
   EXPECT_TRUE([session_controller_ canPruneAllButLastCommittedItem]);
   [session_controller_
       copyStateFromSessionControllerAndPrune:other_session_controller.get()];
@@ skipping 10 matching lines @@
 
 // Tests that |-copyStateFromSessionControllerAndPrune:| is a no-op when the
 // receiver has no last committed item.
 TEST_F(CRWSessionControllerTest, CopyStateToEmptySessionController) {
   EXPECT_FALSE([session_controller_ canPruneAllButLastCommittedItem]);
 
   // Create source session controller with 1 committed item.
   base::scoped_nsobject<CRWSessionController> other_session_controller(
       [[CRWSessionController alloc] initWithBrowserState:&browser_state_]);
   [other_session_controller
-      addPendingItem:GURL("http://www.url.com/0")
-            referrer:web::Referrer()
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.url.com/0")
+                     referrer:web::Referrer()
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [other_session_controller commitPendingItem];
   [other_session_controller
-      addPendingItem:GURL("http://www.url.com/1")
-            referrer:web::Referrer()
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.url.com/1")
+                     referrer:web::Referrer()
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
 
   // Attempt to copy |other_session_controller|'s state and verify that
   // |session_controller_| is unchanged.
   [session_controller_
       copyStateFromSessionControllerAndPrune:other_session_controller];
   EXPECT_TRUE([session_controller_ items].empty());
   EXPECT_EQ(-1, [session_controller_ lastCommittedItemIndex]);
   EXPECT_EQ(-1, [session_controller_ previousItemIndex]);
   EXPECT_FALSE([session_controller_ currentItem]);
   EXPECT_FALSE([session_controller_ pendingItem]);
   EXPECT_EQ(-1, [session_controller_ pendingItemIndex]);
 }
 
 // Tests that |-copyStateFromSessionControllerAndPrune:| is a no-op during a
 // pending history navigation.
 TEST_F(CRWSessionControllerTest, CopyStateDuringPendingHistoryNavigation) {
   // Add 1 committed and 1 pending item to target controller.
   [session_controller_
-      addPendingItem:GURL("http://www.url.com/1")
-            referrer:web::Referrer()
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.url.com/1")
+                     referrer:web::Referrer()
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ commitPendingItem];
   [session_controller_
-      addPendingItem:GURL("http://www.url.com/2")
-            referrer:web::Referrer()
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.url.com/2")
+                     referrer:web::Referrer()
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ commitPendingItem];
 
   // Create source session controller with 1 committed item.
   base::scoped_nsobject<CRWSessionController> other_session_controller(
       [[CRWSessionController alloc] initWithBrowserState:&browser_state_]);
   [other_session_controller
-      addPendingItem:GURL("http://www.url.com/0")
-            referrer:web::Referrer()
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.url.com/0")
+                     referrer:web::Referrer()
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [other_session_controller commitPendingItem];
   [other_session_controller
-      addPendingItem:GURL("http://www.url.com/1")
-            referrer:web::Referrer()
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.url.com/1")
+                     referrer:web::Referrer()
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
 
   // Set the pending item index to the first item.
   [session_controller_ setPendingItemIndex:0];
   EXPECT_FALSE([session_controller_ canPruneAllButLastCommittedItem]);
 
   // Attempt to copy |other_session_controller|'s state and verify that
   // |session_controller_| is unchanged.
   [session_controller_
       copyStateFromSessionControllerAndPrune:other_session_controller];
   EXPECT_EQ(2U, [session_controller_ items].size());
   EXPECT_EQ(1, [session_controller_ lastCommittedItemIndex]);
   EXPECT_EQ(0, [session_controller_ previousItemIndex]);
   EXPECT_EQ(0, [session_controller_ pendingItemIndex]);
   EXPECT_TRUE([session_controller_ pendingItem]);
   EXPECT_EQ([session_controller_ previousItem],
             [session_controller_ pendingItem]);
 }
 
 // Tests that |-copyStateFromSessionControllerAndPrune:| is a when a transient
 // NavigationItem exists.
 TEST_F(CRWSessionControllerTest, CopyStateWithTransientItem) {
   // Add 1 committed and 1 pending item to target controller.
   [session_controller_
-      addPendingItem:GURL("http://www.url.com/1")
-            referrer:web::Referrer()
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.url.com/1")
+                     referrer:web::Referrer()
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ commitPendingItem];
   GURL second_url = GURL("http://www.url.com/2");
   [session_controller_
-      addPendingItem:second_url
-            referrer:web::Referrer()
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:second_url
+                     referrer:web::Referrer()
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ addTransientItemWithURL:second_url];
 
   // Create source session controller with 1 committed item.
   base::scoped_nsobject<CRWSessionController> other_session_controller(
       [[CRWSessionController alloc] initWithBrowserState:&browser_state_]);
   [other_session_controller
-      addPendingItem:GURL("http://www.url.com/0")
-            referrer:web::Referrer()
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.url.com/0")
+                     referrer:web::Referrer()
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [other_session_controller commitPendingItem];
   [other_session_controller
-      addPendingItem:GURL("http://www.url.com/1")
-            referrer:web::Referrer()
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.url.com/1")
+                     referrer:web::Referrer()
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
 
   // Attempt to copy |other_session_controller|'s state and verify that
   // |session_controller_| is unchanged.
   EXPECT_FALSE([session_controller_ canPruneAllButLastCommittedItem]);
   [session_controller_
       copyStateFromSessionControllerAndPrune:other_session_controller];
   EXPECT_EQ(1U, [session_controller_ items].size());
   EXPECT_EQ(0, [session_controller_ lastCommittedItemIndex]);
   EXPECT_EQ(-1, [session_controller_ previousItemIndex]);
   EXPECT_EQ(-1, [session_controller_ pendingItemIndex]);
@@ skipping 64 matching lines @@
   EXPECT_EQ(controller.get().previousItemIndex, -1);
   // Sanity check the current item, the NavigationItem unit test will ensure
   // the entire object is created properly.
   EXPECT_EQ([controller currentItem]->GetURL(), GURL("http://www.yahoo.com"));
 }
 
 // Tests index of previous navigation item.
 TEST_F(CRWSessionControllerTest, PreviousNavigationItem) {
   EXPECT_EQ(session_controller_.get().previousItemIndex, -1);
   [session_controller_
-      addPendingItem:GURL("http://www.url.com")
-            referrer:MakeReferrer("http://www.referer.com")
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.url.com")
+                     referrer:MakeReferrer("http://www.referer.com")
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ commitPendingItem];
   EXPECT_EQ(session_controller_.get().previousItemIndex, -1);
   [session_controller_
-      addPendingItem:GURL("http://www.url1.com")
-            referrer:MakeReferrer("http://www.referer.com")
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.url1.com")
+                     referrer:MakeReferrer("http://www.referer.com")
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ commitPendingItem];
   EXPECT_EQ(session_controller_.get().previousItemIndex, 0);
   [session_controller_
-      addPendingItem:GURL("http://www.url2.com")
-            referrer:MakeReferrer("http://www.referer.com")
-          transition:ui::PAGE_TRANSITION_TYPED
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.url2.com")
+                     referrer:MakeReferrer("http://www.referer.com")
+                   transition:ui::PAGE_TRANSITION_TYPED
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ commitPendingItem];
 
   EXPECT_EQ(session_controller_.get().previousItemIndex, 1);
 
   [session_controller_ goToItemAtIndex:1];
   EXPECT_EQ(session_controller_.get().previousItemIndex, 2);
 
   [session_controller_ goToItemAtIndex:0];
   EXPECT_EQ(session_controller_.get().previousItemIndex, 1);
 
@@ skipping 125 matching lines @@
   replacedItem = [controller currentItem];
   EXPECT_EQ(expectedCount, controller.get().items.size());
   EXPECT_EQ(replacePageGurl2, replacedItem->GetURL());
   EXPECT_FALSE(replacedItem->IsCreatedFromPushState());
   EXPECT_NSEQ(nil, replacedItem->GetSerializedStateObject());
   EXPECT_EQ(GURL("http://www.starturl.com/"), replacedItem->GetReferrer().url);
 }
 
 TEST_F(CRWSessionControllerTest, TestBackwardForwardItems) {
   [session_controller_
-      addPendingItem:GURL("http://www.example.com/0")
-            referrer:MakeReferrer("http://www.example.com/a")
-          transition:ui::PAGE_TRANSITION_LINK
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.example.com/0")
+                     referrer:MakeReferrer("http://www.example.com/a")
+                   transition:ui::PAGE_TRANSITION_LINK
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ commitPendingItem];
   [session_controller_
-      addPendingItem:GURL("http://www.example.com/1")
-            referrer:MakeReferrer("http://www.example.com/b")
-          transition:ui::PAGE_TRANSITION_LINK
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.example.com/1")
+                     referrer:MakeReferrer("http://www.example.com/b")
+                   transition:ui::PAGE_TRANSITION_LINK
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ commitPendingItem];
   [session_controller_
-      addPendingItem:GURL("http://www.example.com/redirect")
-            referrer:MakeReferrer("http://www.example.com/r")
-          transition:ui::PAGE_TRANSITION_CLIENT_REDIRECT
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.example.com/redirect")
+                     referrer:MakeReferrer("http://www.example.com/r")
+                   transition:ui::PAGE_TRANSITION_CLIENT_REDIRECT
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ commitPendingItem];
   [session_controller_
-      addPendingItem:GURL("http://www.example.com/2")
-            referrer:MakeReferrer("http://www.example.com/c")
-          transition:ui::PAGE_TRANSITION_LINK
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.example.com/2")
+                     referrer:MakeReferrer("http://www.example.com/c")
+                   transition:ui::PAGE_TRANSITION_LINK
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ commitPendingItem];
 
   EXPECT_EQ(3, session_controller_.get().lastCommittedItemIndex);
   web::NavigationItemList backItems = [session_controller_ backwardItems];
   EXPECT_EQ(2U, backItems.size());
   EXPECT_TRUE([session_controller_ forwardItems].empty());
   EXPECT_EQ("http://www.example.com/redirect", backItems[0]->GetURL().spec());
 
   [session_controller_ goToItemAtIndex:1];
   EXPECT_EQ(1U, [session_controller_ backwardItems].size());
   EXPECT_EQ(1U, [session_controller_ forwardItems].size());
 
   [session_controller_ goToItemAtIndex:0];
   web::NavigationItemList forwardItems = [session_controller_ forwardItems];
   EXPECT_EQ(0U, [session_controller_ backwardItems].size());
   EXPECT_EQ(2U, forwardItems.size());
   EXPECT_EQ("http://www.example.com/2", forwardItems[1]->GetURL().spec());
 }
 
 // Tests going to items with existing and non-existing indices.
 TEST_F(CRWSessionControllerTest, GoToItemAtIndex) {
   [session_controller_
-      addPendingItem:GURL("http://www.example.com/0")
-            referrer:MakeReferrer("http://www.example.com/a")
-          transition:ui::PAGE_TRANSITION_LINK
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.example.com/0")
+                     referrer:MakeReferrer("http://www.example.com/a")
+                   transition:ui::PAGE_TRANSITION_LINK
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ commitPendingItem];
   [session_controller_
-      addPendingItem:GURL("http://www.example.com/1")
-            referrer:MakeReferrer("http://www.example.com/b")
-          transition:ui::PAGE_TRANSITION_LINK
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.example.com/1")
+                     referrer:MakeReferrer("http://www.example.com/b")
+                   transition:ui::PAGE_TRANSITION_LINK
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ commitPendingItem];
   [session_controller_
-      addPendingItem:GURL("http://www.example.com/redirect")
-            referrer:MakeReferrer("http://www.example.com/r")
-          transition:ui::PAGE_TRANSITION_CLIENT_REDIRECT
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.example.com/redirect")
+                     referrer:MakeReferrer("http://www.example.com/r")
+                   transition:ui::PAGE_TRANSITION_CLIENT_REDIRECT
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ commitPendingItem];
   [session_controller_
-      addPendingItem:GURL("http://www.example.com/2")
-            referrer:MakeReferrer("http://www.example.com/c")
-          transition:ui::PAGE_TRANSITION_LINK
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.example.com/2")
+                     referrer:MakeReferrer("http://www.example.com/c")
+                   transition:ui::PAGE_TRANSITION_LINK
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ commitPendingItem];
   [session_controller_
-      addPendingItem:GURL("http://www.example.com/3")
-            referrer:MakeReferrer("http://www.example.com/d")
-          transition:ui::PAGE_TRANSITION_LINK
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.example.com/3")
+                     referrer:MakeReferrer("http://www.example.com/d")
+                   transition:ui::PAGE_TRANSITION_LINK
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ addTransientItemWithURL:GURL("http://www.example.com")];
   EXPECT_EQ(3, session_controller_.get().lastCommittedItemIndex);
   EXPECT_EQ(2, session_controller_.get().previousItemIndex);
   EXPECT_TRUE([session_controller_ pendingItem]);
   EXPECT_TRUE([session_controller_ transientItem]);
 
   // Going back should discard transient and pending items.
   [session_controller_ goToItemAtIndex:1];
   EXPECT_EQ(1, session_controller_.get().lastCommittedItemIndex);
   EXPECT_EQ(3, session_controller_.get().previousItemIndex);
@@ skipping 28 matching lines @@
   [session_controller_ addTransientItemWithURL:GURL("http://www.example.com")];
   web::NavigationItem* visible_item = [session_controller_ visibleItem];
   ASSERT_TRUE(visible_item);
   EXPECT_EQ("http://www.example.com/", visible_item->GetURL().spec());
 }
 
 // Tests that visible URL is the same as transient URL if there is a committed
 // item.
 TEST_F(CRWSessionControllerTest, VisibleItemWithCommittedAndTransientItems) {
   [session_controller_
-      addPendingItem:GURL("http://www.example.com/0")
-            referrer:MakeReferrer("http://www.example.com/a")
-          transition:ui::PAGE_TRANSITION_LINK
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.example.com/0")
+                     referrer:MakeReferrer("http://www.example.com/a")
+                   transition:ui::PAGE_TRANSITION_LINK
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ commitPendingItem];
   [session_controller_ addTransientItemWithURL:GURL("http://www.example.com")];
   web::NavigationItem* visible_item = [session_controller_ visibleItem];
   ASSERT_TRUE(visible_item);
   EXPECT_EQ("http://www.example.com/", visible_item->GetURL().spec());
 }
 
 // Tests that visible URL is the same as pending URL if it was user-initiated.
 TEST_F(CRWSessionControllerTest,
        VisibleItemWithSingleUserInitiatedPendingItem) {
   [session_controller_
-      addPendingItem:GURL("http://www.example.com/0")
-            referrer:MakeReferrer("http://www.example.com/a")
-          transition:ui::PAGE_TRANSITION_LINK
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.example.com/0")
+                     referrer:MakeReferrer("http://www.example.com/a")
+                   transition:ui::PAGE_TRANSITION_LINK
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   web::NavigationItem* visible_item = [session_controller_ visibleItem];
   ASSERT_TRUE(visible_item);
   EXPECT_EQ("http://www.example.com/0", visible_item->GetURL().spec());
 }
 
 // Tests that visible URL is the same as pending URL if it was user-initiated
 // and there is a committed item.
 TEST_F(CRWSessionControllerTest,
        VisibleItemWithCommittedAndUserInitiatedPendingItem) {
   [session_controller_
-      addPendingItem:GURL("http://www.example.com")
-            referrer:MakeReferrer("http://www.example.com/a")
-          transition:ui::PAGE_TRANSITION_LINK
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.example.com")
+                     referrer:MakeReferrer("http://www.example.com/a")
+                   transition:ui::PAGE_TRANSITION_LINK
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ commitPendingItem];
   [session_controller_
-      addPendingItem:GURL("http://www.example.com/0")
-            referrer:MakeReferrer("http://www.example.com/b")
-          transition:ui::PAGE_TRANSITION_LINK
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.example.com/0")
+                     referrer:MakeReferrer("http://www.example.com/b")
+                   transition:ui::PAGE_TRANSITION_LINK
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   web::NavigationItem* visible_item = [session_controller_ visibleItem];
   ASSERT_TRUE(visible_item);
   EXPECT_EQ("http://www.example.com/0", visible_item->GetURL().spec());
 }
 
 // Tests that visible URL is not the same as pending URL if it was
 // renderer-initiated.
 TEST_F(CRWSessionControllerTest,
        VisibleItemWithSingleRendererInitiatedPendingItem) {
   [session_controller_
-      addPendingItem:GURL("http://www.example.com/0")
-            referrer:MakeReferrer("http://www.example.com/a")
-          transition:ui::PAGE_TRANSITION_LINK
-      initiationType:web::NavigationInitiationType::RENDERER_INITIATED];
+               addPendingItem:GURL("http://www.example.com/0")
+                     referrer:MakeReferrer("http://www.example.com/a")
+                   transition:ui::PAGE_TRANSITION_LINK
+               initiationType:web::NavigationInitiationType::RENDERER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   web::NavigationItem* visible_item = [session_controller_ visibleItem];
   ASSERT_FALSE(visible_item);
 }
 
 // Tests that visible URL is not the same as pending URL if it was
 // renderer-initiated and there is a committed item.
 TEST_F(CRWSessionControllerTest,
        VisibleItemWithCommittedAndRendererInitiatedPendingItem) {
   [session_controller_
-      addPendingItem:GURL("http://www.example.com")
-            referrer:MakeReferrer("http://www.example.com/a")
-          transition:ui::PAGE_TRANSITION_LINK
-      initiationType:web::NavigationInitiationType::RENDERER_INITIATED];
+               addPendingItem:GURL("http://www.example.com")
+                     referrer:MakeReferrer("http://www.example.com/a")
+                   transition:ui::PAGE_TRANSITION_LINK
+               initiationType:web::NavigationInitiationType::RENDERER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ commitPendingItem];
   [session_controller_
-      addPendingItem:GURL("http://www.example.com/0")
-            referrer:MakeReferrer("http://www.example.com/b")
-          transition:ui::PAGE_TRANSITION_LINK
-      initiationType:web::NavigationInitiationType::RENDERER_INITIATED];
+               addPendingItem:GURL("http://www.example.com/0")
+                     referrer:MakeReferrer("http://www.example.com/b")
+                   transition:ui::PAGE_TRANSITION_LINK
+               initiationType:web::NavigationInitiationType::RENDERER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   web::NavigationItem* visible_item = [session_controller_ visibleItem];
   ASSERT_TRUE(visible_item);
   EXPECT_EQ("http://www.example.com/", visible_item->GetURL().spec());
 }
 
 // Tests that visible URL is not the same as pending URL created via pending
 // navigation index.
 TEST_F(CRWSessionControllerTest, VisibleItemWithPendingNavigationIndex) {
   [session_controller_
-      addPendingItem:GURL("http://www.example.com")
-            referrer:MakeReferrer("http://www.example.com/a")
-          transition:ui::PAGE_TRANSITION_LINK
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.example.com")
+                     referrer:MakeReferrer("http://www.example.com/a")
+                   transition:ui::PAGE_TRANSITION_LINK
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ commitPendingItem];
   [session_controller_
-      addPendingItem:GURL("http://www.example.com/0")
-            referrer:MakeReferrer("http://www.example.com/b")
-          transition:ui::PAGE_TRANSITION_LINK
-      initiationType:web::NavigationInitiationType::USER_INITIATED];
+               addPendingItem:GURL("http://www.example.com/0")
+                     referrer:MakeReferrer("http://www.example.com/b")
+                   transition:ui::PAGE_TRANSITION_LINK
+               initiationType:web::NavigationInitiationType::USER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ commitPendingItem];
 
   [session_controller_ setPendingItemIndex:0];
 
   web::NavigationItem* visible_item = [session_controller_ visibleItem];
   ASSERT_TRUE(visible_item);
   EXPECT_EQ("http://www.example.com/0", visible_item->GetURL().spec());
 }
 
 // Tests that |-backwardItems| is empty if all preceding items are
 // redirects.
 TEST_F(CRWSessionControllerTest, BackwardItemsForAllRedirects) {
   [session_controller_
-      addPendingItem:GURL("http://www.example.com")
-            referrer:MakeReferrer("http://www.example.com/a")
-          transition:ui::PAGE_TRANSITION_CLIENT_REDIRECT
-      initiationType:web::NavigationInitiationType::RENDERER_INITIATED];
+               addPendingItem:GURL("http://www.example.com")
+                     referrer:MakeReferrer("http://www.example.com/a")
+                   transition:ui::PAGE_TRANSITION_CLIENT_REDIRECT
+               initiationType:web::NavigationInitiationType::RENDERER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ commitPendingItem];
   [session_controller_
-      addPendingItem:GURL("http://www.example.com/0")
-            referrer:MakeReferrer("http://www.example.com/b")
-          transition:ui::PAGE_TRANSITION_CLIENT_REDIRECT
-      initiationType:web::NavigationInitiationType::RENDERER_INITIATED];
+               addPendingItem:GURL("http://www.example.com/0")
+                     referrer:MakeReferrer("http://www.example.com/b")
+                   transition:ui::PAGE_TRANSITION_CLIENT_REDIRECT
+               initiationType:web::NavigationInitiationType::RENDERER_INITIATED
+      userAgentOverrideOption:web::NavigationManager::UserAgentOverrideOption::
+                                  INHERIT];
   [session_controller_ commitPendingItem];
   EXPECT_EQ(0U, [session_controller_ backwardItems].size());
 }
 

[Eugene But (OOO till 7-30), 2017/04/06 22:53:17]

There was a change in session controller logic. Does NavigationManagerImpTest
cover that logic?

[liaoyuke, 2017/04/07 15:49:08]

Yes.

 }  // anonymous namespace