Fix bug of animated table col/section/row background

third_party/WebKit/Source/core/paint/BoxPaintInvalidator.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/paint/BoxPaintInvalidator.h"
 
 #include "core/frame/Settings.h"
 #include "core/layout/LayoutView.h"
 #include "core/paint/ObjectPaintInvalidator.h"
 #include "core/paint/PaintInvalidator.h"
@@ skipping 94 matching lines @@
       *m_context.paintInvalidationContainer, bottomDelta,
       PaintInvalidationIncremental);
   return true;
 }
 
 PaintInvalidationReason BoxPaintInvalidator::computePaintInvalidationReason() {
   PaintInvalidationReason reason =
       ObjectPaintInvalidatorWithContext(m_box, m_context)
           .computePaintInvalidationReason();
 
-  if (isImmediateFullPaintInvalidationReason(reason) ||
-      reason == PaintInvalidationNone)
+  if (reason != PaintInvalidationIncremental)
     return reason;
 
-  if (m_box.mayNeedPaintInvalidationAnimatedBackgroundImage() &&
-      !m_box.backgroundIsKnownToBeObscured())
-    reason = PaintInvalidationDelayedFull;
-
-  // If the current paint invalidation reason is PaintInvalidationDelayedFull,
-  // then this paint invalidation can delayed if the LayoutBox in question is
-  // not on-screen. The logic to decide whether this is appropriate exists at
-  // the site of the original paint invalidation that chose
-  // PaintInvalidationDelayedFull.
-  if (reason == PaintInvalidationDelayedFull) {
-    // Do regular full paint invalidation if the object is onscreen.
-    return m_box.intersectsVisibleViewport() ? PaintInvalidationFull
-                                             : PaintInvalidationDelayedFull;
-  }
-
-  DCHECK(reason == PaintInvalidationIncremental);
-
   if (m_box.isLayoutView()) {
     const LayoutView& layoutView = toLayoutView(m_box);
     // In normal compositing mode, root background doesn't need to be
     // invalidated for box changes, because the background always covers the
     // whole document rect and clipping is done by
     // compositor()->m_containerLayer. Also the scrollbars are always
     // composited. There are no other box decoration on the LayoutView thus we
     // can safely exit here.
     if (layoutView.usesCompositing() &&
         !RuntimeEnabledFeatures::rootLayerScrollingEnabled())
@@ skipping 155 matching lines @@
 
 LayoutRect BoxPaintInvalidator::previousLayoutOverflowRect() {
   DCHECK(m_box.hasPreviousBoxGeometries() ==
          previousBoxGeometriesMap().contains(&m_box));
   return m_box.hasPreviousBoxGeometries()
              ? previousBoxGeometriesMap().get(&m_box).layoutOverflowRect
              : LayoutRect();
 }
 
 }  // namespace blink