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

Unified Diff: chrome/browser/metrics/perf_provider_chromeos.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: Add more detailed description of meaning of num tabs, given by sky 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/metrics/perf_provider_chromeos.h
diff --git a/chrome/browser/metrics/perf_provider_chromeos.h b/chrome/browser/metrics/perf_provider_chromeos.h
index d0b066ef4640707c7f7f1e0db116e76cf784d6d5..8353be5f59b32f9574b8566fa119a2fc6f065e26 100644
--- a/chrome/browser/metrics/perf_provider_chromeos.h
+++ b/chrome/browser/metrics/perf_provider_chromeos.h
@@ -61,7 +61,7 @@ class PerfProvider : public base::NonThreadSafe,
void OnUserLoggedIn();
// Called when a session restore has finished.
- void OnSessionRestoreDone();
+ void OnSessionRestoreDone(int num_tabs_restored);
// Turns off perf collection. Does not delete any data that was already
// collected and stored in |cached_perf_data_|.
@@ -87,9 +87,11 @@ class PerfProvider : public base::NonThreadSafe,
const base::TimeDelta& time_after_resume);
// Collects perf data after a session restore. |time_after_restore| is how
- // long ago the session restore started.
+ // long ago the session restore started. |num_tabs_restored| is the total
+ // number of tabs being restored.
void CollectPerfDataAfterSessionRestore(
- const base::TimeDelta& time_after_restore);
+ const base::TimeDelta& time_after_restore,
+ int num_tabs_restored);
// Parses a perf data protobuf from the |data| passed in only if the
// |incognito_observer| indicates that no incognito window had been opened
« no previous file with comments | « chrome/browser/extensions/state_store_notification_observer.cc ('k') | chrome/browser/metrics/perf_provider_chromeos.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698