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

Unified Diff: components/sync_sessions/revisit/sessions_page_revisit_observer.cc

Issue 2499023004: [Sync] Introduce SyncedSessionWindow type. (Closed)
Patch Set: Address comments Created 3 years, 9 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: components/sync_sessions/revisit/sessions_page_revisit_observer.cc
diff --git a/components/sync_sessions/revisit/sessions_page_revisit_observer.cc b/components/sync_sessions/revisit/sessions_page_revisit_observer.cc
index ccb603aa25eb55604920c3391ed3b0d4179e04bf..1a99c70a7ddc3105d5cd89f9a52daf3d2c26a025 100644
--- a/components/sync_sessions/revisit/sessions_page_revisit_observer.cc
+++ b/components/sync_sessions/revisit/sessions_page_revisit_observer.cc
@@ -57,7 +57,7 @@ void SessionsPageRevisitObserver::CheckForRevisit(
if (provider_->GetAllForeignSessions(&foreign_sessions)) {
for (const SyncedSession* session : foreign_sessions) {
for (const auto& key_value : session->windows) {
- for (const auto& tab : key_value.second->tabs) {
+ for (const auto& tab : key_value.second->wrapped_window.tabs) {
// These matchers look identical and could easily implement an
// interface and we could iterate through a vector of matchers here.
// However this would cause quite a bit of overhead at the inner most

Powered by Google App Engine
This is Rietveld 408576698