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

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

Issue 2626033002: [Clean] Add unittest target and BrowserCoordinator tests. (Closed)
Patch Set: Review feedback. 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 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 // ====== 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 40 matching lines...) Expand 10 before | Expand all | Expand 10 after
51 // the receiver's browserState as the child's browserState. 51 // the receiver's browserState as the child's browserState.
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 // Methods for adding overlay coordinators. 58 // Methods for adding overlay coordinators.
59 59
60 // Returns YES if the receiver will take |overlayCoordinator| as a child. 60 // Returns YES if the receiver will take |overlayCoordinator| as a child.
61 // The default is to return YES only if the receiver has no children. 61 // The default is to return YES only if the receiver has no children, if
62 // the reciever has a nil -overlayCoordinator, and if |overlayCoordinator|
63 // is not already overlaying.
62 - (BOOL)canAddOverlayCoordinator:(BrowserCoordinator*)overlayCoordinator; 64 - (BOOL)canAddOverlayCoordinator:(BrowserCoordinator*)overlayCoordinator;
63 65
64 // Adds |overlayCoordinator| as a child to the receiver, or if it cannot be 66 // Adds |overlayCoordinator| as a child to the receiver, or if it cannot be
65 // added, recursively add it to the receiver's child. If a receiver has 67 // added, recursively add it to the receiver's child. If a receiver has
66 // multiple children and returns YES from -canAddOverlayCoordinator:, it 68 // multiple children and returns YES from -canAddOverlayCoordinator:, it
67 // must override this method to determines how the overlay is added. 69 // must override this method to determines how the overlay is added.
70 // If neither the reciever or any child can add |overlayCoordinator|, then
71 // nothing happens.
68 - (void)addOverlayCoordinator:(BrowserCoordinator*)overlayCoordinator; 72 - (void)addOverlayCoordinator:(BrowserCoordinator*)overlayCoordinator;
69 73
70 // Removes the current overlay coordinator (if any) as a child from its 74 // Removes the current overlay coordinator (if any) as a child from its
71 // parent. 75 // parent.
72 - (void)removeOverlayCoordinator; 76 - (void)removeOverlayCoordinator;
73 77
74 @end 78 @end
75 79
76 #endif // IOS_CHROME_BROWSER_BROWSER_COORDINATOR_INTERNAL_H_ 80 #endif // IOS_CHROME_BROWSER_BROWSER_COORDINATOR_INTERNAL_H_
OLDNEW
« no previous file with comments | « ios/clean/chrome/browser/browser_coordinator.mm ('k') | ios/clean/chrome/browser/browser_coordinator_unittest.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698