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

Unified Diff: ios/chrome/browser/tabs/tab_model_synced_window_delegate_getter.h

Issue 2628273004: Revert of Use ChromeBrowserStateManager instead of BrowserListIOS. (Closed)
Patch Set: 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/chrome/browser/tabs/tab_model_synced_window_delegate_getter.h
diff --git a/ios/chrome/browser/tabs/tab_model_synced_window_delegate_getter.h b/ios/chrome/browser/tabs/tab_model_synced_window_delegate_getter.h
index 728d14485279e34ea95986067125cbc3cd28068d..922dce17bf53547bf19025f6ed259bb1f7bad529 100644
--- a/ios/chrome/browser/tabs/tab_model_synced_window_delegate_getter.h
+++ b/ios/chrome/browser/tabs/tab_model_synced_window_delegate_getter.h
@@ -15,19 +15,28 @@
class SyncedWindowDelegate;
}
+namespace ios {
+class ChromeBrowserState;
+}
+
class TabModelSyncedWindowDelegatesGetter
: public sync_sessions::SyncedWindowDelegatesGetter {
public:
- TabModelSyncedWindowDelegatesGetter();
+ // TODO(crbug.com/548612): |browser_state| may be unnecessary as iOS does not
+ // supports multi-profile starting with M47. Should it be removed?
+ explicit TabModelSyncedWindowDelegatesGetter(
+ ios::ChromeBrowserState* browser_state);
~TabModelSyncedWindowDelegatesGetter() override;
// sync_sessions::SyncedWindowDelegatesGetter:
std::set<const sync_sessions::SyncedWindowDelegate*>
GetSyncedWindowDelegates() override;
const sync_sessions::SyncedWindowDelegate* FindById(
- SessionID::id_type session_id) override;
+ SessionID::id_type id) override;
private:
+ const ios::ChromeBrowserState* const browser_state_;
+
DISALLOW_COPY_AND_ASSIGN(TabModelSyncedWindowDelegatesGetter);
};

Powered by Google App Engine
This is Rietveld 408576698