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

Unified Diff: chrome/browser/android/foreign_session_helper.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
« no previous file with comments | « no previous file | chrome/browser/extensions/api/sessions/sessions_api.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/android/foreign_session_helper.cc
diff --git a/chrome/browser/android/foreign_session_helper.cc b/chrome/browser/android/foreign_session_helper.cc
index 17006b71dad6c0b7854972b5c587a1d29a19f936..a4177ca4b6f70b934d6b65f571f52de0c1a270f3 100644
--- a/chrome/browser/android/foreign_session_helper.cc
+++ b/chrome/browser/android/foreign_session_helper.cc
@@ -75,7 +75,7 @@ bool ShouldSkipWindow(const sessions::SessionWindow& window) {
bool ShouldSkipSession(const SyncedSession& session) {
for (const auto& window_pair : session.windows) {
- const sessions::SessionWindow& window = *(window_pair.second.get());
+ const sessions::SessionWindow& window = window_pair.second->wrapped_window;
if (!ShouldSkipWindow(window))
return false;
}
@@ -120,7 +120,7 @@ void CopySessionToJava(
const SyncedSession& session,
ScopedJavaLocalRef<jobject>& j_session) {
for (const auto& window_pair : session.windows) {
- const sessions::SessionWindow& window = *(window_pair.second.get());
+ const sessions::SessionWindow& window = window_pair.second->wrapped_window;
if (ShouldSkipWindow(window))
continue;
« no previous file with comments | « no previous file | chrome/browser/extensions/api/sessions/sessions_api.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698