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

Unified Diff: ios/chrome/browser/ui/main/browser_view_wrangler.mm

Issue 2660143002: Removing iPad Tab Switcher experimental flag. (Closed)
Patch Set: Created 3 years, 11 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/ui/main/browser_view_wrangler.mm
diff --git a/ios/chrome/browser/ui/main/browser_view_wrangler.mm b/ios/chrome/browser/ui/main/browser_view_wrangler.mm
index b16f003a763a24dc5fc9bb5235248e5e9d06885b..bd5f41b1d2c51c894d3ea74b6f554cffec57d985 100644
--- a/ios/chrome/browser/ui/main/browser_view_wrangler.mm
+++ b/ios/chrome/browser/ui/main/browser_view_wrangler.mm
@@ -212,16 +212,6 @@
[self updateDeviceSharingManager];
}
-// Called when the number of tabs changes. Updates the switcher button
-// visibility in the various modes on tablet.
-- (void)tabModelDidChangeTabCount:(TabModel*)notifiedTabModel {
- // If in tablet, update the mode switcher icon based on the number of tabs
- // in the incognito tab strip. Doing this all the time simplifies a lot
- // of the state transition logic and setting the property to the same value
- // incurs no re-layout penalty.
- [self updateModeToggle];
-}
-
#pragma mark - Other public methods
- (void)updateDeviceSharingManager {
@@ -290,13 +280,6 @@
}
}
-- (void)updateModeToggle {
- if (IsIPadIdiom()) {
- self.mainBVC.hasModeToggleSwitch = self.otrTabModel.count ? YES : NO;
- self.otrBVC.hasModeToggleSwitch = YES;
- }
-}
-
#pragma mark - Internal methods
- (TabModel*)buildOtrTabModel:(BOOL)empty {

Powered by Google App Engine
This is Rietveld 408576698