Force new surface on BrowserPluginGuest reattach.

content/browser/frame_host/render_widget_host_view_guest_unittest.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/browser/frame_host/render_widget_host_view_guest.h"
 
 #include "base/basictypes.h"
 #include "base/message_loop/message_loop.h"
 #include "base/run_loop.h"
 #include "cc/surfaces/surface.h"
@@ skipping 83 matching lines @@
 class TestBrowserPluginGuest : public BrowserPluginGuest {
  public:
   TestBrowserPluginGuest(WebContentsImpl* web_contents,
                          BrowserPluginGuestDelegate* delegate):
       BrowserPluginGuest(web_contents->HasOpener(), web_contents, delegate),
       last_scale_factor_received_(0.f),
       update_scale_factor_received_(0.f),
       received_delegated_frame_(false) {}
   ~TestBrowserPluginGuest() override {}
 
+  void ResetTestData() {
+    update_frame_size_received_= gfx::Size();
+    update_scale_factor_received_ = 0.f;
+    last_surface_id_received_ = cc::SurfaceId();
+    last_frame_size_received_ = gfx::Size();
+    last_scale_factor_received_ = 0.f;
+    received_delegated_frame_ = false;
+  }
+
+  void set_has_attached_since_surface_set(bool has_attached_since_surface_set) {
+    BrowserPluginGuest::set_has_attached_since_surface_set_for_test(
+        has_attached_since_surface_set);
+  }
+
+  void set_attached(bool attached) {
+    BrowserPluginGuest::set_attached_for_test(attached);
+  }
+
   void UpdateGuestSizeIfNecessary(const gfx::Size& frame_size,
                                   float scale_factor) override {
     update_frame_size_received_= frame_size;
     update_scale_factor_received_ = scale_factor;
   }
 
   void SwapCompositorFrame(uint32_t output_surface_id,
                            int host_process_id,
                            int host_routing_id,
                            scoped_ptr<cc::CompositorFrame> frame) override {
@@ skipping 113 matching lines @@
 TEST_F(RenderWidgetHostViewGuestSurfaceTest, TestGuestSurface) {
   gfx::Size view_size(100, 100);
   gfx::Rect view_rect(view_size);
   float scale_factor = 1.f;
 
   ASSERT_TRUE(browser_plugin_guest_);
 
   view_->SetSize(view_size);
   view_->Show();
 
+  browser_plugin_guest_->set_attached(true);
   view_->OnSwapCompositorFrame(
       0, CreateDelegatedFrame(scale_factor, view_size, view_rect));
 
   EXPECT_EQ(view_size, browser_plugin_guest_->update_frame_size_received_);
   EXPECT_EQ(scale_factor,
             browser_plugin_guest_->update_scale_factor_received_);
 
   if (UseSurfacesEnabled()) {
     cc::SurfaceId id = surface_id();
     if (!id.is_null()) {
@@ skipping 10 matching lines @@
       EXPECT_EQ(id, browser_plugin_guest_->last_surface_id_received_);
       EXPECT_EQ(view_size, browser_plugin_guest_->last_frame_size_received_);
       EXPECT_EQ(scale_factor,
                 browser_plugin_guest_->last_scale_factor_received_);
     }
   } else {
     EXPECT_TRUE(browser_plugin_guest_->received_delegated_frame_);
     EXPECT_EQ(view_size, browser_plugin_guest_->last_frame_size_received_);
     EXPECT_EQ(scale_factor, browser_plugin_guest_->last_scale_factor_received_);
   }
+
+  browser_plugin_guest_->ResetTestData();
+  browser_plugin_guest_->set_has_attached_since_surface_set(true);
+
+  view_->OnSwapCompositorFrame(
+      0, CreateDelegatedFrame(scale_factor, view_size, view_rect));
+
+  if (UseSurfacesEnabled()) {
+    cc::SurfaceId id = surface_id();
+    if (!id.is_null()) {
+#if !defined(OS_ANDROID)
+      ImageTransportFactory* factory = ImageTransportFactory::GetInstance();
+      cc::SurfaceManager* manager = factory->GetSurfaceManager();
+      cc::Surface* surface = manager->GetSurfaceForId(id);
+      EXPECT_TRUE(surface);
+      // There should be a SurfaceSequence created by the RWHVGuest.
+      EXPECT_EQ(1u, surface->GetDestructionDependencyCount());
+#endif
+      // Surface ID should have been passed to BrowserPluginGuest to
+      // be sent to the embedding renderer.
+      EXPECT_EQ(id, browser_plugin_guest_->last_surface_id_received_);
+      EXPECT_EQ(view_size, browser_plugin_guest_->last_frame_size_received_);
+      EXPECT_EQ(scale_factor,
+                browser_plugin_guest_->last_scale_factor_received_);
+    }
+  } else {
+    EXPECT_TRUE(browser_plugin_guest_->received_delegated_frame_);
+    EXPECT_EQ(view_size, browser_plugin_guest_->last_frame_size_received_);
+    EXPECT_EQ(scale_factor, browser_plugin_guest_->last_scale_factor_received_);
+  }
+
+  browser_plugin_guest_->set_attached(false);
+  browser_plugin_guest_->ResetTestData();
+
+  view_->OnSwapCompositorFrame(
+      0, CreateDelegatedFrame(scale_factor, view_size, view_rect));
+  EXPECT_EQ(gfx::Size(), browser_plugin_guest_->update_frame_size_received_);
+  EXPECT_EQ(0.f, browser_plugin_guest_->update_scale_factor_received_);
+  if (UseSurfacesEnabled())
+    EXPECT_TRUE(surface_id().is_null());
 }
 
 }  // namespace content