chrome power profiler chrome side changes

content/browser/power_profiler/power_event.h

Index: content/browser/power_profiler/power_event.h
diff --git a/content/browser/power_profiler/power_event.h b/content/browser/power_profiler/power_event.h
new file mode 100644
index 0000000000000000000000000000000000000000..6df1ce13a3535f052675290c733144f52642d452
--- /dev/null
+++ b/content/browser/power_profiler/power_event.h
@@ -0,0 +1,29 @@
+// Copyright 2013 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_EVENT_H_
+#define CONTENT_BROWSER_POWER_EVENT_H_
+
+#include "base/time/time.h"
+
+namespace content {
+
+// A class used to monitor the power usage and tell profiler the power data
+struct PowerEvent {

[pfeldman, 2014/01/09 11:38:58]

You can define it within the helper.

+  enum Type {

[qsr, 2014/01/17 11:50:14]

Also, pease comment on this variable, at least for the first one. Moreover, am I
missing something but I do not see you ever getting any value for the GPU
counter (in that case, you should remove it from there until you can compute
something). As for the CPU, I'm under the impression you compute a value during
the initialization phase of your Helper implementation, but never after that,
which makes it again not useful at this point.

+    SOC_PACKAGE,
+    CPU,
+    GPU,
+    // counts the number of known PowerEvent
+    ID_COUNT
+  };
+
+  Type type;
+  base::TimeTicks time;
+  double value; // power
+};
+
+}  // namespace content
+
+#endif  // CONTENT_BROWSER_POWER_EVENT_H_