Add test for paint underinvalidation fix with animated transform overflow.

third_party/WebKit/Source/web/tests/PaintInvalidationTest.cpp

+// Copyright 2016 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 "core/frame/FrameView.h"
+#include "core/layout/LayoutView.h"
+#include "core/paint/PaintLayer.h"
+#include "core/paint/PaintLayerScrollableArea.h"
+#include "platform/testing/RuntimeEnabledFeaturesTestHelpers.h"
+#include "platform/testing/TestingPlatformSupport.h"
+#include "platform/testing/UnitTestHelpers.h"
+#include "public/web/WebScriptSource.h"
+#include "public/web/WebSettings.h"
+#include "testing/gtest/include/gtest/gtest.h"
+#include "web/WebLocalFrameImpl.h"
+#include "web/tests/sim/SimDisplayItemList.h"
+#include "web/tests/sim/SimRequest.h"
+#include "web/tests/sim/SimTest.h"
+
+namespace blink {
+
+namespace {
+
+void enableViewport(WebSettings* settings) {
+  settings->setViewportEnabled(true);
+}
+
+class PaintInvalidationTest : public SimTest {
+ public:
+  PaintInvalidationTest() : SimTest(enableViewport)
+  { }
+
+ protected:
+  bool invalidationArrayContains(JSONArray* invalidations, LayoutObject& object) {

[pdr., 2017/02/23 18:24:33]

This unittest is basically equivalent to a layout test, but I don't think we
need to go all the way to invalidation rects. Can you DCHECK that the layout
view item does not need paint invalidation before the animation, then DCHECK
that it does afterwards? This must have taken a long time to write so I feel a
little bad about deleting it :(

[bokan, 2017/02/23 18:41:17]

I don't mind deleting it if I can (I feel bad having yet another place checking
invalidation) but I don't think we can do it though a layout test reliably as it
relies on checking the invalidation mid-animation which is hard to do in a
non-flaky way with layout tests. Worse, it requires turning on the viewport
enabled setting which is a dubious thing to do from JS as the page is already
loaded. We could run this in a virtual test suite but that seems like overkill
and we still have the animation induced in-determinism.

I also can't DCHECK for needs invalidation since the beginFrame both ticks the
animation and then does a full lifecycle update so once we're back in test code
the invalidation is already cleared.

PS: I think this is superior to the layout test since it's less brittle. The
layout test guards against any changes in invalidation which can be problematic
when new features/optimizations cause additional/fewer unrelated invalidations.
I've had this happen before where a new invalidation caused me to have to rebase
a bunch of unrelated tests. By contrast, this only checks that we invalidated
the layout view in some way -- regardless of ordering and number of times.

[pdr., 2017/02/23 19:42:07]

Do we need to run the full lifecycle though? In other words, can you do
something like:
1) dcheck that the view does not yet need paint invalidation
2) update lifecycle to style clean
3) dcheck that the view does need paint invalidation

+    for (size_t i = 0; i < invalidations->size(); ++i) {
+      JSONValue* value = invalidations->at(i);
+      if (!value)
+        continue;
+
+      if (value->getType() != JSONValue::TypeObject)
+        continue;
+
+      JSONObject* invalidationPair = JSONObject::cast(value);
+      DCHECK(invalidationPair);
+      JSONValue* name = invalidationPair->get("object");
+      if (name && name->getType() == JSONValue::TypeString) {
+        String nameString;
+        name->asString(&nameString);
+        if (nameString == object.debugName())
+          return true;
+      }
+    }
+
+    return false;
+  }
+};
+
+class NoCompositingPlatform : public TestingPlatformSupport {
+ public:
+  NoCompositingPlatform() {}
+  ~NoCompositingPlatform() override {}
+  bool isThreadedAnimationEnabled() override { return false; }
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(NoCompositingPlatform);
+};
+
+TEST_F(PaintInvalidationTest, AnimatedTransformOverflowInvalidatesBackground) {
+  // SimCompositor doesn't yet support animations ticked on the compositor so
+  // ensure we tick it on the main thread.
+  ScopedTestingPlatformSupport<NoCompositingPlatform> platform;
+
+  v8::HandleScope handleScope(v8::Isolate::GetCurrent());
+  webView().resize(WebSize(800, 600));
+  SimRequest request("https://example.com/test.html", "text/html");
+  loadURL("https://example.com/test.html");
+  request.complete(
+      "<!DOCTYPE html>"
+      "<style type='text/css'>"
+      "  body, html {"
+      "    width: 95%;"
+      "    height: 95%;"
+      "    margin: 0px;"
+      "  }"
+      "  #container {"
+      "    width: 100%;"
+      "    height: 100%;"
+      "    transform-origin: 0% 100%;"
+      "    transform: rotateZ(0);"
+      "    transition: transform 1s;"
+      "    background-color: red;"
+      "  }"
+      "  #container.tilted {"
+      "    transform: rotateZ(45deg);"
+      "  }"
+      "</style>"
+      "<div id='container'></div>");
+
+  ScrollableArea* scrollableArea = document().view();
+  compositor().beginFrame();
+  ASSERT_EQ(scrollableArea->maximumScrollOffset().height(), 0);
+
+  // Apply the tilted class so that the animated transition begins.
+  mainFrame().executeScript(WebScriptSource(
+      "document.querySelector('#container').className = 'tilted';"));
+
+  // Tick one frame to ensure the animation is started.
+  document().updateStyleAndLayoutTreeIgnorePendingStylesheets();
+  compositor().beginFrame();
+
+  // Reset paint invalidation tracking.
+  document().view()->setTracksPaintInvalidations(false);
+  document().view()->setTracksPaintInvalidations(true);
+
+  // Tick ahead about half way through the animation. The animation isn't
+  // finished but it should have caused overflow to be added so the main frame
+  // is scrollable.
+  document().updateStyleAndLayoutTreeIgnorePendingStylesheets();
+  compositor().beginFrame(0.5);
+  ASSERT_GT(scrollableArea->maximumScrollOffset().height(), 100);
+
+  // Ensure that we invalidated the LayoutView due to the new overflow so that
+  // scrolling it into the viewport doesn't expose an unpainted region.
+  std::unique_ptr<JSONArray> invalidations =
+      document().view()->trackedObjectPaintInvalidationsAsJSON();
+  EXPECT_TRUE(
+      invalidationArrayContains(invalidations.get(), *document().layoutView()));
+}
+
+}  // namespace
+
+}  // namespace blink