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

content/renderer/pepper/pepper_plugin_instance_impl.cc

Index: content/renderer/pepper/pepper_plugin_instance_impl.cc
diff --git a/content/renderer/pepper/pepper_plugin_instance_impl.cc b/content/renderer/pepper/pepper_plugin_instance_impl.cc
index e22101ba7c5f8660dd9d5d8a401b6c1139100266..917756dd6553951236ed37ad2e9dd20e49392d28 100644
--- a/content/renderer/pepper/pepper_plugin_instance_impl.cc
+++ b/content/renderer/pepper/pepper_plugin_instance_impl.cc
@@ -21,6 +21,7 @@
 #include "cc/layers/texture_layer.h"
 #include "cc/trees/layer_tree_host.h"
 #include "content/common/content_constants_internal.h"
+#include "content/common/frame_messages.h"
 #include "content/common/input/web_input_event_traits.h"
 #include "content/public/common/content_constants.h"
 #include "content/public/common/content_switches.h"
@@ -45,6 +46,7 @@
 #include "content/renderer/pepper/pepper_url_loader_host.h"
 #include "content/renderer/pepper/plugin_module.h"
 #include "content/renderer/pepper/plugin_object.h"
+#include "content/renderer/pepper/plugin_power_saver_helper.h"
 #include "content/renderer/pepper/ppapi_preferences_builder.h"
 #include "content/renderer/pepper/ppb_buffer_impl.h"
 #include "content/renderer/pepper/ppb_graphics_3d_impl.h"
@@ -583,9 +585,17 @@ PepperPluginInstanceImpl::PepperPluginInstanceImpl(
       GetContentClient()->renderer()->IsExternalPepperPlugin(module->name()))
     external_document_load_ = true;
 
-  power_saver_enabled_ = CommandLine::ForCurrentProcess()->HasSwitch(
-                             switches::kEnablePluginPowerSaver) &&
-                         IsPeripheralContent();
+  blink::WebRect bounds = container_->element().boundsInViewportSpace();
+  power_saver_enabled_ =
+      CommandLine::ForCurrentProcess()->HasSwitch(
+          switches::kEnablePluginPowerSaver) &&
+      module_->name() == kFlashPluginName &&
+      render_frame_->GetPluginPowerSaverHelper()->ShouldThrottleContent(
+          plugin_url_,
+          bounds.width,
+          bounds.height,
+          base::Bind(&PepperPluginInstanceImpl::DisablePowerSaverAndUnthrottle,
+                     weak_factory_.GetWeakPtr()));
 
   if (power_saver_enabled_) {
     throttler_.reset(new PepperPluginInstanceThrottler(
@@ -3295,27 +3305,6 @@ void PepperPluginInstanceImpl::DidDataFromWebURLResponse(
   }
 }
 
-bool PepperPluginInstanceImpl::IsPeripheralContent() const {
-  if (module_->name() != kFlashPluginName)
-    return false;
-
-  // Peripheral plugin content is defined to be peripheral when the plugin
-  // content's origin differs from the top level frame's origin. For example:
-  //  - Peripheral:      a.com -> b.com/plugin.swf
-  //  - Peripheral:      a.com -> b.com/iframe.html -> b.com/plugin.swf
-  //  - NOT peripheral:  a.com -> b.com/iframe-to-a.html -> a.com/plugin.swf
-
-  // TODO(alexmos): Update this to use the origin of the RemoteFrame when 426512
-  // is fixed. For now, case 3 in the comment above doesn't work in
-  // --site-per-process mode.
-  WebFrame* main_frame = render_frame_->GetWebFrame()->view()->mainFrame();
-  if (main_frame->isWebRemoteFrame())
-     return true;
-
-  GURL main_frame_url = main_frame->document().url();
-  return plugin_url_.GetOrigin() != main_frame_url.GetOrigin();
-}
-
 void PepperPluginInstanceImpl::SetPluginThrottled(bool throttled) {
   // Do not throttle if we've already disabled power saver.
   if (!power_saver_enabled_ && throttled)
@@ -3325,4 +3314,10 @@ void PepperPluginInstanceImpl::SetPluginThrottled(bool throttled) {
   SendDidChangeView();
 }
 
+void PepperPluginInstanceImpl::DisablePowerSaverAndUnthrottle() {
+  DCHECK(power_saver_enabled_);
+  power_saver_enabled_ = false;
+  SetPluginThrottled(false);
+}
+
 }  // namespace content