Chrome Performance Monitor: MetricDetails refactor.

chrome/browser/performance_monitor/metric_details.h

 // 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_METRIC_DETAILS_H_
 #define CHROME_BROWSER_PERFORMANCE_MONITOR_METRIC_DETAILS_H_
 
 #include <string>
 
 namespace performance_monitor {
 
+// Keep this enum synced with kMetricDetailsList in the cc file.
 enum MetricType {
   METRIC_SAMPLE,

[eaugusti, 2012/07/21 02:00:40]

Are there any more metrics that we can add now?
CPU? MEM? JANK?

[Matt Tytel, 2012/07/21 02:11:40]

In Mitchell's CL:
http://codereview.chromium.org/10656052/

[Matt Tytel, 2012/07/23 22:24:00]

Done.

   METRIC_NUMBER_OF_METRICS
 };
 
-const char* MetricTypeToString(MetricType event_type);
+struct MetricDetails {
+  const char* const name;
+  const char* const description;
+  const char* const units;
+  const double max_value;
+};
 
-struct MetricDetails {
-  MetricDetails();
-  MetricDetails(const std::string& metric_name,
-                const std::string& metric_description);
-  ~MetricDetails();
-
-  std::string name;
-  std::string description;
-};
+const MetricDetails* GetMetricDetails(MetricType event_type);
 
 }  // namespace performance_monitor
 
 #endif  // CHROME_BROWSER_PERFORMANCE_MONITOR_METRIC_DETAILS_H_