Plugin Power Saver: Implement size-based heuristic for peripheral content.

content/renderer/render_frame_impl.cc

Index: content/renderer/render_frame_impl.cc
diff --git a/content/renderer/render_frame_impl.cc b/content/renderer/render_frame_impl.cc
index 63ab694e12698d2fc7483b90c10e6358d8559c8a..dfacbf26f58eaa7e6a4de57e9d949dbb4e755029 100644
--- a/content/renderer/render_frame_impl.cc
+++ b/content/renderer/render_frame_impl.cc
@@ -81,6 +81,7 @@
 #include "content/renderer/notification_permission_dispatcher.h"
 #include "content/renderer/notification_provider.h"
 #include "content/renderer/npapi/plugin_channel_host.h"
+#include "content/renderer/pepper/plugin_power_saver_helper.h"

[Lei Zhang, 2014/10/29 09:26:47]

move to line 137

[tommycli, 2014/10/29 19:59:23]

Done.

 #include "content/renderer/push_messaging_dispatcher.h"
 #include "content/renderer/render_frame_proxy.h"
 #include "content/renderer/render_process.h"
@@ -546,6 +547,7 @@ RenderFrameImpl::RenderFrameImpl(RenderViewImpl* render_view, int routing_id)
       is_swapped_out_(false),
       render_frame_proxy_(NULL),
       is_detaching_(false),
+      plugin_power_saver_helper_(NULL),

[Lei Zhang, 2014/10/29 09:26:47]

ifdef

[tommycli, 2014/10/29 19:59:23]

Done.

       cookie_jar_(this),
       selection_text_offset_(0),
       selection_range_(gfx::Range::InvalidRange()),
@@ -812,6 +814,12 @@ void RenderFrameImpl::OnImeConfirmComposition(
   pepper_composition_text_.clear();
 }
 
+PluginPowerSaverHelper* RenderFrameImpl::GetPluginPowerSaverHelper() {
+  if (!plugin_power_saver_helper_)
+    plugin_power_saver_helper_ = new PluginPowerSaverHelper(this);

[Lei Zhang, 2014/10/29 09:26:47]

Who owns this / frees it?

[tommycli, 2014/10/29 19:59:23]

As a RenderFrameObserver, it's deleted when the RenderFrame gets deleted. Should
I add a comment to this effect?

[Lei Zhang, 2014/10/29 22:31:22]

Yes please.

[tommycli, 2014/10/29 22:53:22]

Done.

+  return plugin_power_saver_helper_;
+}
+
 #endif  // ENABLE_PLUGINS
 
 MediaStreamDispatcher* RenderFrameImpl::GetMediaStreamDispatcher() {