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

Unified 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 side-by-side diff with in-line comments
Download patch
Index: ios/clean/chrome/browser/browser_coordinator+internal.h
diff --git a/ios/clean/chrome/browser/browser_coordinator+internal.h b/ios/clean/chrome/browser/browser_coordinator+internal.h
index a58d3473362de853a70148ebac426f74819e43f2..43b4a5e6d8a7395c8ee1ea5ca8375bcbf7ccf5a4 100644
--- a/ios/clean/chrome/browser/browser_coordinator+internal.h
+++ b/ios/clean/chrome/browser/browser_coordinator+internal.h
@@ -58,13 +58,17 @@
// Methods for adding overlay coordinators.
// Returns YES if the receiver will take |overlayCoordinator| as a child.
-// The default is to return YES only if the receiver has no children.
+// The default is to return YES only if the receiver has no children, if
+// the reciever has a nil -overlayCoordinator, and if |overlayCoordinator|
+// is not already overlaying.
- (BOOL)canAddOverlayCoordinator:(BrowserCoordinator*)overlayCoordinator;
// Adds |overlayCoordinator| as a child to the receiver, or if it cannot be
// added, recursively add it to the receiver's child. If a receiver has
// multiple children and returns YES from -canAddOverlayCoordinator:, it
// must override this method to determines how the overlay is added.
+// If neither the reciever or any child can add |overlayCoordinator|, then
+// nothing happens.
- (void)addOverlayCoordinator:(BrowserCoordinator*)overlayCoordinator;
// Removes the current overlay coordinator (if any) as a child from its
« 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