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

Unified Diff: chrome/browser/sessions/session_restore.cc

Issue 2930013005: [Tab Metrics] Measure FP, FCP and FMP for Foreground Tab during Session Restore (Closed)
Patch Set: Revise session restore start signal. Created 3 years, 5 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 | « chrome/browser/resource_coordinator/tab_manager.cc ('k') | chrome/browser/sessions/tab_loader.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/sessions/session_restore.cc
diff --git a/chrome/browser/sessions/session_restore.cc b/chrome/browser/sessions/session_restore.cc
index 49831a27f087a8f5ec11c6b7b8b9b58499f6cffd..eed1b4b4ee1e299ee3badeddea85d93a880f9d58 100644
--- a/chrome/browser/sessions/session_restore.cc
+++ b/chrome/browser/sessions/session_restore.cc
@@ -131,6 +131,8 @@ class SessionRestoreImpl : public content::NotificationObserver {
keep_alive_.reset(new ScopedKeepAlive(KeepAliveOrigin::SESSION_RESTORE,
KeepAliveRestartOption::DISABLED));
+ for (auto& observer : SessionRestore::observers())
+ observer.OnSessionRestoreStartedLoadingTabs();
ducbui 2017/07/04 05:34:39 I move the start of session restore to here. There
ducbui 2017/07/04 15:28:25 The revision of the session restore start signal d
}
bool synchronous() const { return synchronous_; }
« no previous file with comments | « chrome/browser/resource_coordinator/tab_manager.cc ('k') | chrome/browser/sessions/tab_loader.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698