Add a document phase for calculating paint properties

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

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 177 matching lines @@
         break;
     case InPaintInvalidation:
         return nextState == PaintInvalidationClean;
     case PaintInvalidationClean:
         if (nextState == InStyleRecalc)
             return true;
         if (nextState == InPreLayout)
             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:
         if (nextState == PaintClean && RuntimeEnabledFeatures::slimmingPaintSynchronizedPaintingEnabled())
             return true;
         break;
     case PaintClean:
         if (!RuntimeEnabledFeatures::slimmingPaintSynchronizedPaintingEnabled())
             break;
         if (!RuntimeEnabledFeatures::slimmingPaintV2Enabled()) {
@@ skipping 61 matching lines @@
 void DocumentLifecycle::ensureStateAtMost(State state)
 {
     ASSERT(state == VisualUpdatePending || state == StyleClean || state == LayoutClean);
     if (m_state <= state)
         return;
     ASSERT(canRewindTo(state));
     m_state = state;
 }
 
 }