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

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

Issue 2712743006: Reland v5 of Sessions Refactor (Closed)
Patch Set: Fix typo 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
« no previous file with comments | « ios/chrome/browser/tabs/tab_model_synced_window_delegate_getter.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ios/chrome/browser/tabs/tab_model_synced_window_delegate_getter.mm
diff --git a/ios/chrome/browser/tabs/tab_model_synced_window_delegate_getter.mm b/ios/chrome/browser/tabs/tab_model_synced_window_delegate_getter.mm
index df84d972776e0b59da8a02a901940af915744cc9..4e24de1ce3487c0e1e5eac62c884d98444845515 100644
--- a/ios/chrome/browser/tabs/tab_model_synced_window_delegate_getter.mm
+++ b/ios/chrome/browser/tabs/tab_model_synced_window_delegate_getter.mm
@@ -16,9 +16,9 @@ TabModelSyncedWindowDelegatesGetter::TabModelSyncedWindowDelegatesGetter() {}
TabModelSyncedWindowDelegatesGetter::~TabModelSyncedWindowDelegatesGetter() {}
-std::set<const sync_sessions::SyncedWindowDelegate*>
+TabModelSyncedWindowDelegatesGetter::SyncedWindowDelegateMap
TabModelSyncedWindowDelegatesGetter::GetSyncedWindowDelegates() {
- std::set<const sync_sessions::SyncedWindowDelegate*> synced_window_delegates;
+ SyncedWindowDelegateMap synced_window_delegates;
std::vector<ios::ChromeBrowserState*> browser_states =
GetApplicationContext()
@@ -31,7 +31,10 @@ TabModelSyncedWindowDelegatesGetter::GetSyncedWindowDelegates() {
GetTabModelsForChromeBrowserState(browser_state);
for (TabModel* tabModel in tabModels) {
if (tabModel.currentTab) {
- synced_window_delegates.insert([tabModel syncedWindowDelegate]);
+ sync_sessions::SyncedWindowDelegate* synced_window_delegate =
+ [tabModel syncedWindowDelegate];
+ synced_window_delegates[synced_window_delegate->GetSessionId()] =
+ synced_window_delegate;
}
}
}
@@ -41,9 +44,9 @@ TabModelSyncedWindowDelegatesGetter::GetSyncedWindowDelegates() {
const sync_sessions::SyncedWindowDelegate*
TabModelSyncedWindowDelegatesGetter::FindById(SessionID::id_type session_id) {
- for (const auto* delegate : GetSyncedWindowDelegates()) {
- if (session_id == delegate->GetSessionId())
- return delegate;
+ for (const auto& iter : GetSyncedWindowDelegates()) {
+ if (session_id == iter.second->GetSessionId())
+ return iter.second;
}
return nullptr;
}
« no previous file with comments | « ios/chrome/browser/tabs/tab_model_synced_window_delegate_getter.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698