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

Unified Diff: Source/core/layout/LayoutImage.cpp

Issue 1025603002: Revert of [S.P.] Don't draw frames of animated images that are offscreen. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 5 years, 9 months 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
« no previous file with comments | « Source/core/layout/LayoutImage.h ('k') | Source/core/layout/LayoutObject.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/layout/LayoutImage.cpp
diff --git a/Source/core/layout/LayoutImage.cpp b/Source/core/layout/LayoutImage.cpp
index 237c5aa70db2b5f54ce9f24aecacf901736ee274..d93280a590121c736797572bd0154ec2b1a3282f 100644
--- a/Source/core/layout/LayoutImage.cpp
+++ b/Source/core/layout/LayoutImage.cpp
@@ -39,7 +39,6 @@
#include "core/html/HTMLInputElement.h"
#include "core/html/HTMLMapElement.h"
#include "core/layout/HitTestResult.h"
-#include "core/layout/LayoutPart.h"
#include "core/layout/LayoutView.h"
#include "core/layout/PaintInfo.h"
#include "core/layout/TextRunConstructor.h"
@@ -176,30 +175,6 @@
contentChanged(ImageChanged);
}
-bool LayoutImage::rectIntersectsVisibleViewport(const LayoutRect& localRect)
-{
- LayoutRect rect = localRect;
- LayoutView* layoutView = view();
- while (layoutView->frame()->ownerLayoutObject())
- layoutView = layoutView->frame()->ownerLayoutObject()->view();
- mapRectToPaintInvalidationBacking(layoutView, rect, 0);
- return rect.intersects(LayoutRect(layoutView->frameView()->visualViewportRect()));
-}
-
-PaintInvalidationReason LayoutImage::invalidatePaintIfNeeded(const PaintInvalidationState& paintInvalidationState, const LayoutBoxModelObject& paintInvalidationContainer)
-{
- if (!RuntimeEnabledFeatures::slimmingPaintEnabled())
- return LayoutReplaced::invalidatePaintIfNeeded(paintInvalidationState, paintInvalidationContainer);
-
- if (!imageResource() || !imageResource()->image() || !imageResource()->image()->maybeAnimated()
- || rectIntersectsVisibleViewport(visualOverflowRect())) {
- return LayoutReplaced::invalidatePaintIfNeeded(paintInvalidationState, paintInvalidationContainer);
- }
-
- return PaintInvalidationDelayedFull;
-}
-
-
void LayoutImage::notifyFinished(Resource* newImage)
{
if (!m_imageResource)
« no previous file with comments | « Source/core/layout/LayoutImage.h ('k') | Source/core/layout/LayoutObject.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698