Created multi-process-friendly PowerMonitor interface.

content/common/gpu/client/gpu_channel_host.h

Index: content/common/gpu/client/gpu_channel_host.h
diff --git a/content/common/gpu/client/gpu_channel_host.h b/content/common/gpu/client/gpu_channel_host.h
index c06989d040c06ee9b64414e05fcce174445dd0d0..f6642cba4295aaafa5cb8a48f42c0d5148a43ea6 100644
--- a/content/common/gpu/client/gpu_channel_host.h
+++ b/content/common/gpu/client/gpu_channel_host.h
@@ -49,6 +49,7 @@ namespace content {
 class CommandBufferProxyImpl;
 class GpuChannelHost;
 struct GpuRenderingStats;
+class PowerMonitorMessageBroadcaster;
 
 struct GpuListenerInfo {
   GpuListenerInfo();
@@ -248,6 +249,8 @@ class GpuChannelHost : public IPC::Sender,
   typedef base::hash_map<int, CommandBufferProxyImpl*> ProxyMap;
   ProxyMap proxies_;
 
+  scoped_ptr<PowerMonitorMessageBroadcaster> power_monitor_broadcaster_;

[vandebo (ex-Chrome), 2013/06/26 21:40:33]

direct member?

[bajones, 2013/06/26 22:37:22]

This class needs to be provided with the Sender*. I could do that as a separate
call, but I felt like during the constructor was a bit more natural. 

[vandebo (ex-Chrome), 2013/06/26 22:50:12]

I may be misunderstanding your reply... you can still initialize this in the
constructor (and in fact you have to), by doing:
  power_monitor_broadcaster_(this)
instead of:
  power_monitor_broadcaster_(new PowerMonitorMessageBroadcaster(this))

[bajones, 2013/06/26 22:57:50]

*facepalm*

This is what too much Javascript does to your C++ skills... Thanks for reminding
me how the language works. :)

+
   DISALLOW_COPY_AND_ASSIGN(GpuChannelHost);
 };