Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(91)

Unified Diff: ios/chrome/browser/web_state_list/web_state_list_unittest.mm

Issue 2799723002: [ios] Move WebStateList to ios/chrome/browser/web_state_list. (Closed)
Patch Set: Rebase. Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ios/chrome/browser/web_state_list/web_state_list_unittest.mm
diff --git a/ios/shared/chrome/browser/tabs/web_state_list_unittest.mm b/ios/chrome/browser/web_state_list/web_state_list_unittest.mm
similarity index 90%
rename from ios/shared/chrome/browser/tabs/web_state_list_unittest.mm
rename to ios/chrome/browser/web_state_list/web_state_list_unittest.mm
index a786270f700803b1e8b7da895dec4de410eb4e05..5bbade1b71d445f1ee29c72c0cbe559c002a954a 100644
--- a/ios/shared/chrome/browser/tabs/web_state_list_unittest.mm
+++ b/ios/chrome/browser/web_state_list/web_state_list_unittest.mm
@@ -2,14 +2,14 @@
// 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"
+#import "ios/chrome/browser/web_state_list/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/chrome/browser/web_state_list/fake_web_state_list_delegate.h"
+#import "ios/chrome/browser/web_state_list/web_state_list_observer.h"
+#import "ios/chrome/browser/web_state_list/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"
@@ -416,30 +416,24 @@ TEST_F(WebStateListTest, OpenersChildsAfterOpener) {
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, web_state_list_.GetIndexOfNextWebStateOpenedBy(
+ opener, start_index, false));
+ EXPECT_EQ(2, web_state_list_.GetIndexOfLastWebStateOpenedBy(
+ opener, start_index, false));
- EXPECT_EQ(1,
- web_state_list_.GetIndexOfNextWebStateOpenedBy(opener, start_index,
- true));
- EXPECT_EQ(2,
- web_state_list_.GetIndexOfLastWebStateOpenedBy(opener, start_index,
- true));
+ EXPECT_EQ(1, web_state_list_.GetIndexOfNextWebStateOpenedBy(
+ opener, start_index, true));
+ EXPECT_EQ(2, web_state_list_.GetIndexOfLastWebStateOpenedBy(
+ opener, start_index, true));
// Simulate a navigation on the opener, results should not change if not
// using groups, but should now be kInvalidIndex otherwise.
opener->GetNavigationManager()->GoForward();
- 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, web_state_list_.GetIndexOfNextWebStateOpenedBy(
+ opener, start_index, false));
+ EXPECT_EQ(2, web_state_list_.GetIndexOfLastWebStateOpenedBy(
+ opener, start_index, false));
EXPECT_EQ(WebStateList::kInvalidIndex,
web_state_list_.GetIndexOfNextWebStateOpenedBy(opener, start_index,
@@ -454,19 +448,15 @@ TEST_F(WebStateListTest, OpenersChildsAfterOpener) {
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(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));
+ 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) {

Powered by Google App Engine
This is Rietveld 408576698