Reland: Plugin Power Saver: Improve Poster behavior for essential plugins.

content/renderer/pepper/plugin_power_saver_helper.cc

 // Copyright 2014 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/plugin_power_saver_helper.h"
 
 #include <string>
 
 #include "base/command_line.h"
 #include "base/metrics/histogram.h"
 #include "base/strings/string_number_conversions.h"
 #include "content/common/frame_messages.h"
 #include "content/public/common/content_switches.h"
 #include "content/public/renderer/render_frame.h"
 #include "content/renderer/peripheral_content_heuristic.h"
 #include "ppapi/shared_impl/ppapi_constants.h"
 #include "third_party/WebKit/public/web/WebLocalFrame.h"
+#include "ui/gfx/geometry/size.h"
 
 namespace content {
 
 namespace {
 
 const char kPeripheralHeuristicHistogram[] =
     "Plugin.PowerSaver.PeripheralHeuristic";
 
 }  // namespace
 
@@ skipping 49 matching lines @@
   }
 }
 
 void PluginPowerSaverHelper::RegisterPeripheralPlugin(
     const url::Origin& content_origin,
     const base::Closure& unthrottle_callback) {
   peripheral_plugins_.push_back(
       PeripheralPlugin(content_origin, unthrottle_callback));
 }
 
-bool PluginPowerSaverHelper::ShouldThrottleContent(
+RenderFrame::PeripheralContentStatus
+PluginPowerSaverHelper::GetPeripheralContentStatus(
     const url::Origin& main_frame_origin,
     const url::Origin& content_origin,
-    int width,
-    int height,
-    bool* cross_origin_main_content) const {
-  if (cross_origin_main_content)
-    *cross_origin_main_content = false;
-
+    const gfx::Size& unobscured_size) const {
   if (base::CommandLine::ForCurrentProcess()->GetSwitchValueASCII(
           switches::kOverridePluginPowerSaverForTesting) == "always") {
-    return true;
+    return RenderFrame::CONTENT_STATUS_PERIPHERAL;
   }
 
-  auto decision = PeripheralContentHeuristic::GetPeripheralStatus(
-      origin_whitelist_, main_frame_origin, content_origin, width, height);
+  auto status = PeripheralContentHeuristic::GetPeripheralStatus(
+      origin_whitelist_, main_frame_origin, content_origin, unobscured_size);
+  if (status == RenderFrame::CONTENT_STATUS_ESSENTIAL_UNKNOWN_SIZE) {
+    // Early exit here to avoid recording a UMA. Every plugin will call this
+    // method once before the size is known (to faciliate early-exit for
+    // same-origin and whitelisted-origin content).
+    return status;
+  }
 
-  UMA_HISTOGRAM_ENUMERATION(
-      kPeripheralHeuristicHistogram, decision,
-      PeripheralContentHeuristic::HEURISTIC_DECISION_NUM_ITEMS);
-
-  if (decision == PeripheralContentHeuristic::
-                      HEURISTIC_DECISION_ESSENTIAL_CROSS_ORIGIN_BIG &&
-      cross_origin_main_content)
-    *cross_origin_main_content = true;
-
-  return decision == PeripheralContentHeuristic::HEURISTIC_DECISION_PERIPHERAL;
+  UMA_HISTOGRAM_ENUMERATION(kPeripheralHeuristicHistogram, status,
+                            RenderFrame::CONTENT_STATUS_NUM_ITEMS);
+  return status;
 }
 
 void PluginPowerSaverHelper::WhitelistContentOrigin(
     const url::Origin& content_origin) {
   if (origin_whitelist_.insert(content_origin).second) {
     Send(new FrameHostMsg_PluginContentOriginAllowed(
         render_frame()->GetRoutingID(), content_origin));
   }
 }
 
 }  // namespace content