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

Unified Diff: chrome/browser/performance_monitor/startup_timer.h

Issue 10834015: Add Startup Timing to CPM (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Requested changes added Created 8 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
Index: chrome/browser/performance_monitor/startup_timer.h
diff --git a/chrome/browser/performance_monitor/startup_timer.h b/chrome/browser/performance_monitor/startup_timer.h
new file mode 100644
index 0000000000000000000000000000000000000000..449bbda3d56f4d152fa708f57d1eb9c68e5c0aa2
--- /dev/null
+++ b/chrome/browser/performance_monitor/startup_timer.h
@@ -0,0 +1,106 @@
+// Copyright (c) 2012 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CHROME_BROWSER_PERFORMANCE_MONITOR_STARTUP_TIMER_H_
+#define CHROME_BROWSER_PERFORMANCE_MONITOR_STARTUP_TIMER_H_
+
+#include "base/time.h"
+#include "content/public/browser/notification_observer.h"
+#include "content/public/browser/notification_registrar.h"
+
+namespace performance_monitor {
+
+// This class is responsible for recording the startup and session restore times
+// (if applicable) for PerformanceMonitor. This allows us to initialize this
+// relatively small object early in the startup process, and start the whole of
+// PerformanceMonitor at a later time. StartupTimer will record the times and
+// insert them into PerformanceMonitor's database.
+class StartupTimer : public content::NotificationObserver {
+ public:
+ // Indicates the type of startup; i.e. either a normal startup or a testing
+ // environment.
+ enum StartupType {
+ STARTUP_NORMAL,
+ STARTUP_TEST
+ };
+
+ StartupTimer();
+ virtual ~StartupTimer();
+
+ // Inform StartupTimer that the startup process has been completed; stop the
+ // timer. Returns false if the timer has already stopped.
+ bool SignalStartupComplete(StartupType startup_type);
+
+ // Pauses the timer until UnpauseTimer() is called; any time spent within a
+ // pause does not count towards the measured startup time. Returns false if
+ // PauseTimer() is called while paused or UnpauseTimer() is called while
+ // unpaused.
+ static bool PauseTimer();
+ static bool UnpauseTimer();
+
+ // content::NotificationObserver
+ // We keep track of whether or not PerformanceMonitor has been started via
+ // the PERFORMANCE_MONITOR_INITIALIZED notification; we need to know this so
+ // we know when to insert startup data into the database. We either insert
+ // data as we gather it (if PerformanceMonitor is started prior to data
+ // collection) or at the notification (if PerformanceMonitor is started
+ // later).
+ virtual void Observe(int type,
+ const content::NotificationSource& source,
+ const content::NotificationDetails& details) OVERRIDE;
+
+ static void SetElapsedSessionRestoreTime(
+ const base::TimeDelta& elapsed_session_restore_time);
+
+ private:
+ bool PauseTimerImpl();
+ bool UnpauseTimerImpl();
+
+ // Insert the elapsed time measures into PerformanceMonitor's database.
+ void InsertElapsedStartupTime();
+ void InsertElapsedSessionRestoreTime();
+
+ // The time at which the startup process begins (the creation of
+ // ChromeBrowserMain).
+ base::TimeTicks startup_begin_;
+
+ // The time at which the timer was most recently paused, or null if the timer
+ // is not currently paused.
+ base::TimeTicks pause_started_;
+
+ // The total duration for which the timer has been paused.
+ base::TimeDelta total_pause_;
+
+ // A flag of whether or not this was a "normal" startup (e.g. whether or not
+ // this was in a testing environment, which would change the startup time
+ // values). If it is not a normal startup, we use a different metric.
+ StartupType startup_type_;
+
+ // The total duration of the startup process, minus any pauses.
+ base::TimeDelta elapsed_startup_time_;
+
+ // The total duration of the session restore, if one occurred. This is
+ // independent of the startup time, because:
+ // - If the user has auto-restore on, the restore is synchronous, and we pause
+ // the startup timer during the session restore; the restore will not
+ // interfere with startup timing.
+ // - If Chrome crashed and the user chooses to restore the crashed session,
+ // then the startup is already completed; the restore will not interfere
+ // with startup timing.
+ base::TimeDelta elapsed_session_restore_time_;
+
+ // Flag whether or not PerformanceMonitor has been fully started.
+ bool performance_monitor_initialized_;
+
+ content::NotificationRegistrar registrar_;
+
+ // The singleton of this class.
+ static StartupTimer* g_startup_timer_;
+
+ DISALLOW_COPY_AND_ASSIGN(StartupTimer);
+};
+
+} // namespace performance_monitor
+
+#endif // CHROME_BROWSER_PERFORMANCE_MONITOR_STARTUP_TIMER_H_

Powered by Google App Engine
This is Rietveld 408576698