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

Unified Diff: third_party/WebKit/Source/core/css/CSSImageValue.cpp

Issue 2423683002: Add Blink support for showing image placeholders using range requests. (Closed)
Patch Set: Created 4 years, 2 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
Index: third_party/WebKit/Source/core/css/CSSImageValue.cpp
diff --git a/third_party/WebKit/Source/core/css/CSSImageValue.cpp b/third_party/WebKit/Source/core/css/CSSImageValue.cpp
index 94237b4e0f56820d5bbcaf8fd75b9d05a7ceff0c..04568a6a82513b5f565315661efa2b1d9eab159d 100644
--- a/third_party/WebKit/Source/core/css/CSSImageValue.cpp
+++ b/third_party/WebKit/Source/core/css/CSSImageValue.cpp
@@ -26,6 +26,7 @@
#include "core/fetch/FetchRequest.h"
#include "core/fetch/ImageResource.h"
#include "core/fetch/ResourceFetcher.h"
+#include "core/frame/Settings.h"
#include "core/loader/MixedContentChecker.h"
#include "core/style/StyleFetchedImage.h"
#include "core/style/StyleInvalidImage.h"
@@ -64,6 +65,10 @@ StyleImage* CSSImageValue::cacheImage(const Document& document,
if (crossOrigin != CrossOriginAttributeNotSet)
request.setCrossOriginAccessControl(document.getSecurityOrigin(),
crossOrigin);
+ request.setPlaceholderImageRequestType(
+ document.settings() && document.settings()->fetchImagePlaceholders()
+ ? FetchRequest::AllowPlaceholder
+ : FetchRequest::DisallowPlaceholder);
if (ImageResource* cachedImage =
ImageResource::fetch(request, document.fetcher()))

Powered by Google App Engine
This is Rietveld 408576698