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

Unified Diff: content/browser/appcache/appcache_url_loader_request.cc

Issue 2974733002: Add support for subresource request fallback in AppCache for the network service.. (Closed)
Patch Set: Format changes Created 3 years, 5 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: content/browser/appcache/appcache_url_loader_request.cc
diff --git a/content/browser/appcache/appcache_url_loader_request.cc b/content/browser/appcache/appcache_url_loader_request.cc
index 11c85ba497b80b8c979f9f8541e20730b2345d77..4669be97d1549279b6b824ed450ffb90ea486a25 100644
--- a/content/browser/appcache/appcache_url_loader_request.cc
+++ b/content/browser/appcache/appcache_url_loader_request.cc
@@ -3,6 +3,7 @@
// found in the LICENSE file.
#include "content/browser/appcache/appcache_url_loader_request.h"
+#include "content/public/common/resource_type.h"
#include "net/url_request/url_request.h"
namespace content {
@@ -33,7 +34,8 @@ const GURL AppCacheURLLoaderRequest::GetReferrer() const {
}
bool AppCacheURLLoaderRequest::IsSuccess() const {
- return false;
+ int response_code = GetResponseCode();
+ return (response_code >= 200 && response_code <= 226);
}
bool AppCacheURLLoaderRequest::IsCancelled() const {
@@ -45,6 +47,8 @@ bool AppCacheURLLoaderRequest::IsError() const {
}
int AppCacheURLLoaderRequest::GetResponseCode() const {
+ if (response_.headers)
+ return response_.headers->response_code();
return 0;
}
@@ -57,6 +61,10 @@ ResourceRequest* AppCacheURLLoaderRequest::GetResourceRequest() {
return &request_;
}
+AppCacheURLLoaderRequest* AppCacheURLLoaderRequest::AsURLLoaderRequest() {
+ return this;
+}
+
AppCacheURLLoaderRequest::AppCacheURLLoaderRequest(
const ResourceRequest& request)
: request_(request) {}
« no previous file with comments | « content/browser/appcache/appcache_url_loader_request.h ('k') | content/browser/appcache/appcache_url_request.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698