Add a quic_in_memory_cache_test data file.

net/tools/quic/quic_in_memory_cache.cc

 // Copyright (c) 2012 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/tools/quic/quic_in_memory_cache.h"
 
 #include "base/file_util.h"
 #include "base/files/file_enumerator.h"
 #include "base/stl_util.h"
 
@@ skipping 68 matching lines @@
 };
 
 }  // namespace
 
 QuicInMemoryCache* QuicInMemoryCache::GetInstance() {
   return Singleton<QuicInMemoryCache>::get();
 }
 
 const QuicInMemoryCache::Response* QuicInMemoryCache::GetResponse(
     const BalsaHeaders& request_headers) const {
-  string key = GetKey(request_headers);
-  StringPiece url(key);
-  // Removing the leading https:// or http://.
-  if (StringPieceUtils::StartsWithIgnoreCase(url, "https://")) {
-    url.remove_prefix(8);
-  } else if (StringPieceUtils::StartsWithIgnoreCase(url, "http://")) {
-    url.remove_prefix(7);
-  }
-
-  ResponseMap::const_iterator it = responses_.find(url.as_string());
+  ResponseMap::const_iterator it = responses_.find(GetKey(request_headers));
   if (it == responses_.end()) {
     return NULL;
   }
   return it->second;
 }
 
 void QuicInMemoryCache::AddResponse(const BalsaHeaders& request_headers,
                                     const BalsaHeaders& response_headers,
                                     StringPiece response_body) {
   LOG(INFO) << "Adding response for: " << GetKey(request_headers);
   if (ContainsKey(responses_, GetKey(request_headers))) {
     LOG(DFATAL) << "Response for given request already exists!";
   }
   Response* new_response = new Response();
   new_response->set_headers(response_headers);
   new_response->set_body(response_body);
   responses_[GetKey(request_headers)] = new_response;
 }
 
 void QuicInMemoryCache::ResetForTests() {
   STLDeleteValues(&responses_);

[Ryan Hamilton, 2013/07/04 14:46:03]

I think you should call Initialize here, as well.

[honghaiz, 2013/07/04 17:34:04]

Done.

 }
 
 QuicInMemoryCache::QuicInMemoryCache() {
+  Initialize();
+}
+
+void QuicInMemoryCache::Initialize() {
   // If there's no defined cache dir, we have no initialization to do.
   if (FLAGS_quic_in_memory_cache_dir.empty()) {
     LOG(WARNING) << "No cache directory found. Skipping initialization.";
     return;
   }
   LOG(INFO) << "Attempting to initialize QuicInMemoryCache from directory: "
             << FLAGS_quic_in_memory_cache_dir;
 
   FilePath directory(FLAGS_quic_in_memory_cache_dir);
   base::FileEnumerator file_list(directory,
@@ skipping 67 matching lines @@
 
     file = file_list.Next();
   }
 }
 
 QuicInMemoryCache::~QuicInMemoryCache() {
   STLDeleteValues(&responses_);
 }
 
 string QuicInMemoryCache::GetKey(const BalsaHeaders& request_headers) const {
-  return request_headers.GetHeader("host").as_string() +
-      request_headers.request_uri().as_string();
+  StringPiece uri = request_headers.request_uri();
+  if (StringPieceUtils::StartsWithIgnoreCase(uri, "https://")) {
+    uri.remove_prefix(8);
+    return uri.as_string();
+  } else if (StringPieceUtils::StartsWithIgnoreCase(uri, "http://")) {
+    uri.remove_prefix(7);
+    return uri.as_string();
+  }
+  return request_headers.GetHeader("host").as_string() + uri.as_string();

[Ryan Hamilton, 2013/07/04 14:46:03]

So if the host header is set and the URI is fully qualified this will do the
wrong thing, right?

[honghaiz, 2013/07/04 17:34:04]

Fixed. 
On 2013/07/04 14:46:03, Ryan Hamilton wrote:

 }
 
 }  // namespace tools
 }  // namespace net