Refactor Metrics

chrome/browser/performance_monitor/database.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_DATABASE_H_
 #define CHROME_BROWSER_PERFORMANCE_MONITOR_DATABASE_H_
 
 #include <vector>
 #include <set>
 #include <string>
 
 #include "base/file_path.h"
 #include "base/gtest_prod_util.h"
 #include "base/memory/linked_ptr.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/time.h"
 #include "chrome/browser/performance_monitor/constants.h"
 #include "chrome/browser/performance_monitor/event.h"
-#include "chrome/browser/performance_monitor/metric_info.h"
-#include "chrome/browser/performance_monitor/metric_details.h"
+#include "chrome/browser/performance_monitor/metric.h"
 #include "third_party/leveldatabase/src/include/leveldb/db.h"
 
 namespace performance_monitor {
 
 struct TimeRange {
   TimeRange();
   TimeRange(base::Time start_time, base::Time end_time);
   ~TimeRange();
 
   base::Time start;
@@ skipping 50 matching lines @@
 // Value: Statistic
 //
 // Metric DB:
 // Stores the statistics for different metrics. Having the time before the
 // activity ensures that the search space can only be as large as the time
 // interval.
 // Key: Metric - Time - Activity
 // Value: Statistic
 class Database {
  public:
-  typedef std::vector<linked_ptr<Event> > EventList;
   typedef std::set<EventType> EventTypeSet;
-  typedef std::vector<MetricInfo> MetricInfoVector;
-  typedef std::map<std::string, linked_ptr<MetricInfoVector> > MetricVectorMap;
+  typedef std::vector<linked_ptr<Event> > EventVector;
+  typedef std::set<MetricType> MetricTypeSet;
+  typedef std::vector<Metric> MetricVector;
+  typedef std::map<std::string, linked_ptr<MetricVector> > MetricVectorMap;
 
   static const char kDatabaseSequenceToken[];
 
   // The class that the database will use to infer time. Abstracting out the
   // time mechanism allows for easy testing and mock data insetion.
   class Clock {
    public:
     Clock() {}
     virtual ~Clock() {}
     virtual base::Time GetTime() = 0;
   };
 
   virtual ~Database();
 
   static scoped_ptr<Database> Create(FilePath path);
 
   // A "state" value is anything that can only have one value at a time, and
   // usually describes the state of the browser eg. version.
   bool AddStateValue(const std::string& key, const std::string& value);
 
   std::string GetStateValue(const std::string& key);
 
   // Add an event to the database.
   bool AddEvent(const Event& event);
 
   // Retrieve the events from the database. These methods populate the provided
   // vector, and will search on the given criteria.
-  EventList GetEvents(EventType type, const base::Time& start,
-                      const base::Time& end);
+  EventVector GetEvents(EventType type,
+                        const base::Time& start,
+                        const base::Time& end);
 
-  EventList GetEvents(const base::Time& start, const base::Time& end) {
+  EventVector GetEvents(const base::Time& start, const base::Time& end) {
     return GetEvents(EVENT_UNDEFINED, start, end);
   }
 
-  EventList GetEvents(EventType type) {
+  EventVector GetEvents(EventType type) {
     return GetEvents(type, base::Time(), clock_->GetTime());
   }
 
-  EventList GetEvents() {
+  EventVector GetEvents() {
     return GetEvents(EVENT_UNDEFINED, base::Time(), clock_->GetTime());
   }
 
   EventTypeSet GetEventTypes(const base::Time& start, const base::Time& end);
 
   EventTypeSet GetEventTypes() {
     return GetEventTypes(base::Time(), clock_->GetTime());
   }
 
   // Add a metric instance to the database.
   bool AddMetric(const std::string& activity,
                  MetricType metric_type,
                  const std::string& value);
 
   bool AddMetric(MetricType metric_type, const std::string& value) {
     return AddMetric(kProcessChromeAggregate, metric_type, value);
   }
 
   // Get the metrics that are active for the given process between |start|
   // (inclusive) and |end| (exclusive).
-  std::vector<const MetricDetails*> GetActiveMetrics(const base::Time& start,
-                                                     const base::Time& end);
+  MetricTypeSet GetActiveMetrics(const base::Time& start,
+                                 const base::Time& end);
 
   // Get the activities that are active for the given metric after |start|.
-  std::vector<std::string> GetActiveActivities(MetricType metric_type,
-                                               const base::Time& start);
+  std::set<std::string> GetActiveActivities(MetricType metric_type,
+                                            const base::Time& start);
 
   // Populate info with the most recent activity. Return false if populate
   // was unsuccessful.
   bool GetRecentStatsForActivityAndMetric(const std::string& activity,
-                                          MetricType metric,
-                                          MetricInfo* info);
+                                          MetricType metric_type,
+                                          Metric* metric);
 
-  bool GetRecentStatsForActivityAndMetric(MetricType metric, MetricInfo* info) {
+  bool GetRecentStatsForActivityAndMetric(MetricType metric_type,
+                                           Metric* metric) {
     return GetRecentStatsForActivityAndMetric(kProcessChromeAggregate,
-                                              metric,
-                                              info);
+                                              metric_type,
+                                              metric);
   }
 
   // Query given |metric_type| and |activity|.
-  MetricInfoVector GetStatsForActivityAndMetric(const std::string& activity,
-                                                MetricType metric_type,
-                                                const base::Time& start,
-                                                const base::Time& end);
+  MetricVector GetStatsForActivityAndMetric(const std::string& activity,
+                                            MetricType metric_type,
+                                            const base::Time& start,
+                                            const base::Time& end);
 
-  MetricInfoVector GetStatsForActivityAndMetric(MetricType metric_type,
-                                                const base::Time& start,
-                                                const base::Time& end) {
+  MetricVector GetStatsForActivityAndMetric(MetricType metric_type,
+                                            const base::Time& start,
+                                            const base::Time& end) {
     return GetStatsForActivityAndMetric(kProcessChromeAggregate, metric_type,
                                         start, end);
   }
 
-  MetricInfoVector GetStatsForActivityAndMetric(const std::string& activity,
-                                                MetricType metric_type) {
+  MetricVector GetStatsForActivityAndMetric(const std::string& activity,
+                                            MetricType metric_type) {
     return GetStatsForActivityAndMetric(activity, metric_type, base::Time(),
                                         clock_->GetTime());
   }
 
-  MetricInfoVector GetStatsForActivityAndMetric(MetricType metric_type) {
+  MetricVector GetStatsForActivityAndMetric(MetricType metric_type) {
     return GetStatsForActivityAndMetric(kProcessChromeAggregate, metric_type,
                                         base::Time(), clock_->GetTime());
   }
 
   // Query given |metric_type|. The returned map is keyed by activity.
   MetricVectorMap GetStatsForMetricByActivity(MetricType metric_type,
                                               const base::Time& start,
                                               const base::Time& end);
 
   MetricVectorMap GetStatsForMetricByActivity(MetricType metric_type) {
@@ skipping 66 matching lines @@
 
   leveldb::ReadOptions read_options_;
   leveldb::WriteOptions write_options_;
 
   DISALLOW_COPY_AND_ASSIGN(Database);
 };
 
 }  // namespace performance_monitor
 
 #endif  // CHROME_BROWSER_PERFORMANCE_MONITOR_DATABASE_H_