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

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

Issue 1304063016: Refactor the API for setting dynamic resource load priorities (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 5 years, 3 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 53fa66bdae50ef55887434f8153c654b47680a81..fed963428b608d23de0d52c21ea615399cabd7f4 100644
--- a/Source/core/layout/LayoutImage.cpp
+++ b/Source/core/layout/LayoutImage.cpp
@@ -31,7 +31,6 @@
#include "core/HTMLNames.h"
#include "core/editing/FrameSelection.h"
#include "core/fetch/ImageResource.h"
-#include "core/fetch/ResourceLoadPriorityOptimizer.h"
#include "core/fetch/ResourceLoader.h"
#include "core/frame/LocalFrame.h"
#include "core/html/HTMLAreaElement.h"
@@ -57,7 +56,6 @@ LayoutImage::LayoutImage(Element* element)
, m_isGeneratedContent(false)
, m_imageDevicePixelRatio(1.0f)
{
- ResourceLoadPriorityOptimizer::resourceLoadPriorityOptimizer()->addLayoutObject(this);
}
LayoutImage* LayoutImage::createAnonymous(Document* document)
@@ -307,36 +305,6 @@ void LayoutImage::layout()
updateInnerContentRect();
}
-bool LayoutImage::updateImageLoadingPriorities()
-{
- if (!m_imageResource || !m_imageResource->cachedImage() || m_imageResource->cachedImage()->isLoaded())
- return false;
-
- LayoutRect viewBounds = viewRect();
- LayoutRect objectBounds = LayoutRect(absoluteContentBox());
-
- // The object bounds might be empty right now, so intersects will fail since it doesn't deal
- // with empty rects. Use LayoutRect::contains in that case.
- bool isVisible;
- if (!objectBounds.isEmpty())
- isVisible = viewBounds.intersects(objectBounds);
- else
- isVisible = viewBounds.contains(objectBounds);
-
- ResourceLoadPriorityOptimizer::VisibilityStatus status = isVisible ?
- ResourceLoadPriorityOptimizer::Visible : ResourceLoadPriorityOptimizer::NotVisible;
-
- LayoutRect screenArea;
- if (!objectBounds.isEmpty()) {
- screenArea = viewBounds;
- screenArea.intersect(objectBounds);
- }
-
- ResourceLoadPriorityOptimizer::resourceLoadPriorityOptimizer()->notifyImageResourceVisibility(m_imageResource->cachedImage(), status, screenArea);
-
- return true;
-}
-
void LayoutImage::computeIntrinsicRatioInformation(FloatSize& intrinsicSize, double& intrinsicRatio) const
{
LayoutReplaced::computeIntrinsicRatioInformation(intrinsicSize, intrinsicRatio);
« 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