//net changes for stale-while-revalidate

net/http/http_cache_transaction.cc

Index: net/http/http_cache_transaction.cc
diff --git a/net/http/http_cache_transaction.cc b/net/http/http_cache_transaction.cc
index 84c7f4ff76c0bc79d41ba21dd0a3a70ef0f726bb..eeb6063e66e0eea54fd5440fa0e031bb934c8cf8 100644
--- a/net/http/http_cache_transaction.cc
+++ b/net/http/http_cache_transaction.cc
@@ -2119,6 +2119,13 @@ int HttpCache::Transaction::BeginCacheValidation() {
 
   bool skip_validation = (required_validation == VALIDATION_NONE);
 
+  if ((effective_load_flags_ & LOAD_SUPPORT_ASYNC_REVALIDATION) &&
+      required_validation == VALIDATION_ASYNCHRONOUS) {
+    DCHECK(request_->method == "HEAD" || request_->method == "GET");

[hiroshige, 2015/06/12 05:58:03]

The doc says: "Only the GET method is in scope." and //content change allows
only "GET", so should we allow only "GET" here also?

[Adam Rice, 2015/06/12 06:41:57]

You're right. Fixed.

+    skip_validation = true;
+    response_.async_revalidation_required = true;
+  }
+
   if (request_->method == "HEAD" &&
       (truncated_ || response_.headers->response_code() == 206)) {
     DCHECK(!partial_);