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

Unified Diff: ios/shared/chrome/browser/tabs/web_state_list_unittest.mm

Issue 2766063002: Remove the concept of currentItemIndex (Closed)
Patch Set: Created 3 years, 9 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/shared/chrome/browser/tabs/web_state_list_unittest.mm
diff --git a/ios/shared/chrome/browser/tabs/web_state_list_unittest.mm b/ios/shared/chrome/browser/tabs/web_state_list_unittest.mm
index 2d2d149b6df6dbdb38c8f974efcff2268caf7d6d..babb8c15a20eaef10a7acf33e20d2fd17f9fc100 100644
--- a/ios/shared/chrome/browser/tabs/web_state_list_unittest.mm
+++ b/ios/shared/chrome/browser/tabs/web_state_list_unittest.mm
@@ -111,8 +111,6 @@ void WebStateDetachedAt(WebStateList* web_state_list,
FakeNavigationManager() = default;
// web::NavigationManager implementation.
- int GetCurrentItemIndex() const override { return current_item_index_; }
-
int GetLastCommittedItemIndex() const override { return current_item_index_; }
kkhorimoto 2017/03/21 23:54:16 As long as you're touching this, can we s/current_
liaoyuke 2017/03/22 18:50:37 Done.
bool CanGoBack() const override { return current_item_index_ > 0; }

Powered by Google App Engine
This is Rietveld 408576698