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

Side by Side Diff: ios/clean/chrome/browser/browser_coordinator+internal.h

Issue 2755653002: Coordinators are now notified when moving to a parent coordinator. (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 unified diff | Download patch
« no previous file with comments | « ios/clean/chrome/browser/browser_coordinator.mm ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 // ====== New Architecture ===== 5 // ====== New Architecture =====
6 // = This code is only used in the new iOS Chrome architecture. = 6 // = This code is only used in the new iOS Chrome architecture. =
7 // ============================================================================ 7 // ============================================================================
8 8
9 #ifndef IOS_CHROME_BROWSER_BROWSER_COORDINATOR_INTERNAL_H_ 9 #ifndef IOS_CHROME_BROWSER_BROWSER_COORDINATOR_INTERNAL_H_
10 #define IOS_CHROME_BROWSER_BROWSER_COORDINATOR_INTERNAL_H_ 10 #define IOS_CHROME_BROWSER_BROWSER_COORDINATOR_INTERNAL_H_
(...skipping 37 matching lines...) Expand 10 before | Expand all | Expand 10 after
48 48
49 // Adds |coordinator| as a child, taking ownership of it, setting the receiver's 49 // Adds |coordinator| as a child, taking ownership of it, setting the receiver's
50 // viewController (if any) as the child's baseViewController, and setting 50 // viewController (if any) as the child's baseViewController, and setting
51 // the receiver's |browser| as the child's |browser|. 51 // the receiver's |browser| as the child's |browser|.
52 - (void)addChildCoordinator:(BrowserCoordinator*)coordinator; 52 - (void)addChildCoordinator:(BrowserCoordinator*)coordinator;
53 53
54 // Removes |coordinator| as a child, relinquishing ownership of it. If 54 // Removes |coordinator| as a child, relinquishing ownership of it. If
55 // |coordinator| isn't a child of the receiver, this method does nothing. 55 // |coordinator| isn't a child of the receiver, this method does nothing.
56 - (void)removeChildCoordinator:(BrowserCoordinator*)coordinator; 56 - (void)removeChildCoordinator:(BrowserCoordinator*)coordinator;
57 57
58 // Called when this coordinator moves to a new parent coordinator. Subclasses
59 // can override this method to run code that requires a configured
60 // CoordinatorContext.
61 - (void)coordinatorDidMoveToParentCoordinator;
lpromero 2017/03/15 12:15:08 Note that I don't pass the parentCoordinator anymo
62
58 // Called when a child coordinator did start. This is a blank template method. 63 // Called when a child coordinator did start. This is a blank template method.
59 // Subclasses can override this method when they need to know when their 64 // Subclasses can override this method when they need to know when their
60 // children start. 65 // children start.
61 - (void)childCoordinatorDidStart:(BrowserCoordinator*)childCoordinator; 66 - (void)childCoordinatorDidStart:(BrowserCoordinator*)childCoordinator;
62 67
63 // Called when a child coordinator will stop. This is a blank template method. 68 // Called when a child coordinator will stop. This is a blank template method.
64 // Subclasses can override this method when they need to know when their 69 // Subclasses can override this method when they need to know when their
65 // children start. 70 // children start.
66 - (void)childCoordinatorWillStop:(BrowserCoordinator*)childCoordinator; 71 - (void)childCoordinatorWillStop:(BrowserCoordinator*)childCoordinator;
67 72
(...skipping 13 matching lines...) Expand all
81 // nothing happens. 86 // nothing happens.
82 - (void)addOverlayCoordinator:(BrowserCoordinator*)overlayCoordinator; 87 - (void)addOverlayCoordinator:(BrowserCoordinator*)overlayCoordinator;
83 88
84 // Removes the current overlay coordinator (if any) as a child from its 89 // Removes the current overlay coordinator (if any) as a child from its
85 // parent. 90 // parent.
86 - (void)removeOverlayCoordinator; 91 - (void)removeOverlayCoordinator;
87 92
88 @end 93 @end
89 94
90 #endif // IOS_CHROME_BROWSER_BROWSER_COORDINATOR_INTERNAL_H_ 95 #endif // IOS_CHROME_BROWSER_BROWSER_COORDINATOR_INTERNAL_H_
OLDNEW
« no previous file with comments | « ios/clean/chrome/browser/browser_coordinator.mm ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698