[android_webview] Do not declare WebView visible on detaching from window.

android_webview/browser/browser_view_renderer_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 <map>
 #include <queue>
 #include <utility>
 
 #include "android_webview/browser/browser_view_renderer.h"
 #include "android_webview/browser/child_frame.h"
@@ skipping 273 matching lines @@
     }
     on_draw_count_++;
   }
 
  private:
   int on_draw_count_;
 };
 
 RENDERING_TEST_F(CompositorNoFrameTest);
 
+class ClientIsVisibleOnConstructionTest : public RenderingTest {
+  void SetUpTestHarness() override {
+    browser_view_renderer_.reset(
+        new BrowserViewRenderer(this, base::ThreadTaskRunnerHandle::Get()));
+  }
+
+  void StartTest() override {
+    EXPECT_FALSE(browser_view_renderer_->attached_to_window());
+    EXPECT_FALSE(browser_view_renderer_->window_visible_for_tests());
+    EXPECT_TRUE(browser_view_renderer_->IsClientVisible());
+    EndTest();
+  }
+};
+
+RENDERING_TEST_F(ClientIsVisibleOnConstructionTest);
+
+class ClientIsVisibleAfterAttachTest : public RenderingTest {
+  void StartTest() override {
+    EXPECT_TRUE(browser_view_renderer_->attached_to_window());
+    EXPECT_TRUE(browser_view_renderer_->window_visible_for_tests());
+
+    EXPECT_TRUE(browser_view_renderer_->IsClientVisible());
+    EndTest();
+  }
+};
+
+RENDERING_TEST_F(ClientIsVisibleAfterAttachTest);
+
+class ClientIsInvisibleAfterWindowGoneTest : public RenderingTest {
+  void StartTest() override {
+    browser_view_renderer_->SetWindowVisibility(false);
+    EXPECT_FALSE(browser_view_renderer_->IsClientVisible());
+    EndTest();
+  }
+};
+
+RENDERING_TEST_F(ClientIsInvisibleAfterWindowGoneTest);
+
+class ClientIsInvisibleAfterDetachTest : public RenderingTest {
+  void StartTest() override {
+    browser_view_renderer_->OnDetachedFromWindow();
+    EXPECT_FALSE(browser_view_renderer_->IsClientVisible());
+    EndTest();
+  }
+};
+
+RENDERING_TEST_F(ClientIsInvisibleAfterDetachTest);
+
 class ResourceRenderingTest : public RenderingTest {
  public:
   using ResourceCountMap = std::map<cc::ResourceId, int>;
   using CompositorFrameSinkResourceCountMap =
       std::map<uint32_t, ResourceCountMap>;
 
   virtual std::unique_ptr<content::SynchronousCompositor::Frame> GetFrame(
       int frame_number) = 0;
 
   void StartTest() override {
@@ skipping 193 matching lines @@
   }
 
  private:
   std::unique_ptr<FakeFunctor> saved_functor_;
   CompositorFrameSinkResourceCountMap expected_return_count_;
 };
 
 RENDERING_TEST_F(RenderThreadManagerSwitchTest);
 
 }  // namespace android_webview