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

Unified Diff: Source/core/fetch/ImageResource.cpp

Issue 46313002: Support the DPR header for client hints. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 7 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: Source/core/fetch/ImageResource.cpp
diff --git a/Source/core/fetch/ImageResource.cpp b/Source/core/fetch/ImageResource.cpp
index 90011895438217c44e913ea1e761de59110dad67..7603cbec2126e42d791717649d688cddf4527530 100644
--- a/Source/core/fetch/ImageResource.cpp
+++ b/Source/core/fetch/ImageResource.cpp
@@ -23,6 +23,7 @@
#include "config.h"
#include "core/fetch/ImageResource.h"
+#include "RuntimeEnabledFeatures.h"
pdr. 2013/10/26 01:26:31 Nit: Can you put this include one line lower? ...
#include "core/fetch/ImageResourceClient.h"
#include "core/fetch/MemoryCache.h"
@@ -44,8 +45,10 @@ namespace WebCore {
ImageResource::ImageResource(const ResourceRequest& resourceRequest)
: Resource(resourceRequest, Image)
+ , m_dprHeaderValue(1.0)
, m_image(0)
, m_loadingMultipartContent(false)
+ , m_hasDprHeaderValue(false)
{
setStatus(Unknown);
setCustomAcceptHeader();
@@ -377,6 +380,13 @@ void ImageResource::responseReceived(const ResourceResponse& response)
finishOnePart();
else if (response.isMultipart())
m_loadingMultipartContent = true;
+ if (RuntimeEnabledFeatures::clientHintsDprEnabled()) {
+ m_dprHeaderValue = response.httpHeaderField("DPR").toFloat(&m_hasDprHeaderValue);
+ if (!m_hasDprHeaderValue || m_dprHeaderValue <= 0.0) {
+ m_dprHeaderValue = 1.0;
+ m_hasDprHeaderValue = false;
+ }
+ }
Resource::responseReceived(response);
}

Powered by Google App Engine
This is Rietveld 408576698