Plugin Power Saver: Improve Poster behavior for essential plugins.

content/renderer/pepper/plugin_power_saver_helper_browsertest.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 "base/run_loop.h"
 #include "content/common/frame_messages.h"
 #include "content/common/view_message_enums.h"
 #include "content/public/common/content_constants.h"
 #include "content/public/test/frame_load_waiter.h"
 #include "content/public/test/render_view_test.h"
 #include "content/renderer/pepper/plugin_power_saver_helper.h"
 #include "content/renderer/render_frame_impl.h"
 #include "content/renderer/render_view_impl.h"
 #include "content/test/test_render_frame.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "third_party/WebKit/public/web/WebDocument.h"
 #include "third_party/WebKit/public/web/WebLocalFrame.h"
 #include "third_party/WebKit/public/web/WebPluginParams.h"
+#include "ui/gfx/geometry/size.h"
 #include "url/gurl.h"
 
 namespace content {
 
 class PluginPowerSaverHelperTest : public RenderViewTest {
  public:
   PluginPowerSaverHelperTest() : sink_(NULL) {}
 
   void SetUp() override {
     RenderViewTest::SetUp();
     sink_ = &render_thread_->sink();
   }
 
   RenderFrameImpl* frame() {
     return static_cast<RenderFrameImpl*>(view_->GetMainRenderFrame());
   }
 
-  blink::WebPluginParams MakeParams(const std::string& url,
-                                    const std::string& poster,
-                                    const std::string& width,
-                                    const std::string& height) {
-    const size_t size = 3;
-    blink::WebVector<blink::WebString> names(size);
-    blink::WebVector<blink::WebString> values(size);
-
-    blink::WebPluginParams params;
-    params.url = GURL(url);
-
-    params.attributeNames.swap(names);
-    params.attributeValues.swap(values);
-
-    params.attributeNames[0] = "poster";
-    params.attributeNames[1] = "height";
-    params.attributeNames[2] = "width";
-    params.attributeValues[0] = blink::WebString::fromUTF8(poster);
-    params.attributeValues[1] = blink::WebString::fromUTF8(height);
-    params.attributeValues[2] = blink::WebString::fromUTF8(width);
-
-    return params;
-  }
-
  protected:
   IPC::TestSink* sink_;
 
   DISALLOW_COPY_AND_ASSIGN(PluginPowerSaverHelperTest);
 };
 
-TEST_F(PluginPowerSaverHelperTest, AllowSameOrigin) {
-  EXPECT_FALSE(frame()->ShouldThrottleContent(
-      url::Origin(GURL("http://same.com")),
-      url::Origin(GURL("http://same.com")), 100, 100, nullptr));
-  EXPECT_FALSE(frame()->ShouldThrottleContent(
-      url::Origin(GURL("http://same.com")),
-      url::Origin(GURL("http://same.com")), 1000, 1000, nullptr));
-}
-
-TEST_F(PluginPowerSaverHelperTest, DisallowCrossOriginUnlessLarge) {
-  bool cross_origin_main_content = false;
-  EXPECT_TRUE(
-      frame()->ShouldThrottleContent(url::Origin(GURL("http://same.com")),
-                                     url::Origin(GURL("http://other.com")), 100,
-                                     100, &cross_origin_main_content));
-  EXPECT_FALSE(cross_origin_main_content);
-
-  EXPECT_FALSE(
-      frame()->ShouldThrottleContent(url::Origin(GURL("http://same.com")),
-                                     url::Origin(GURL("http://other.com")),
-                                     1000, 1000, &cross_origin_main_content));
-  EXPECT_TRUE(cross_origin_main_content);
-}
-
-TEST_F(PluginPowerSaverHelperTest, AlwaysAllowTinyContent) {
-  bool cross_origin_main_content = false;
-  EXPECT_FALSE(frame()->ShouldThrottleContent(
-      url::Origin(GURL("http://same.com")),
-      url::Origin(GURL("http://same.com")), 1, 1, nullptr));
-  EXPECT_FALSE(cross_origin_main_content);
-
-  EXPECT_FALSE(frame()->ShouldThrottleContent(
-      url::Origin(GURL("http://same.com")),
-      url::Origin(GURL("http://other.com")), 1, 1, &cross_origin_main_content));
-  EXPECT_FALSE(cross_origin_main_content);
-
-  EXPECT_FALSE(frame()->ShouldThrottleContent(
-      url::Origin(GURL("http://same.com")),
-      url::Origin(GURL("http://other.com")), 5, 5, &cross_origin_main_content));
-  EXPECT_FALSE(cross_origin_main_content);
-
-  EXPECT_TRUE(
-      frame()->ShouldThrottleContent(url::Origin(GURL("http://same.com")),
-                                     url::Origin(GURL("http://other.com")), 10,
-                                     10, &cross_origin_main_content));
-  EXPECT_FALSE(cross_origin_main_content);
-}
-
 TEST_F(PluginPowerSaverHelperTest, TemporaryOriginWhitelist) {
-  bool cross_origin_main_content = false;
-  EXPECT_TRUE(
-      frame()->ShouldThrottleContent(url::Origin(GURL("http://same.com")),
-                                     url::Origin(GURL("http://other.com")), 100,
-                                     100, &cross_origin_main_content));
-  EXPECT_FALSE(cross_origin_main_content);
+  EXPECT_EQ(RenderFrame::CONTENT_STATUS_PERIPHERAL,

[groby-ooo-7-16, 2015/12/09 02:17:15]

It bugs me these have to be browser tests. Any chance we can move to unit tests?

[tommycli, 2015/12/09 23:25:45]

For this particular test, I think it needs to be a RenderViewTest (almost a
browser test) to verify that: 1. IPCs are sent, and 2. Navigations clear the
whitelist.

The heuristic tests that don't require a RenderView have already been moved to
this unit test:
https://code.google.com/p/chromium/codesearch#chromium/src/content/renderer/p...

+            frame()->GetPeripheralContentStatus(
+                url::Origin(GURL("http://same.com")),
+                url::Origin(GURL("http://other.com")), gfx::Size(100, 100)));
 
   // Clear out other messages so we find just the plugin power saver IPCs.
   sink_->ClearMessages();
 
   frame()->WhitelistContentOrigin(url::Origin(GURL("http://other.com")));
-  EXPECT_FALSE(
-      frame()->ShouldThrottleContent(url::Origin(GURL("http://same.com")),
-                                     url::Origin(GURL("http://other.com")), 100,
-                                     100, &cross_origin_main_content));
-  EXPECT_FALSE(cross_origin_main_content);
+
+  EXPECT_EQ(RenderFrame::CONTENT_STATUS_ESSENTIAL_CROSS_ORIGIN_WHITELISTED,
+            frame()->GetPeripheralContentStatus(
+                url::Origin(GURL("http://same.com")),
+                url::Origin(GURL("http://other.com")), gfx::Size(100, 100)));
 
   // Test that we've sent an IPC to the browser.
   ASSERT_EQ(1u, sink_->message_count());
   const IPC::Message* msg = sink_->GetMessageAt(0);
   EXPECT_EQ(FrameHostMsg_PluginContentOriginAllowed::ID, msg->type());
   FrameHostMsg_PluginContentOriginAllowed::Param params;
   FrameHostMsg_PluginContentOriginAllowed::Read(msg, &params);
   EXPECT_TRUE(url::Origin(GURL("http://other.com"))
                   .IsSameOriginWith(base::get<0>(params)));
 }
 
 TEST_F(PluginPowerSaverHelperTest, UnthrottleOnExPostFactoWhitelist) {
   base::RunLoop loop;
   frame()->RegisterPeripheralPlugin(url::Origin(GURL("http://other.com")),
                                     loop.QuitClosure());
 
   std::set<url::Origin> origin_whitelist;
   origin_whitelist.insert(url::Origin(GURL("http://other.com")));
   frame()->OnMessageReceived(FrameMsg_UpdatePluginContentOriginWhitelist(
       frame()->GetRoutingID(), origin_whitelist));
 
   // Runs until the unthrottle closure is run.
   loop.Run();
 }
 
 TEST_F(PluginPowerSaverHelperTest, ClearWhitelistOnNavigate) {
   frame()->WhitelistContentOrigin(url::Origin(GURL("http://other.com")));
 
-  EXPECT_FALSE(frame()->ShouldThrottleContent(
-      url::Origin(GURL("http://same.com")),
-      url::Origin(GURL("http://other.com")), 100, 100, nullptr));
+  EXPECT_EQ(RenderFrame::CONTENT_STATUS_ESSENTIAL_CROSS_ORIGIN_WHITELISTED,
+            frame()->GetPeripheralContentStatus(
+                url::Origin(GURL("http://same.com")),
+                url::Origin(GURL("http://other.com")), gfx::Size(100, 100)));
 
   LoadHTML("<html></html>");
 
-  EXPECT_TRUE(frame()->ShouldThrottleContent(
-      url::Origin(GURL("http://same.com")),
-      url::Origin(GURL("http://other.com")), 100, 100, nullptr));
+  EXPECT_EQ(RenderFrame::CONTENT_STATUS_PERIPHERAL,
+            frame()->GetPeripheralContentStatus(
+                url::Origin(GURL("http://same.com")),
+                url::Origin(GURL("http://other.com")), gfx::Size(100, 100)));
 }
 
 }  // namespace content