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

Unified Diff: ios/clean/chrome/browser/ui/tab_strip/tab_strip_container_view_controller.mm

Issue 2709013002: [ios clean] Rename removeChildViewController -> detachChildViewController (Closed)
Patch Set: Created 3 years, 10 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/clean/chrome/browser/ui/tab_strip/tab_strip_container_view_controller.mm
diff --git a/ios/clean/chrome/browser/ui/tab_strip/tab_strip_container_view_controller.mm b/ios/clean/chrome/browser/ui/tab_strip/tab_strip_container_view_controller.mm
index 0fcdfe743d3923792fcf0e9c34656ea640328993..e29d6a950291a8884cebaa1278d4aa7c8873850d 100644
--- a/ios/clean/chrome/browser/ui/tab_strip/tab_strip_container_view_controller.mm
+++ b/ios/clean/chrome/browser/ui/tab_strip/tab_strip_container_view_controller.mm
@@ -37,7 +37,7 @@ CGFloat kStripHeight = 200.0;
@property(nonatomic, strong) NSLayoutConstraint* stripHeightConstraint;
// Contained view controller utility methods.
-- (void)removeChildViewController:(UIViewController*)viewController;
+- (void)removeChildViewControllerFromParent:(UIViewController*)viewController;
// Called after a new content view controller is set, but before
// |-didMoveToParentViewController:| is called on that view controller.
@@ -76,7 +76,7 @@ CGFloat kStripHeight = 200.0;
[NSLayoutConstraint
deactivateConstraints:self.contentConstraintsWithoutStrip];
[NSLayoutConstraint deactivateConstraints:self.contentConstraintsWithStrip];
- [self removeChildViewController:self.contentViewController];
+ [self removeChildViewControllerFromParent:self.contentViewController];
// Add the new content view controller.
[self addChildViewController:contentViewController];
@@ -95,7 +95,7 @@ CGFloat kStripHeight = 200.0;
// Remove the current strip view controller, if any.
[NSLayoutConstraint deactivateConstraints:self.stripConstraints];
[NSLayoutConstraint deactivateConstraints:self.contentConstraintsWithStrip];
- [self removeChildViewController:self.tabStripViewController];
+ [self removeChildViewControllerFromParent:self.tabStripViewController];
// Add the new strip view controller.
[self addChildViewController:tabStripViewController];
@@ -212,7 +212,7 @@ CGFloat kStripHeight = 200.0;
#pragma mark - Private methods
-- (void)removeChildViewController:(UIViewController*)viewController {
+- (void)removeChildViewControllerFromParent:(UIViewController*)viewController {
if (viewController.parentViewController != self)
return;
[viewController willMoveToParentViewController:nil];

Powered by Google App Engine
This is Rietveld 408576698