Fix LayoutObject::SetSubtreeNeedsPaintPropertyUpdate() to set ancestor flags

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 247 matching lines @@
     case kPrePaintClean:
       if (next_state == kInPaint)
         return true;
       if (next_state == kInStyleRecalc)
         return true;
       if (next_state == kInPreLayout)
         return true;
       if (!RuntimeEnabledFeatures::slimmingPaintV2Enabled() &&
           next_state == kInCompositingUpdate)
         return true;
-      if (RuntimeEnabledFeatures::slimmingPaintV2Enabled() &&
+      if (RuntimeEnabledFeatures::slimmingPaintInvalidationEnabled() &&

[Xianzhu, 2017/04/20 21:01:13]

This is for the test to work.

This also makes transition of kInPrePaint consistent with other transitions: we
allow entering a previous kInXXX state from a later clean state. The transition
happens in tests though, because we always start from the kInStyleRecalc for
lifecycle updates in non-testing code. 

           next_state == kInPrePaint)
         return true;
       break;
     case kInPaint:
       if (next_state == kPaintClean)
         return true;
       break;
     case kPaintClean:
       if (next_state == kInStyleRecalc)
         return true;
       if (next_state == kInPreLayout)
         return true;
       if (!RuntimeEnabledFeatures::slimmingPaintV2Enabled() &&
           next_state == kInCompositingUpdate)
         return true;
-      if (RuntimeEnabledFeatures::slimmingPaintV2Enabled() &&
+      if (RuntimeEnabledFeatures::slimmingPaintInvalidationEnabled() &&
           next_state == kInPrePaint)
         return true;
       break;
     case kStopping:
       return next_state == kStopped;
     case kStopped:
       return false;
   }
   return false;
 }
@@ skipping 75 matching lines @@
       << " to " << StateAsDebugString(state) << ".";
 #endif
   state_ = state;
 }
 
 bool DocumentLifecycle::ThrottlingAllowed() const {
   return g_allow_throttling_count;
 }
 
 }  // namespace blink