Unit Test for WebView animating in and out of screen

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 "android_webview/browser/browser_view_renderer.h"
 #include "android_webview/browser/test/rendering_test.h"
 
 namespace android_webview {
 
 class SmokeTest : public RenderingTest {
@@ skipping 41 matching lines @@
 
   void DidDrawOnRT(SharedRendererState* functor) override {
     EndTest();
   }
  private:
   size_t on_draw_count_;
 };
 
 RENDERING_TEST_F(ClearViewTest);
 
+class TestForceInvalidateWithEmptyGlobalVisibleRect : public RenderingTest {
+ public:
+  TestForceInvalidateWithEmptyGlobalVisibleRect() : on_draw_count_(0u) {}
+
+  void StartTest() override {
+    browser_view_renderer_->SetContinuousInvalidate(true);
+  }
+
+  void WillOnDraw() override {
+    browser_view_renderer_->PrepareToDraw(gfx::Vector2d(), gfx::Rect());
+    browser_view_renderer_->SetContinuousInvalidate(false);
+  }
+
+  void DidOnDraw(bool success) override {
+    on_draw_count_++;
+
+    ParentCompositorDrawConstraints initialConstraint;
+    ParentCompositorDrawConstraints newConstraint =
+        ParentCompositorDrawConstraints(false, gfx::Transform(),
+                                        gfx::Rect(window_->surface_size()));
+    EXPECT_TRUE(success);
+    // The second onDraw will be the result of a forced invalidation because
+    // an empty global visible rect was used in the first onDraw.
+    if (on_draw_count_ == 1u) {
+      EXPECT_TRUE(
+          browser_view_renderer_->parent_draw_constraints_for_testing().Equals(
+              initialConstraint));
+    }
+
+    if (on_draw_count_ == 2u) {
+      EXPECT_TRUE(
+          browser_view_renderer_->parent_draw_constraints_for_testing().Equals(
+              newConstraint));
+      EndTest();
+    }
+  }
+
+ private:
+  size_t on_draw_count_;
+};
+
+RENDERING_TEST_F(TestForceInvalidateWithEmptyGlobalVisibleRect);
+
+class TestAnimateInAndOutOfScreen : public RenderingTest {

[boliu, 2015/03/13 19:14:22]

I think this should replace the test above..

[hush (inactive), 2015/03/13 23:24:04]

Yes. Indeed.

+ public:
+  TestAnimateInAndOutOfScreen() : on_draw_count_(0u) {}
+
+  void StartTest() override {
+    browser_view_renderer_->SetContinuousInvalidate(true);
+  }
+
+  void WillOnDraw() override {
+    on_draw_count_++;
+    // First and third onDraws are onscreen.
+    // Second onDraw is offscreen.
+    if (on_draw_count_ == 2u) {
+      browser_view_renderer_->PrepareToDraw(gfx::Vector2d(), gfx::Rect());
+      browser_view_renderer_->SetContinuousInvalidate(false);

[boliu, 2015/03/13 19:14:22]

Let's see

First draw is normal, and ParentConstraints gets updated (or I guess maybe not
because Equals return true, doesn't matter

Second draw sets visible rect to empty, and DrawGL gets skipped.

Third draw comes from the force invalidate from the second one. I assume? But
how does this work when the DrawGL from 2 got skipped (and we didn't schedule
another one?

[hush (inactive), 2015/03/13 23:24:04]

Please the see the new patch. I added some new synchronization logic in fake
window so that I can guarantee that the ModeDraw is finished before the next
onDraw on UI thread. In this way, when the 2nd onDraw runs, the new parent draw
constraints are already posted to UI thread.

+    }
+  }
+
+  void DidOnDraw(bool success) override {
+    ParentCompositorDrawConstraints initialConstraint;
+    ParentCompositorDrawConstraints newConstraint =
+        ParentCompositorDrawConstraints(false, gfx::Transform(),
+                                        gfx::Rect(window_->surface_size()));
+    EXPECT_TRUE(success);
+    if (on_draw_count_ == 1u) {
+      EXPECT_TRUE(
+          browser_view_renderer_->parent_draw_constraints_for_testing().Equals(
+              initialConstraint));
+    }
+
+    if (on_draw_count_ == 2u) {
+      EXPECT_TRUE(
+          browser_view_renderer_->parent_draw_constraints_for_testing().Equals(
+              newConstraint));
+    }
+
+    if (on_draw_count_ == 3u) {
+      EXPECT_TRUE(
+          browser_view_renderer_->parent_draw_constraints_for_testing().Equals(
+              newConstraint));
+      EndTest();
+    }
+  }
+
+  bool WillDrawOnRT(SharedRendererState* functor) override {
+    if (on_draw_count_ == 2u)
+      return false;
+
+    return true;
+  }
+
+ private:
+  size_t on_draw_count_;
+};
+
+RENDERING_TEST_F(TestAnimateInAndOutOfScreen);
+
 }  // namespace android_webview