Inform disk cache of WebKit memory cache hits.

net/http/http_cache.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "net/http/http_cache.h"
 
 #include <algorithm>
 
 #include "base/compiler_specific.h"
 
@@ skipping 445 matching lines @@
 }
 
 void HttpCache::CloseAllConnections() {
   net::HttpNetworkLayer* network =
       static_cast<net::HttpNetworkLayer*>(network_layer_.get());
   HttpNetworkSession* session = network->GetSession();
   if (session)
     session->CloseAllConnections();
 }
 
+class ReportExternalCacheHitJob {

[rvargas (doing something else), 2011/07/27 23:26:17]

Maybe we can simplify this code if the disk cache method doesn't return a value
(and if  we don't worry about the ActiveEntry). Then, there is no need for a
transaction :)

+ public:
+  ReportExternalCacheHitJob();
+  void Start(HttpCache* cache, const GURL& url, const std::string& http_method);
+
+ private:
+  void OnIOComplete(int rv);
+
+  HttpRequestInfo request_info_;
+  CompletionCallbackImpl<ReportExternalCacheHitJob> io_callback_;
+  scoped_ptr<HttpTransaction> trans_;
+};
+
+ReportExternalCacheHitJob::ReportExternalCacheHitJob() :
+    ALLOW_THIS_IN_INITIALIZER_LIST(
+        io_callback_(this, &ReportExternalCacheHitJob::OnIOComplete)) {
+}
+
+void ReportExternalCacheHitJob::Start(HttpCache* cache, const GURL& url,
+                                      const std::string& http_method) {
+  DCHECK(cache);
+  cache->CreateTransaction(&trans_);
+  request_info_.url = url;
+  request_info_.method = http_method;
+  request_info_.load_flags = LOAD_ONLY_FROM_CACHE;
+  int rv = trans_->Start(&request_info_, &io_callback_, BoundNetLog());
+  if (rv != ERR_IO_PENDING) {
+    OnIOComplete(rv);
+  }
+}
+
+void ReportExternalCacheHitJob::OnIOComplete(int rv) {
+  DCHECK(trans_.get());
+  if (rv == OK) {
+    trans_->UpdateRankForExternalCacheHit();
+  }
+  delete this;
+}
+
+void HttpCache::ReportExternalCacheHit(const GURL& url,
+                                       const std::string& http_method) {
+  ReportExternalCacheHitJob* job = new ReportExternalCacheHitJob;
+  job->Start(this, url, http_method);
+  // Job will delete itself when finished.
+}
+
 int HttpCache::CreateTransaction(scoped_ptr<HttpTransaction>* trans) {
   // Do lazy initialization of disk cache if needed.
   if (!disk_cache_.get())
     CreateBackend(NULL, NULL);  // We don't care about the result.
 
   trans->reset(new HttpCache::Transaction(this));
   return OK;
 }
 
 HttpCache* HttpCache::GetCache() {
@@ skipping 657 matching lines @@
     building_backend_ = false;
     DeletePendingOp(pending_op);
   }
 
   // The cache may be gone when we return from the callback.
   if (!item->DoCallback(result, backend))
     item->NotifyTransaction(result, NULL);
 }
 
 }  // namespace net