[ios] Change API to inform WebStateList of opener-opened relationship.

ios/shared/chrome/browser/tabs/web_state_list_unittest.mm

 // Copyright 2017 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/shared/chrome/browser/tabs/web_state_list.h"
 
 #include "base/macros.h"
 #include "base/memory/ptr_util.h"
 #include "base/supports_user_data.h"
 #import "ios/shared/chrome/browser/tabs/fake_web_state_list_delegate.h"
 #import "ios/shared/chrome/browser/tabs/web_state_list_observer.h"
+#import "ios/shared/chrome/browser/tabs/web_state_opener.h"
 #import "ios/web/public/test/fakes/test_navigation_manager.h"
 #import "ios/web/public/test/fakes/test_web_state.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "testing/platform_test.h"
 
 #if !defined(__has_feature) || !__has_feature(objc_arc)
 #error "This file requires ARC support."
 #endif
 
 namespace {
@@ skipping 145 matching lines @@
   }
 
  private:
   DISALLOW_COPY_AND_ASSIGN(WebStateListTest);
 };
 
 TEST_F(WebStateListTest, IsEmpty) {
   EXPECT_EQ(0, web_state_list_.count());
   EXPECT_TRUE(web_state_list_.empty());
 
-  web_state_list_.InsertWebState(0, CreateWebState(kURL0), nullptr);
+  web_state_list_.InsertWebState(0, CreateWebState(kURL0));
 
   EXPECT_TRUE(observer_.web_state_inserted_called());
   EXPECT_EQ(1, web_state_list_.count());
   EXPECT_FALSE(web_state_list_.empty());
 }
 
 TEST_F(WebStateListTest, InsertUrlSingle) {
-  web_state_list_.InsertWebState(0, CreateWebState(kURL0), nullptr);
+  web_state_list_.InsertWebState(0, CreateWebState(kURL0));
 
   EXPECT_TRUE(observer_.web_state_inserted_called());
   EXPECT_EQ(1, web_state_list_.count());
   EXPECT_EQ(kURL0, web_state_list_.GetWebStateAt(0)->GetVisibleURL().spec());
 }
 
 TEST_F(WebStateListTest, InsertUrlMultiple) {
-  web_state_list_.InsertWebState(0, CreateWebState(kURL0), nullptr);
-  web_state_list_.InsertWebState(0, CreateWebState(kURL1), nullptr);
-  web_state_list_.InsertWebState(1, CreateWebState(kURL2), nullptr);
+  web_state_list_.InsertWebState(0, CreateWebState(kURL0));
+  web_state_list_.InsertWebState(0, CreateWebState(kURL1));
+  web_state_list_.InsertWebState(1, CreateWebState(kURL2));
 
   EXPECT_TRUE(observer_.web_state_inserted_called());
   EXPECT_EQ(3, web_state_list_.count());
   EXPECT_EQ(kURL1, web_state_list_.GetWebStateAt(0)->GetVisibleURL().spec());
   EXPECT_EQ(kURL2, web_state_list_.GetWebStateAt(1)->GetVisibleURL().spec());
   EXPECT_EQ(kURL0, web_state_list_.GetWebStateAt(2)->GetVisibleURL().spec());
 }
 
 TEST_F(WebStateListTest, MoveWebStateAtRightByOne) {
-  web_state_list_.InsertWebState(0, CreateWebState(kURL0), nullptr);
-  web_state_list_.InsertWebState(1, CreateWebState(kURL1), nullptr);
-  web_state_list_.InsertWebState(2, CreateWebState(kURL2), nullptr);
+  web_state_list_.InsertWebState(0, CreateWebState(kURL0));
+  web_state_list_.InsertWebState(1, CreateWebState(kURL1));
+  web_state_list_.InsertWebState(2, CreateWebState(kURL2));
 
   // Sanity check before closing WebState.
   EXPECT_EQ(3, web_state_list_.count());
   EXPECT_EQ(kURL0, web_state_list_.GetWebStateAt(0)->GetVisibleURL().spec());
   EXPECT_EQ(kURL1, web_state_list_.GetWebStateAt(1)->GetVisibleURL().spec());
   EXPECT_EQ(kURL2, web_state_list_.GetWebStateAt(2)->GetVisibleURL().spec());
 
   observer_.ResetStatistics();
   web_state_list_.MoveWebStateAt(0, 1);
 
   EXPECT_TRUE(observer_.web_state_moved_called());
   EXPECT_EQ(3, web_state_list_.count());
   EXPECT_EQ(kURL1, web_state_list_.GetWebStateAt(0)->GetVisibleURL().spec());
   EXPECT_EQ(kURL0, web_state_list_.GetWebStateAt(1)->GetVisibleURL().spec());
   EXPECT_EQ(kURL2, web_state_list_.GetWebStateAt(2)->GetVisibleURL().spec());
 }
 
 TEST_F(WebStateListTest, MoveWebStateAtRightByMoreThanOne) {
-  web_state_list_.InsertWebState(0, CreateWebState(kURL0), nullptr);
-  web_state_list_.InsertWebState(1, CreateWebState(kURL1), nullptr);
-  web_state_list_.InsertWebState(2, CreateWebState(kURL2), nullptr);
+  web_state_list_.InsertWebState(0, CreateWebState(kURL0));
+  web_state_list_.InsertWebState(1, CreateWebState(kURL1));
+  web_state_list_.InsertWebState(2, CreateWebState(kURL2));
 
   // Sanity check before closing WebState.
   EXPECT_EQ(3, web_state_list_.count());
   EXPECT_EQ(kURL0, web_state_list_.GetWebStateAt(0)->GetVisibleURL().spec());
   EXPECT_EQ(kURL1, web_state_list_.GetWebStateAt(1)->GetVisibleURL().spec());
   EXPECT_EQ(kURL2, web_state_list_.GetWebStateAt(2)->GetVisibleURL().spec());
 
   observer_.ResetStatistics();
   web_state_list_.MoveWebStateAt(0, 2);
 
   EXPECT_TRUE(observer_.web_state_moved_called());
   EXPECT_EQ(3, web_state_list_.count());
   EXPECT_EQ(kURL1, web_state_list_.GetWebStateAt(0)->GetVisibleURL().spec());
   EXPECT_EQ(kURL2, web_state_list_.GetWebStateAt(1)->GetVisibleURL().spec());
   EXPECT_EQ(kURL0, web_state_list_.GetWebStateAt(2)->GetVisibleURL().spec());
 }
 
 TEST_F(WebStateListTest, MoveWebStateAtLeftByOne) {
-  web_state_list_.InsertWebState(0, CreateWebState(kURL0), nullptr);
-  web_state_list_.InsertWebState(1, CreateWebState(kURL1), nullptr);
-  web_state_list_.InsertWebState(2, CreateWebState(kURL2), nullptr);
+  web_state_list_.InsertWebState(0, CreateWebState(kURL0));
+  web_state_list_.InsertWebState(1, CreateWebState(kURL1));
+  web_state_list_.InsertWebState(2, CreateWebState(kURL2));
 
   // Sanity check before closing WebState.
   EXPECT_EQ(3, web_state_list_.count());
   EXPECT_EQ(kURL0, web_state_list_.GetWebStateAt(0)->GetVisibleURL().spec());
   EXPECT_EQ(kURL1, web_state_list_.GetWebStateAt(1)->GetVisibleURL().spec());
   EXPECT_EQ(kURL2, web_state_list_.GetWebStateAt(2)->GetVisibleURL().spec());
 
   observer_.ResetStatistics();
   web_state_list_.MoveWebStateAt(2, 1);
 
   EXPECT_TRUE(observer_.web_state_moved_called());
   EXPECT_EQ(3, web_state_list_.count());
   EXPECT_EQ(kURL0, web_state_list_.GetWebStateAt(0)->GetVisibleURL().spec());
   EXPECT_EQ(kURL2, web_state_list_.GetWebStateAt(1)->GetVisibleURL().spec());
   EXPECT_EQ(kURL1, web_state_list_.GetWebStateAt(2)->GetVisibleURL().spec());
 }
 
 TEST_F(WebStateListTest, MoveWebStateAtLeftByMoreThanOne) {
-  web_state_list_.InsertWebState(0, CreateWebState(kURL0), nullptr);
-  web_state_list_.InsertWebState(1, CreateWebState(kURL1), nullptr);
-  web_state_list_.InsertWebState(2, CreateWebState(kURL2), nullptr);
+  web_state_list_.InsertWebState(0, CreateWebState(kURL0));
+  web_state_list_.InsertWebState(1, CreateWebState(kURL1));
+  web_state_list_.InsertWebState(2, CreateWebState(kURL2));
 
   // Sanity check before closing WebState.
   EXPECT_EQ(3, web_state_list_.count());
   EXPECT_EQ(kURL0, web_state_list_.GetWebStateAt(0)->GetVisibleURL().spec());
   EXPECT_EQ(kURL1, web_state_list_.GetWebStateAt(1)->GetVisibleURL().spec());
   EXPECT_EQ(kURL2, web_state_list_.GetWebStateAt(2)->GetVisibleURL().spec());
 
   observer_.ResetStatistics();
   web_state_list_.MoveWebStateAt(2, 0);
 
   EXPECT_TRUE(observer_.web_state_moved_called());
   EXPECT_EQ(3, web_state_list_.count());
   EXPECT_EQ(kURL2, web_state_list_.GetWebStateAt(0)->GetVisibleURL().spec());
   EXPECT_EQ(kURL0, web_state_list_.GetWebStateAt(1)->GetVisibleURL().spec());
   EXPECT_EQ(kURL1, web_state_list_.GetWebStateAt(2)->GetVisibleURL().spec());
 }
 
 TEST_F(WebStateListTest, MoveWebStateAtSameIndex) {
-  web_state_list_.InsertWebState(0, CreateWebState(kURL0), nullptr);
-  web_state_list_.InsertWebState(1, CreateWebState(kURL1), nullptr);
-  web_state_list_.InsertWebState(2, CreateWebState(kURL2), nullptr);
+  web_state_list_.InsertWebState(0, CreateWebState(kURL0));
+  web_state_list_.InsertWebState(1, CreateWebState(kURL1));
+  web_state_list_.InsertWebState(2, CreateWebState(kURL2));
 
   // Sanity check before closing WebState.
   EXPECT_EQ(3, web_state_list_.count());
   EXPECT_EQ(kURL0, web_state_list_.GetWebStateAt(0)->GetVisibleURL().spec());
   EXPECT_EQ(kURL1, web_state_list_.GetWebStateAt(1)->GetVisibleURL().spec());
   EXPECT_EQ(kURL2, web_state_list_.GetWebStateAt(2)->GetVisibleURL().spec());
 
   observer_.ResetStatistics();
   web_state_list_.MoveWebStateAt(2, 2);
 
   EXPECT_FALSE(observer_.web_state_moved_called());
   EXPECT_EQ(3, web_state_list_.count());
   EXPECT_EQ(kURL0, web_state_list_.GetWebStateAt(0)->GetVisibleURL().spec());
   EXPECT_EQ(kURL1, web_state_list_.GetWebStateAt(1)->GetVisibleURL().spec());
   EXPECT_EQ(kURL2, web_state_list_.GetWebStateAt(2)->GetVisibleURL().spec());
 }
 
 TEST_F(WebStateListTest, ReplaceWebStateAt) {
-  web_state_list_.InsertWebState(0, CreateWebState(kURL0), nullptr);
-  web_state_list_.InsertWebState(1, CreateWebState(kURL1), nullptr);
+  web_state_list_.InsertWebState(0, CreateWebState(kURL0));
+  web_state_list_.InsertWebState(1, CreateWebState(kURL1));
 
   // Sanity check before replacing WebState.
   EXPECT_EQ(2, web_state_list_.count());
   EXPECT_EQ(kURL0, web_state_list_.GetWebStateAt(0)->GetVisibleURL().spec());
   EXPECT_EQ(kURL1, web_state_list_.GetWebStateAt(1)->GetVisibleURL().spec());
 
   observer_.ResetStatistics();
   std::unique_ptr<web::WebState> old_web_state(
-      web_state_list_.ReplaceWebStateAt(1, CreateWebState(kURL2), nullptr));
+      web_state_list_.ReplaceWebStateAt(1, CreateWebState(kURL2)));
 
   EXPECT_TRUE(observer_.web_state_replaced_called());
   EXPECT_EQ(2, web_state_list_.count());
   EXPECT_EQ(kURL0, web_state_list_.GetWebStateAt(0)->GetVisibleURL().spec());
   EXPECT_EQ(kURL2, web_state_list_.GetWebStateAt(1)->GetVisibleURL().spec());
   EXPECT_EQ(kURL1, old_web_state->GetVisibleURL().spec());
 }
 
 TEST_F(WebStateListTest, DetachWebStateAtIndexBegining) {
-  web_state_list_.InsertWebState(0, CreateWebState(kURL0), nullptr);
-  web_state_list_.InsertWebState(1, CreateWebState(kURL1), nullptr);
-  web_state_list_.InsertWebState(2, CreateWebState(kURL2), nullptr);
+  web_state_list_.InsertWebState(0, CreateWebState(kURL0));
+  web_state_list_.InsertWebState(1, CreateWebState(kURL1));
+  web_state_list_.InsertWebState(2, CreateWebState(kURL2));
 
   // Sanity check before closing WebState.
   EXPECT_EQ(3, web_state_list_.count());
   EXPECT_EQ(kURL0, web_state_list_.GetWebStateAt(0)->GetVisibleURL().spec());
   EXPECT_EQ(kURL1, web_state_list_.GetWebStateAt(1)->GetVisibleURL().spec());
   EXPECT_EQ(kURL2, web_state_list_.GetWebStateAt(2)->GetVisibleURL().spec());
 
   observer_.ResetStatistics();
   std::unique_ptr<web::WebState> old_web_state(
       web_state_list_.DetachWebStateAt(0));
 
   EXPECT_TRUE(observer_.web_state_detached_called());
   EXPECT_EQ(2, web_state_list_.count());
   EXPECT_EQ(kURL1, web_state_list_.GetWebStateAt(0)->GetVisibleURL().spec());
   EXPECT_EQ(kURL2, web_state_list_.GetWebStateAt(1)->GetVisibleURL().spec());
 }
 
 TEST_F(WebStateListTest, DetachWebStateAtIndexMiddle) {
-  web_state_list_.InsertWebState(0, CreateWebState(kURL0), nullptr);
-  web_state_list_.InsertWebState(1, CreateWebState(kURL1), nullptr);
-  web_state_list_.InsertWebState(2, CreateWebState(kURL2), nullptr);
+  web_state_list_.InsertWebState(0, CreateWebState(kURL0));
+  web_state_list_.InsertWebState(1, CreateWebState(kURL1));
+  web_state_list_.InsertWebState(2, CreateWebState(kURL2));
 
   // Sanity check before closing WebState.
   EXPECT_EQ(3, web_state_list_.count());
   EXPECT_EQ(kURL0, web_state_list_.GetWebStateAt(0)->GetVisibleURL().spec());
   EXPECT_EQ(kURL1, web_state_list_.GetWebStateAt(1)->GetVisibleURL().spec());
   EXPECT_EQ(kURL2, web_state_list_.GetWebStateAt(2)->GetVisibleURL().spec());
 
   observer_.ResetStatistics();
   std::unique_ptr<web::WebState> old_web_state(
       web_state_list_.DetachWebStateAt(1));
 
   EXPECT_TRUE(observer_.web_state_detached_called());
   EXPECT_EQ(2, web_state_list_.count());
   EXPECT_EQ(kURL0, web_state_list_.GetWebStateAt(0)->GetVisibleURL().spec());
   EXPECT_EQ(kURL2, web_state_list_.GetWebStateAt(1)->GetVisibleURL().spec());
 }
 
 TEST_F(WebStateListTest, DetachWebStateAtIndexLast) {
-  web_state_list_.InsertWebState(0, CreateWebState(kURL0), nullptr);
-  web_state_list_.InsertWebState(1, CreateWebState(kURL1), nullptr);
-  web_state_list_.InsertWebState(2, CreateWebState(kURL2), nullptr);
+  web_state_list_.InsertWebState(0, CreateWebState(kURL0));
+  web_state_list_.InsertWebState(1, CreateWebState(kURL1));
+  web_state_list_.InsertWebState(2, CreateWebState(kURL2));
 
   // Sanity check before closing WebState.
   EXPECT_EQ(3, web_state_list_.count());
   EXPECT_EQ(kURL0, web_state_list_.GetWebStateAt(0)->GetVisibleURL().spec());
   EXPECT_EQ(kURL1, web_state_list_.GetWebStateAt(1)->GetVisibleURL().spec());
   EXPECT_EQ(kURL2, web_state_list_.GetWebStateAt(2)->GetVisibleURL().spec());
 
   observer_.ResetStatistics();
   std::unique_ptr<web::WebState> old_web_state(
       web_state_list_.DetachWebStateAt(2));
 
   EXPECT_TRUE(observer_.web_state_detached_called());
   EXPECT_EQ(2, web_state_list_.count());
   EXPECT_EQ(kURL0, web_state_list_.GetWebStateAt(0)->GetVisibleURL().spec());
   EXPECT_EQ(kURL1, web_state_list_.GetWebStateAt(1)->GetVisibleURL().spec());
 }
 
 TEST_F(WebStateListTest, OwnershipBorrowed) {
   bool web_state_was_killed = false;
   auto test_web_state = base::MakeUnique<web::TestWebState>();
   test_web_state->SetUserData(
       &kSupportsUserDataDeathGuardKey,
       base::MakeUnique<SupportsUserDataDeathGuard>(&web_state_was_killed));
 
   FakeWebStateListDelegate web_state_list_delegate;
   auto web_state_list = base::MakeUnique<WebStateList>(
       &web_state_list_delegate, WebStateList::WebStateBorrowed);
-  web_state_list->InsertWebState(0, test_web_state.get(), nullptr);
+  web_state_list->InsertWebState(0, test_web_state.get());
   EXPECT_FALSE(web_state_was_killed);
 
   web_state_list.reset();
   EXPECT_FALSE(web_state_was_killed);
 }
 
 TEST_F(WebStateListTest, OwnershipOwned) {
   bool web_state_was_killed = false;
   auto test_web_state = base::MakeUnique<web::TestWebState>();
   test_web_state->SetUserData(
       &kSupportsUserDataDeathGuardKey,
       base::MakeUnique<SupportsUserDataDeathGuard>(&web_state_was_killed));
 
   FakeWebStateListDelegate web_state_list_delegate;
   auto web_state_list = base::MakeUnique<WebStateList>(
       &web_state_list_delegate, WebStateList::WebStateOwned);
-  web_state_list->InsertWebState(0, test_web_state.release(), nullptr);
+  web_state_list->InsertWebState(0, test_web_state.release());
   EXPECT_FALSE(web_state_was_killed);
 
   web_state_list.reset();
   EXPECT_TRUE(web_state_was_killed);
 }
 
 TEST_F(WebStateListTest, OpenersEmptyList) {
   EXPECT_TRUE(web_state_list_.empty());
 
   EXPECT_EQ(WebStateList::kInvalidIndex,
             web_state_list_.GetIndexOfNextWebStateOpenedBy(
                 nullptr, WebStateList::kInvalidIndex, false));
   EXPECT_EQ(WebStateList::kInvalidIndex,
             web_state_list_.GetIndexOfLastWebStateOpenedBy(
                 nullptr, WebStateList::kInvalidIndex, false));
 
   EXPECT_EQ(WebStateList::kInvalidIndex,
             web_state_list_.GetIndexOfNextWebStateOpenedBy(
                 nullptr, WebStateList::kInvalidIndex, true));
   EXPECT_EQ(WebStateList::kInvalidIndex,
             web_state_list_.GetIndexOfLastWebStateOpenedBy(
                 nullptr, WebStateList::kInvalidIndex, true));
 }
 
 TEST_F(WebStateListTest, OpenersNothingOpened) {
-  web_state_list_.InsertWebState(0, CreateWebState(kURL0), nullptr);
-  web_state_list_.InsertWebState(1, CreateWebState(kURL1), nullptr);
-  web_state_list_.InsertWebState(2, CreateWebState(kURL2), nullptr);
+  web_state_list_.InsertWebState(0, CreateWebState(kURL0));
+  web_state_list_.InsertWebState(1, CreateWebState(kURL1));
+  web_state_list_.InsertWebState(2, CreateWebState(kURL2));
 
   for (int index = 0; index < web_state_list_.count(); ++index) {
     web::WebState* opener = web_state_list_.GetWebStateAt(index);
     EXPECT_EQ(
         WebStateList::kInvalidIndex,
         web_state_list_.GetIndexOfNextWebStateOpenedBy(opener, index, false));
     EXPECT_EQ(
         WebStateList::kInvalidIndex,
         web_state_list_.GetIndexOfLastWebStateOpenedBy(opener, index, false));
 
     EXPECT_EQ(
         WebStateList::kInvalidIndex,
         web_state_list_.GetIndexOfNextWebStateOpenedBy(opener, index, true));
     EXPECT_EQ(
         WebStateList::kInvalidIndex,
         web_state_list_.GetIndexOfLastWebStateOpenedBy(opener, index, true));
   }
 }
 
 TEST_F(WebStateListTest, OpenersChildsAfterOpener) {
-  web_state_list_.InsertWebState(0, CreateWebState(kURL0), nullptr);
+  web_state_list_.InsertWebState(0, CreateWebState(kURL0));
   web::WebState* opener = web_state_list_.GetWebStateAt(0);
 
-  web_state_list_.InsertWebState(1, CreateWebState(kURL1), opener);
-  web_state_list_.InsertWebState(2, CreateWebState(kURL2), opener);
+  web_state_list_.InsertWebState(1, CreateWebState(kURL1));
+  web_state_list_.SetOpenerOfWebStateAt(1, WebStateOpener(opener));
+
+  web_state_list_.InsertWebState(2, CreateWebState(kURL2));
+  web_state_list_.SetOpenerOfWebStateAt(2, WebStateOpener(opener));
 
   const int start_index = web_state_list_.GetIndexOfWebState(opener);
   EXPECT_EQ(1,
             web_state_list_.GetIndexOfNextWebStateOpenedBy(opener, start_index,
                                                            false));
   EXPECT_EQ(2,
             web_state_list_.GetIndexOfLastWebStateOpenedBy(opener, start_index,
                                                            false));
 
   EXPECT_EQ(1,
@@ skipping 17 matching lines @@
   EXPECT_EQ(WebStateList::kInvalidIndex,
             web_state_list_.GetIndexOfNextWebStateOpenedBy(opener, start_index,
                                                            true));
   EXPECT_EQ(WebStateList::kInvalidIndex,
             web_state_list_.GetIndexOfLastWebStateOpenedBy(opener, start_index,
                                                            true));
 
   // Add a new WebState with the same opener. It should be considered the next
   // WebState if groups are considered and the last independently on whether
   // groups are used or not.
-  web_state_list_.InsertWebState(3, CreateWebState(kURL2), opener);
+  web_state_list_.InsertWebState(3, CreateWebState(kURL2));
+  web_state_list_.SetOpenerOfWebStateAt(3, WebStateOpener(opener));
 
   EXPECT_EQ(1,
             web_state_list_.GetIndexOfNextWebStateOpenedBy(opener, start_index,
                                                            false));
   EXPECT_EQ(3,
             web_state_list_.GetIndexOfLastWebStateOpenedBy(opener, start_index,
                                                            false));
 
   EXPECT_EQ(3,
             web_state_list_.GetIndexOfNextWebStateOpenedBy(opener, start_index,
                                                            true));
   EXPECT_EQ(3,
             web_state_list_.GetIndexOfLastWebStateOpenedBy(opener, start_index,
                                                            true));
 }
 
 TEST_F(WebStateListTest, OpenersChildsBeforeOpener) {
-  web_state_list_.InsertWebState(0, CreateWebState(kURL0), nullptr);
+  web_state_list_.InsertWebState(0, CreateWebState(kURL0));
   web::WebState* opener = web_state_list_.GetWebStateAt(0);
 
-  web_state_list_.InsertWebState(0, CreateWebState(kURL1), opener);
-  web_state_list_.InsertWebState(1, CreateWebState(kURL2), opener);
+  web_state_list_.InsertWebState(0, CreateWebState(kURL1));
+  web_state_list_.SetOpenerOfWebStateAt(0, WebStateOpener(opener));
+
+  web_state_list_.InsertWebState(1, CreateWebState(kURL2));
+  web_state_list_.SetOpenerOfWebStateAt(1, WebStateOpener(opener));
 
   const int start_index = web_state_list_.GetIndexOfWebState(opener);
   EXPECT_EQ(WebStateList::kInvalidIndex,
             web_state_list_.GetIndexOfNextWebStateOpenedBy(opener, start_index,
                                                            false));
   EXPECT_EQ(WebStateList::kInvalidIndex,
             web_state_list_.GetIndexOfLastWebStateOpenedBy(opener, start_index,
                                                            false));
 
   EXPECT_EQ(WebStateList::kInvalidIndex,
             web_state_list_.GetIndexOfNextWebStateOpenedBy(opener, start_index,
                                                            true));
   EXPECT_EQ(WebStateList::kInvalidIndex,
             web_state_list_.GetIndexOfLastWebStateOpenedBy(opener, start_index,
                                                            true));
 }