Implement first part of supporting global extension commands.

chrome/browser/performance_monitor/startup_timer.cc

 // 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.
 
 #include "chrome/browser/performance_monitor/startup_timer.h"
 
 #include "base/bind.h"
 #include "base/logging.h"
 #include "base/strings/string_number_conversions.h"
 #include "chrome/browser/chrome_notification_types.h"
@@ skipping 92 matching lines @@
 void StartupTimer::SetElapsedSessionRestoreTime(
     const base::TimeDelta& elapsed_session_restore_time) {
   g_startup_timer_->elapsed_session_restore_times_.push_back(
       elapsed_session_restore_time);
 
   if (g_startup_timer_->performance_monitor_initialized_)
     g_startup_timer_->InsertElapsedSessionRestoreTime();
 }
 
 void StartupTimer::InsertElapsedStartupTime() {
+  performance_monitor::Database* database =

[Yoyo Zhou, 2013/10/03 22:08:50]

You should be able to remove this now.

[Finnur, 2013/10/04 17:57:02]

Indeed.

+      PerformanceMonitor::GetInstance()->database();
+  // TODO(finnur): This is temporary to work around bug 303719 where the
+  // startup timer is taking requests to log without the performance monitor
+  // having created a database to store it.
+  if (!database)
+    return;
   content::BrowserThread::PostBlockingPoolSequencedTask(
       Database::kDatabaseSequenceToken,
       FROM_HERE,
       base::Bind(
           &AddMetricToDatabaseOnBackgroundThread,
-          base::Unretained(PerformanceMonitor::GetInstance()->database()),
+          base::Unretained(database),
           Metric(startup_type_ == STARTUP_NORMAL ? METRIC_STARTUP_TIME
                                                  : METRIC_TEST_STARTUP_TIME,
                  base::Time::Now(),
                  static_cast<double>(
                      elapsed_startup_time_.ToInternalValue()))));
 }
 
 void StartupTimer::InsertElapsedSessionRestoreTime() {
   for (std::vector<base::TimeDelta>::const_iterator iter =
            elapsed_session_restore_times_.begin();
        iter != elapsed_session_restore_times_.end(); ++iter) {
+    performance_monitor::Database* database =
+        PerformanceMonitor::GetInstance()->database();
+    // TODO(finnur): This is temporary to work around bug 303719 where the
+    // startup timer is taking requests to log without the performance monitor
+    // having created a database to store it.
+    if (!database)
+      return;
     content::BrowserThread::PostBlockingPoolSequencedTask(
         Database::kDatabaseSequenceToken,
         FROM_HERE,
         base::Bind(
             &AddMetricToDatabaseOnBackgroundThread,
-            base::Unretained(PerformanceMonitor::GetInstance()->database()),
+            base::Unretained(database),
             Metric(METRIC_SESSION_RESTORE_TIME,
                    base::Time::Now(),
                    static_cast<double>(iter->ToInternalValue()))));
   }
 }
 
 }  // namespace performance_monitor