Plugin Power Saver: Throttle 'large' plugins that appear small.

content/renderer/pepper/pepper_plugin_instance_impl.cc

 // Copyright (c) 2012 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.
 
 #include "content/renderer/pepper/pepper_plugin_instance_impl.h"
 
 #include "base/bind.h"
 #include "base/callback_helpers.h"
 #include "base/logging.h"
 #include "base/memory/linked_ptr.h"
@@ skipping 791 matching lines @@
 bool PepperPluginInstanceImpl::Initialize(
     const std::vector<std::string>& arg_names,
     const std::vector<std::string>& arg_values,
     bool full_frame,
     scoped_ptr<PluginInstanceThrottlerImpl> throttler) {
   DCHECK(!throttler_);
 
   if (!render_frame_)
     return false;
 
-  if (is_flash_plugin_ && RenderThread::Get()) {
-    RenderThread::Get()->RecordAction(
-        base::UserMetricsAction("Flash.PluginInstanceCreated"));
-    blink::WebRect bounds = container()->element().boundsInViewportSpace();
-    RecordFlashSizeMetric(bounds.width, bounds.height);
-  }
-
   if (throttler) {
     throttler_ = throttler.Pass();
     throttler_->AddObserver(this);
-    throttler_->Initialize(render_frame_, plugin_url_.GetOrigin(),
-                           module()->name(),
-                           container()->element().boundsInViewportSpace());
   }
 
   message_channel_ = MessageChannel::Create(this, &message_channel_object_);
 
   full_frame_ = full_frame;
 
   UpdateTouchEventRequest();
   container_->setWantsWheelEvents(IsAcceptingWheelEvents());
 
   SetGPUHistogram(ppapi::Preferences(PpapiPreferencesBuilder::Build(
@@ skipping 391 matching lines @@
   RecordFlashJavaScriptUse();
 
   // If the plugin supports the private instance interface, try to retrieve its
   // instance object.
   if (LoadPrivateInterface())
     return plugin_private_interface_->GetInstanceObject(pp_instance());
   return PP_MakeUndefined();
 }
 
 void PepperPluginInstanceImpl::ViewChanged(
-    const gfx::Rect& position,
+    const gfx::Rect& window,
     const gfx::Rect& clip,
+    const gfx::Rect& unobscured,
     const std::vector<gfx::Rect>& cut_outs_rects) {
   // WebKit can give weird (x,y) positions for empty clip rects (since the
   // position technically doesn't matter). But we want to make these
   // consistent since this is given to the plugin, so force everything to 0
   // in the "everything is clipped" case.
   gfx::Rect new_clip;
   if (!clip.IsEmpty())
     new_clip = clip;
 
+  unobscured_ = unobscured;
+
   cut_outs_rects_ = cut_outs_rects;
 
-  view_data_.rect = PP_FromGfxRect(position);
+  view_data_.rect = PP_FromGfxRect(window);
   view_data_.clip_rect = PP_FromGfxRect(clip);
   view_data_.device_scale = container_->deviceScaleFactor();
   view_data_.css_scale =
       container_->pageZoomFactor() * container_->pageScaleFactor();
 
   gfx::Size scroll_offset =
       container_->element().document().frame()->scrollOffset();
   view_data_.scroll_offset = PP_MakePoint(scroll_offset.width(),
                                           scroll_offset.height());
 
@@ skipping 21 matching lines @@
       // Reset the size attributes that we hacked to fill in the screen and
       // retrigger ViewChanged. Make sure we don't forward duplicates of
       // this view to the plugin.
       ResetSizeAttributesAfterFullscreen();
       return;
     }
   }
 
   UpdateFlashFullscreenState(fullscreen_container_ != NULL);
 
-  SendDidChangeView();
+  // During plugin initialization, there are often re-layouts. Avoid sending
+  // intermediate sizes the plugin and throttler.
+  if (sent_initial_did_change_view_)
+    SendDidChangeView();
+  else
+    ScheduleAsyncDidChangeView();
 }
 
 void PepperPluginInstanceImpl::SetWebKitFocus(bool has_focus) {
   if (has_webkit_focus_ == has_focus)
     return;
 
   bool old_plugin_focus = PluginHasFocus();
   has_webkit_focus_ = has_focus;
   if (PluginHasFocus() != old_plugin_focus)
     SendFocusChangeNotification();
@@ skipping 321 matching lines @@
   SendDidChangeView();
 }
 
 void PepperPluginInstanceImpl::SendDidChangeView() {
   // Don't send DidChangeView to crashed plugins.
   if (module()->is_crashed())
     return;
 
   ppapi::ViewData view_data = view_data_;
 
+  if (!sent_initial_did_change_view_) {

[piman, 2015/04/01 01:02:41]

So, you may be getting into this code more than once. SendDidChangeView is
called from various places, but we will just early out if there is a pending
posted task, on l.1655, in which case sent_initial_did_change_view_ will not be
set to true.

The following order is possible:

- Initialization (sent_initial_did_change_view_ is false)
- ScheduleAsyncDidChangeView (sent_initial_did_change_view_ is still false)
- something calls SendDidChangeView directly (e.g. ResetAsProxied or
PageVisibilityChanged), and we get here (sent_initial_did_change_view_ is still
false) but we early exit on l.1655 because a task is pending, so
sent_initial_did_change_view_ is not set to true
- the task executes, and we get here again, sent_initial_did_change_view_ is
still false


Maybe both this code and the code below (the if (throttler_ &&
throttler_->IsThrottled()) block) should move after l.1655, right before
sent_initial_did_change_view_ is set to true (that way you're guaranteed it's
done exactly once).

[tommycli, 2015/04/01 18:55:28]

Done. Thanks for the explanation. That's a pretty tricky setup.

I was unable to move that block under 1655 as that conditional uses the
throttler-modified view data. However, I was able to move the async view change
early exit to the very top.

+    if (is_flash_plugin_ && RenderThread::Get()) {
+      RenderThread::Get()->RecordAction(
+          base::UserMetricsAction("Flash.PluginInstanceCreated"));
+      RecordFlashSizeMetric(unobscured_.width(), unobscured_.height());
+    }
+
+    // Initialize the throttler if necessary.
+    if (throttler_) {
+      throttler_->Initialize(render_frame_, plugin_url_.GetOrigin(),
+                             module()->name(), unobscured_.size());
+    }
+  }
+
   // When plugin content is throttled, fake the page being offscreen. We cannot
   // send empty view data here, as some plugins rely on accurate view data.
   if (throttler_ && throttler_->IsThrottled()) {
     view_data.is_page_visible = false;
     view_data.clip_rect.point.x = 0;
     view_data.clip_rect.point.y = 0;
     view_data.clip_rect.size.width = 0;
     view_data.clip_rect.size.height = 0;
   }
 
@@ skipping 1669 matching lines @@
 
 void PepperPluginInstanceImpl::RecordFlashJavaScriptUse() {
   if (initialized_ && !javascript_used_ && is_flash_plugin_) {
     javascript_used_ = true;
     RenderThread::Get()->RecordAction(
         base::UserMetricsAction("Flash.JavaScriptUsed"));
   }
 }
 
 }  // namespace content