Chrome power profiler service

content/browser/power_profiler/power_profiler_service.h

Index: content/browser/power_profiler/power_profiler_service.h
diff --git a/content/browser/power_profiler/power_profiler_service.h b/content/browser/power_profiler/power_profiler_service.h
new file mode 100644
index 0000000000000000000000000000000000000000..23173c384cfd9f5e9ace1df7ac14f8b58b0eeb4b
--- /dev/null
+++ b/content/browser/power_profiler/power_profiler_service.h
@@ -0,0 +1,76 @@
+// Copyright 2014 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 CONTENT_BROWSER_POWER_PROFILER_POWER_PROFILER_SERVICE_H_
+#define CONTENT_BROWSER_POWER_PROFILER_POWER_PROFILER_SERVICE_H_
+
+#include "base/basictypes.h"
+#include "base/memory/singleton.h"
+#include "base/observer_list.h"
+#include "base/task_runner.h"
+#include "base/timer/timer.h"
+#include "content/browser/power_profiler/power_data_provider.h"
+#include "content/browser/power_profiler/power_profiler_observer.h"
+#include "content/common/content_export.h"
+
+namespace content {
+
+// A class used to query power information and notify the observers.
+class CONTENT_EXPORT PowerProfilerService {
+ public:
+  static PowerProfilerService* GetInstance();
+
+  // Add and remove an observer.
+  void AddObserver(PowerProfilerObserver* observer);
+  void RemoveObserver(PowerProfilerObserver* observer);
+
+  bool IsAvailable();
+
+ private:
+  enum Status {
+    UNINITIALIZED,  // Uninitialized.

[jeremy, 2014/02/17 06:31:13]

comment is redundant.

[Pan, 2014/02/17 07:53:37]

Done.

+    INITIALIZED,  // Initialized, not in profiling.

[jeremy, 2014/02/17 06:31:13]

not in profiling -> profiling has not started.

[Pan, 2014/02/17 07:53:37]

Done.

+    PROFILING,  // In profiling.

[jeremy, 2014/02/17 06:31:13]

I think you can remove this comment.

[Pan, 2014/02/17 07:53:37]

Done.

+  };
+
+  friend struct DefaultSingletonTraits<PowerProfilerService>;
+  friend class PowerProfilerServiceTest;
+
+  PowerProfilerService();
+  virtual ~PowerProfilerService();
+
+  PowerProfilerService(scoped_ptr<PowerDataProvider> provider,
+                       scoped_refptr<base::TaskRunner> task_runner,
+                       const base::TimeDelta& delay);
+
+  void Start();
+  void Stop();
+
+  // Query power data by PowerDataProvider, executes on the WorkerPool thread.

[jeremy, 2014/02/17 06:31:13]

by -> from

[Pan, 2014/02/17 07:53:37]

Done.

+  void QueryDataOnTaskRunner();
+
+  // Initiate the query on the UI thread, the task is delegated to
+  // QueryDataOnTaskRunner.
+  void QueryData();
+
+  // Notify executes on the UI thread.

[jeremy, 2014/02/17 06:31:13]

Notify executes -> Executes

[Pan, 2014/02/17 07:53:37]

Done.

+  void Notify(const PowerEventVector&);
+
+  base::RepeatingTimer<PowerProfilerService> query_power_timer_;
+  scoped_refptr<base::TaskRunner> task_runner_;
+
+  Status status_;
+
+  // Interval of reading the power data by provider.

[jeremy, 2014/02/17 06:31:13]

// Sampling period of power data measurement.

[Pan, 2014/02/17 07:53:37]

Done.

+  const base::TimeDelta sample_period_;
+  ObserverList<PowerProfilerObserver> observers_;
+
+  scoped_ptr<PowerDataProvider> data_provider_;
+
+  DISALLOW_COPY_AND_ASSIGN(PowerProfilerService);
+};
+
+}  // namespace content
+
+#endif  // CONTENT_BROWSER_POWER_PROFILER_POWER_PROFILER_SERVICE_H_