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

Unified Diff: ios/chrome/browser/ui/reading_list/reading_list_coordinator.h

Issue 2693863005: Create ReadingListCoordinator (Closed)
Patch Set: Reviewable 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/chrome/browser/ui/reading_list/reading_list_coordinator.h
diff --git a/ios/chrome/browser/ui/reading_list/reading_list_coordinator.h b/ios/chrome/browser/ui/reading_list/reading_list_coordinator.h
new file mode 100644
index 0000000000000000000000000000000000000000..fa54307bfdd7a6e37ce651aba2437b409434a100
--- /dev/null
+++ b/ios/chrome/browser/ui/reading_list/reading_list_coordinator.h
@@ -0,0 +1,34 @@
+// Copyright 2017 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef IOS_CHROME_BROWSER_UI_READING_LIST_READING_LIST_COORDINATOR_H_
+#define IOS_CHROME_BROWSER_UI_READING_LIST_READING_LIST_COORDINATOR_H_
+
+#import "ios/chrome/browser/chrome_coordinator.h"
+#import "ios/chrome/browser/ui/reading_list/reading_list_collection_view_controller.h"
+#import "ios/chrome/browser/ui/reading_list/reading_list_commands.h"
+
+namespace ios {
+class ChromeBrowserState;
+}
+
+@protocol UrlLoader;
+
+// Coordinator for Reading List, displaying the Reading List when starting.
+@interface ReadingListCoordinator
+ : ChromeCoordinator<ReadingListCommands,
+ ReadingListCollectionViewControllerDelegate>
+
+- (nullable instancetype)
+initWithBaseViewController:(nullable UIViewController*)viewController
Olivier 2017/02/16 15:16:09 What is the point of a coordinator without baseVie
gambard 2017/02/16 15:45:56 Test for example. And the baseViewController is on
+ browserState:(nonnull ios::ChromeBrowserState*)browserState
Olivier 2017/02/16 15:16:09 I think it is better to pass the service you depen
gambard 2017/02/16 15:45:56 It seems that the pattern is to pass the BrowserSt
+ loader:(nullable id<UrlLoader>)loader
+ NS_DESIGNATED_INITIALIZER;
+
+- (nullable instancetype)initWithBaseViewController:
+ (nullable UIViewController*)viewController NS_UNAVAILABLE;
+
+@end
+
+#endif // IOS_CHROME_BROWSER_UI_READING_LIST_READING_LIST_COORDINATOR_H_

Powered by Google App Engine
This is Rietveld 408576698