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

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

Issue 2615003002: Use ChromeBrowserStateManager instead of BrowserListIOS. (Closed)
Patch Set: Fix ios_chrome_perftests. 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 922dce17bf53547bf19025f6ed259bb1f7bad529..728d14485279e34ea95986067125cbc3cd28068d 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,28 +15,19 @@ namespace browser_sync {
class SyncedWindowDelegate;
}
-namespace ios {
-class ChromeBrowserState;
-}
-
class TabModelSyncedWindowDelegatesGetter
: public sync_sessions::SyncedWindowDelegatesGetter {
public:
- // 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();
~TabModelSyncedWindowDelegatesGetter() override;
// sync_sessions::SyncedWindowDelegatesGetter:
std::set<const sync_sessions::SyncedWindowDelegate*>
GetSyncedWindowDelegates() override;
const sync_sessions::SyncedWindowDelegate* FindById(
- SessionID::id_type id) override;
+ SessionID::id_type session_id) override;
private:
- const ios::ChromeBrowserState* const browser_state_;
-
DISALLOW_COPY_AND_ASSIGN(TabModelSyncedWindowDelegatesGetter);
};

Powered by Google App Engine
This is Rietveld 408576698