Preparation for enabling slimming paint synchronized painting

third_party/WebKit/Source/core/paint/PaintControllerPaintTest.h

 // Copyright 2015 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.
 
 #ifndef PaintControllerPaintTest_h
 #define PaintControllerPaintTest_h
 
 #include "core/frame/FrameView.h"
 #include "core/layout/LayoutTestHelper.h"
 #include "core/layout/LayoutView.h"
 #include "core/paint/PaintLayer.h"
 #include "platform/graphics/GraphicsLayer.h"
 #include <gtest/gtest.h>
 
 namespace blink {
 
-class PaintControllerPaintTest : public RenderingTest {
+class PaintControllerPaintTestBase : public RenderingTest {
 public:
-    PaintControllerPaintTest()
+    PaintControllerPaintTestBase(bool enableSlimmingPaintV2)
         : m_originalSlimmingPaintSynchronizedPaintingEnabled(RuntimeEnabledFeatures::slimmingPaintSynchronizedPaintingEnabled())
         , m_originalSlimmingPaintOffsetCachingEnabled(RuntimeEnabledFeatures::slimmingPaintOffsetCachingEnabled())
-        { }
+        , m_originalSlimmingPaintV2Enabled(RuntimeEnabledFeatures::slimmingPaintV2Enabled())
+        , m_enableSlimmingPaintV2(enableSlimmingPaintV2)
+    { }
 
 protected:
     LayoutView& layoutView() { return *document().layoutView(); }
     PaintController& rootPaintController() { return *layoutView().layer()->graphicsLayerBacking()->paintController(); }
 
-private:
     void SetUp() override
     {
         RenderingTest::SetUp();
         enableCompositing();
         GraphicsLayer::setDrawDebugRedFillForTesting(false);
+        RuntimeEnabledFeatures::setSlimmingPaintV2Enabled(m_enableSlimmingPaintV2);
     }
     void TearDown() override
     {
         RuntimeEnabledFeatures::setSlimmingPaintSynchronizedPaintingEnabled(m_originalSlimmingPaintSynchronizedPaintingEnabled);
         RuntimeEnabledFeatures::setSlimmingPaintOffsetCachingEnabled(m_originalSlimmingPaintOffsetCachingEnabled);
+        RuntimeEnabledFeatures::setSlimmingPaintV2Enabled(m_originalSlimmingPaintV2Enabled);
         GraphicsLayer::setDrawDebugRedFillForTesting(true);
     }
 
-    bool m_originalSlimmingPaintSynchronizedPaintingEnabled;
-    bool m_originalSlimmingPaintOffsetCachingEnabled;
-};
-
-// Slimming paint v2 has subtly different behavior on some paint tests. This
-// class is used to test only v2 behavior while maintaining v1 test coverage.
-class PaintControllerPaintTestForSlimmingPaintV2 : public RenderingTest {
-public:
-    PaintControllerPaintTestForSlimmingPaintV2()
-        : m_originalSlimmingPaintV2Enabled(RuntimeEnabledFeatures::slimmingPaintV2Enabled()) { }
-
-protected:
-    LayoutView& layoutView() { return *document().layoutView(); }
-    PaintController& rootPaintController() { return *layoutView().layer()->graphicsLayerBacking()->paintController(); }
-
     // Expose some document lifecycle steps for checking new display items before commiting.
     void updateLifecyclePhasesToPaintClean(const LayoutRect* interestRect = nullptr)
     {
         document().view()->updateLifecyclePhasesInternal(FrameView::OnlyUpToCompositingCleanPlusScrolling, nullptr);
         document().view()->invalidateTreeIfNeededRecursive();
         document().view()->updatePaintProperties();
         document().view()->synchronizedPaint(interestRect);
     }
     void compositeForSlimmingPaintV2() { document().view()->compositeForSlimmingPaintV2(); }
 
 private:
-    void SetUp() override
-    {
-        RuntimeEnabledFeatures::setSlimmingPaintV2Enabled(true);
+    bool m_originalSlimmingPaintSynchronizedPaintingEnabled;
+    bool m_originalSlimmingPaintOffsetCachingEnabled;
+    bool m_originalSlimmingPaintV2Enabled;
+    bool m_enableSlimmingPaintV2;
+};
 
-        RenderingTest::SetUp();
-        enableCompositing();
-        GraphicsLayer::setDrawDebugRedFillForTesting(false);
-    }
+class PaintControllerPaintTest : public PaintControllerPaintTestBase {
+public:
+    PaintControllerPaintTest() : PaintControllerPaintTestBase(false) { }
+};
 
-    void TearDown() override
-    {
-        GraphicsLayer::setDrawDebugRedFillForTesting(true);
-        RuntimeEnabledFeatures::setSlimmingPaintV2Enabled(m_originalSlimmingPaintV2Enabled);
-    }
+class PaintControllerPaintTestForSlimmingPaintV2 : public PaintControllerPaintTestBase {
+public:
+    PaintControllerPaintTestForSlimmingPaintV2() : PaintControllerPaintTestBase(true) { }
+};
 
-    bool m_originalSlimmingPaintV2Enabled;
+class PaintControllerPaintTestForSlimmingPaintV1AndV2

[Xianzhu, 2015/10/28 18:10:23]

With this we can run the same test for both v1 and v2.

+    : public PaintControllerPaintTestBase
+    , public testing::WithParamInterface<bool> {
+public:
+    PaintControllerPaintTestForSlimmingPaintV1AndV2() : PaintControllerPaintTestBase(GetParam()) { }
 };
 
 class TestDisplayItem final : public DisplayItem {
 public:
     TestDisplayItem(const DisplayItemClientWrapper& client, Type type) : DisplayItem(client, type, sizeof(*this)) { }
 
     void replay(GraphicsContext&) const final { ASSERT_NOT_REACHED(); }
     void appendToWebDisplayItemList(WebDisplayItemList*) const final { ASSERT_NOT_REACHED(); }
 };
 
@@ skipping 23 matching lines @@
 const DisplayItem::Type cachedBackgroundType = DisplayItem::drawingTypeToCachedDrawingType(backgroundType);
 const DisplayItem::Type foregroundType = DisplayItem::paintPhaseToDrawingType(PaintPhaseForeground);
 const DisplayItem::Type cachedForegroundType = DisplayItem::drawingTypeToCachedDrawingType(foregroundType);
 const DisplayItem::Type subsequenceType = DisplayItem::SubsequenceNormalFlowAndPositiveZOrder;
 const DisplayItem::Type endSubsequenceType = DisplayItem::subsequenceTypeToEndSubsequenceType(subsequenceType);
 const DisplayItem::Type cachedSubsequenceType = DisplayItem::subsequenceTypeToCachedSubsequenceType(subsequenceType);
 
 } // namespace blink
 
 #endif // PaintControllerPaintTest_h