Add Startup Timing to CPM

chrome/browser/performance_monitor/metric_details.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/metric_details.h"
 
 #include "base/logging.h"
 #include "chrome/browser/performance_monitor/constants.h"
 
 namespace performance_monitor {
@@ skipping 13 matching lines @@
     kMetricPrivateMemoryUsageDescription,
     kMetricPrivateMemoryUsageUnits,
     kMetricPrivateMemoryUsageTickSize
   },
   {
     kMetricSharedMemoryUsageName,
     kMetricSharedMemoryUsageDescription,
     kMetricSharedMemoryUsageUnits,
     kMetricSharedMemoryUsageTickSize
   },
+  {
+    kMetricStartupTimeName,
+    kMetricStartupTimeDescription,
+    kMetricStartupTimeUnits,
+    kMetricStartupTimeTickSize
+  },

[Nico, 2012/08/10 01:27:58]

This adds a static initializer to chrome. Because our baseline was too high, the
bots didn't notice.

This is because kMetricStartupTimeName and friends are defined in another
translation unit and hence the compiler doesn't know their value when compiling
this file.

+  {
+    kMetricTestStartupTimeName,
+    kMetricTestStartupTimeDescription,
+    kMetricTestStartupTimeUnits,
+    kMetricTestStartupTimeTickSize
+  },
+  {
+    kMetricSessionRestoreTimeName,
+    kMetricSessionRestoreTimeDescription,
+    kMetricSessionRestoreTimeUnits,
+    kMetricSessionRestoreTimeTickSize
+  }
 };
 COMPILE_ASSERT(ARRAYSIZE_UNSAFE(kMetricDetailsList) == METRIC_NUMBER_OF_METRICS,
                metric_names_incorrect_size);
 
 }  // namespace
 
 const MetricDetails* GetMetricDetails(MetricType metric_type) {
   DCHECK_GT(METRIC_NUMBER_OF_METRICS, metric_type);
   return &kMetricDetailsList[metric_type];
 }
 
 }  // namespace performance_monitor