Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(297)

Unified Diff: third_party/WebKit/Source/core/paint/BoxPaintInvalidator.cpp

Issue 2501833002: Fix bug of animated table col/section/row background (Closed)
Patch Set: - Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/paint/BoxPaintInvalidator.cpp
diff --git a/third_party/WebKit/Source/core/paint/BoxPaintInvalidator.cpp b/third_party/WebKit/Source/core/paint/BoxPaintInvalidator.cpp
index a97bd7d22720ecf4d17918d77bc6d24471214cd7..34c56d65e6562dc8b404d9aaa7d16690874965c0 100644
--- a/third_party/WebKit/Source/core/paint/BoxPaintInvalidator.cpp
+++ b/third_party/WebKit/Source/core/paint/BoxPaintInvalidator.cpp
@@ -112,27 +112,9 @@ PaintInvalidationReason BoxPaintInvalidator::computePaintInvalidationReason() {
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

Powered by Google App Engine
This is Rietveld 408576698