Add a document phase for calculating paint properties

third_party/WebKit/Source/core/dom/DocumentLifecycle.cpp

Index: third_party/WebKit/Source/core/dom/DocumentLifecycle.cpp
diff --git a/third_party/WebKit/Source/core/dom/DocumentLifecycle.cpp b/third_party/WebKit/Source/core/dom/DocumentLifecycle.cpp
index 12a710f80ef25f92954903502582d25b2493a29e..3968daa4b851d2ff034cdad6083386f144986b7b 100644
--- a/third_party/WebKit/Source/core/dom/DocumentLifecycle.cpp
+++ b/third_party/WebKit/Source/core/dom/DocumentLifecycle.cpp
@@ -195,7 +195,20 @@ bool DocumentLifecycle::canAdvanceTo(State nextState) const
             return true;
         if (nextState == InCompositingUpdate)
             return true;
-        if (nextState == InPaint && RuntimeEnabledFeatures::slimmingPaintSynchronizedPaintingEnabled())
+        if (RuntimeEnabledFeatures::slimmingPaintV2Enabled()) {
+            if (nextState == InCalcPaintProps)
+                return true;
+        } else {
+            if (nextState == InPaint && RuntimeEnabledFeatures::slimmingPaintSynchronizedPaintingEnabled())

[chrishtr, 2015/09/30 21:36:33]

Why this part?

[pdr., 2015/09/30 21:53:14]

slimmingPaintSynchronizedPaintingEnabled will soon work for spv1 and we want to
skip over the paint property step for spv1, but disallow skipping it for spv2.

SPV1 with synchronized painting:
InPaintInvalidation->PaintInvalidationClean->InPaint->PaintClean

SPV2 (synchronized painting implied):
InPaintInvalidation->PaintInvalidationClean->InCalcPaintProps->CalcPaintPropsClean->InPaint->PaintClean

+                return true;
+        }
+        break;
+    case InCalcPaintProps:
+        if (nextState == CalcPaintPropsClean && RuntimeEnabledFeatures::slimmingPaintV2Enabled())
+            return true;
+        break;
+    case CalcPaintPropsClean:
+        if (nextState == InPaint && RuntimeEnabledFeatures::slimmingPaintV2Enabled())
             return true;
         break;
     case InPaint: