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

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

Issue 902613003: Record the number of tabs in a session restore in SampledProfile (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 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
Index: chrome/browser/sessions/session_restore.h
diff --git a/chrome/browser/sessions/session_restore.h b/chrome/browser/sessions/session_restore.h
index f756f10a0e6ae9940f161e5a749bd4630f71a451..9d90d314d8f2a69621a889754fac6ac7b314b447 100644
--- a/chrome/browser/sessions/session_restore.h
+++ b/chrome/browser/sessions/session_restore.h
@@ -40,12 +40,12 @@ class SessionRestore {
};
// Notification callback list.
- using CallbackList = base::CallbackList<void(void)>;
+ using CallbackList = base::CallbackList<void(int)>;
// Used by objects calling RegisterOnSessionRestoredCallback() to de-register
// themselves when they are destroyed.
using CallbackSubscription =
- scoped_ptr<base::CallbackList<void(void)>::Subscription>;
+ scoped_ptr<base::CallbackList<void(int)>::Subscription>;
// Restores the last session. |behavior| is a bitmask of Behaviors, see it
// for details. If |browser| is non-null the tabs for the first window are
@@ -94,7 +94,7 @@ class SessionRestore {
// Register callbacks for session restore events. These callbacks are stored
// in on_session_restored_callbacks_.
sky 2015/02/04 21:22:11 You need to document what the int means.
Simon Que 2015/02/05 00:16:01 Done.
static CallbackSubscription RegisterOnSessionRestoredCallback(
- const base::Closure& callback);
+ const base::Callback<void(int)>& callback);
// The max number of non-selected tabs SessionRestore loads when restoring
// a session. A value of 0 indicates all tabs are loaded at once.

Powered by Google App Engine
This is Rietveld 408576698