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

Unified Diff: chrome/browser/sync/sessions/sync_sessions_router_tab_helper.cc

Issue 2824073002: [sync] Prevent sessions flare on startup (Closed)
Patch Set: Disable test on android Created 3 years, 8 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: chrome/browser/sync/sessions/sync_sessions_router_tab_helper.cc
diff --git a/chrome/browser/sync/sessions/sync_sessions_router_tab_helper.cc b/chrome/browser/sync/sessions/sync_sessions_router_tab_helper.cc
index 38298ff75e2079081dc3304ff7a94fe8c32d3664..2a84102bc3b1ae98e63a66dcfb03795e22d1d3ef 100644
--- a/chrome/browser/sync/sessions/sync_sessions_router_tab_helper.cc
+++ b/chrome/browser/sync/sessions/sync_sessions_router_tab_helper.cc
@@ -52,7 +52,7 @@ void SyncSessionsRouterTabHelper::WebContentsDestroyed() {
void SyncSessionsRouterTabHelper::DidFinishLoad(
content::RenderFrameHost* render_frame_host,
const GURL& validated_url) {
- NotifyRouter();
+ NotifyRouter(true);
}
void SyncSessionsRouterTabHelper::DidOpenRequestedURL(
@@ -74,9 +74,9 @@ void SyncSessionsRouterTabHelper::DidOpenRequestedURL(
NotifyRouter();
}
-void SyncSessionsRouterTabHelper::NotifyRouter() {
+void SyncSessionsRouterTabHelper::NotifyRouter(bool page_load_completed) {
if (router_)
- router_->NotifyTabModified(web_contents());
+ router_->NotifyTabModified(web_contents(), page_load_completed);
}
} // namespace sync_sessions

Powered by Google App Engine
This is Rietveld 408576698