Chrome Performance Monitor: MetricDetails refactor.

chrome/browser/performance_monitor/metric_details.h

Index: chrome/browser/performance_monitor/metric_details.h
diff --git a/chrome/browser/performance_monitor/metric_details.h b/chrome/browser/performance_monitor/metric_details.h
index 6efae2fa14a61947a66daa2cc9002e6763de2cdc..e97e58f63860cd68308d2e23302f3dda0c3e0b43 100644
--- a/chrome/browser/performance_monitor/metric_details.h
+++ b/chrome/browser/performance_monitor/metric_details.h
@@ -9,23 +9,21 @@
 
 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 {
-  MetricDetails();
-  MetricDetails(const std::string& metric_name,
-                const std::string& metric_description);
-  ~MetricDetails();
-
-  std::string name;
-  std::string description;
+  const char* const name;
+  const char* const description;
+  const char* const units;
+  const double max_value;
 };
 
+const MetricDetails* GetMetricDetails(MetricType event_type);
+
 }  // namespace performance_monitor
 
 #endif  // CHROME_BROWSER_PERFORMANCE_MONITOR_METRIC_DETAILS_H_