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

Side by Side Diff: ios/clean/chrome/browser/ui/tab/tab_coordinator.mm

Issue 2948463002: [iOS Clean] Removed old overlay implementation.
Patch Set: rebase & Mark's comments Created 3 years, 6 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #import "ios/clean/chrome/browser/ui/tab/tab_coordinator.h" 5 #import "ios/clean/chrome/browser/ui/tab/tab_coordinator.h"
6 6
7 #include <memory> 7 #include <memory>
8 8
9 #include "base/mac/foundation_util.h" 9 #include "base/mac/foundation_util.h"
10 #include "base/memory/ptr_util.h" 10 #include "base/memory/ptr_util.h"
(...skipping 116 matching lines...) Expand 10 before | Expand all | Expand 10 after
127 127
128 - (void)childCoordinatorWillStop:(BrowserCoordinator*)childCoordinator { 128 - (void)childCoordinatorWillStop:(BrowserCoordinator*)childCoordinator {
129 if ([childCoordinator isKindOfClass:[FindInPageCoordinator class]]) { 129 if ([childCoordinator isKindOfClass:[FindInPageCoordinator class]]) {
130 self.viewController.findBarViewController = nil; 130 self.viewController.findBarViewController = nil;
131 } else if ([childCoordinator isKindOfClass:[WebCoordinator class]] || 131 } else if ([childCoordinator isKindOfClass:[WebCoordinator class]] ||
132 [childCoordinator isKindOfClass:[NTPCoordinator class]]) { 132 [childCoordinator isKindOfClass:[NTPCoordinator class]]) {
133 self.viewController.contentViewController = nil; 133 self.viewController.contentViewController = nil;
134 } 134 }
135 } 135 }
136 136
137 - (BOOL)canAddOverlayCoordinator:(BrowserCoordinator*)overlayCoordinator {
138 // This coordinator will always accept overlay coordinators.
139 return YES;
140 }
141
142 #pragma mark - Experiment support 137 #pragma mark - Experiment support
143 138
144 - (BOOL)usesBottomToolbar { 139 - (BOOL)usesBottomToolbar {
145 NSUserDefaults* defaults = [NSUserDefaults standardUserDefaults]; 140 NSUserDefaults* defaults = [NSUserDefaults standardUserDefaults];
146 NSString* bottomToolbarPreference = 141 NSString* bottomToolbarPreference =
147 [defaults stringForKey:@"EnableBottomToolbar"]; 142 [defaults stringForKey:@"EnableBottomToolbar"];
148 return [bottomToolbarPreference isEqualToString:@"Enabled"]; 143 return [bottomToolbarPreference isEqualToString:@"Enabled"];
149 } 144 }
150 145
151 // Creates and returns a new view controller for use as a tab container; 146 // Creates and returns a new view controller for use as a tab container;
(...skipping 58 matching lines...) Expand 10 before | Expand all | Expand 10 after
210 self.webState->GetNavigationManager()->LoadURLWithParams(params); 205 self.webState->GetNavigationManager()->LoadURLWithParams(params);
211 } 206 }
212 207
213 #pragma mark - TabStripCommands 208 #pragma mark - TabStripCommands
214 209
215 - (void)showTabStrip { 210 - (void)showTabStrip {
216 self.viewController.tabStripVisible = YES; 211 self.viewController.tabStripVisible = YES;
217 } 212 }
218 213
219 @end 214 @end
OLDNEW
« no previous file with comments | « ios/clean/chrome/browser/ui/settings/settings_coordinator.mm ('k') | ios/clean/chrome/browser/ui/tab_grid/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698