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(100, 100));

[boliu, 2015/03/13 17:53:34]

Can we get these from FakeWindow instead of hard coding?

[hush (inactive), 2015/03/13 18:33:30]

Yes. Sure. 

[boliu, 2015/03/13 18:50:37]

Ehh, get the other two as well? You can get FakeWindow to return the whole
ParentConstraints

+    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));

[boliu, 2015/03/13 17:53:34]

Shouldn't this be newConstraint? Then drop initialConstraint.

[hush (inactive), 2015/03/13 18:33:30]

No. After the first onDraw, the new constraint from parent compositor hasn't
been posted back to UI yet. The new constraint will be set within the second
onDrawHardware.

[boliu, 2015/03/13 18:50:37]

Then test setup is still wrong, no? Test is supposed to test what's in BVR
already matches the "new" constraints, so without this "force invalidate" thing,
a new invalidate would not happen.

And on a practical note, doesn't Equals ignore surface size if is_layer is
false?

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

The test setup has some synchronization problems I think... I added more
capabilities to the fake_window class and gave the test more control.

+    }
+
+    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);
+
 }  // namespace android_webview