[ServiceWorkerCache] Put should replace an existing entry to meet spec.

content/browser/service_worker/service_worker_cache_unittest.cc

Index: content/browser/service_worker/service_worker_cache_unittest.cc
diff --git a/content/browser/service_worker/service_worker_cache_unittest.cc b/content/browser/service_worker/service_worker_cache_unittest.cc
index b1cd1d93b6b6bb4207a0f154fef1782faef026c1..e7176b24a52c31d8030cb60e5033dd51fadb4faf 100644
--- a/content/browser/service_worker/service_worker_cache_unittest.cc
+++ b/content/browser/service_worker/service_worker_cache_unittest.cc
@@ -328,6 +328,23 @@ TEST_F(ServiceWorkerCacheTest, PutBodyDropBlobRef) {
   EXPECT_EQ(ServiceWorkerCache::ErrorTypeOK, callback_error_);
 }
 
+TEST_P(ServiceWorkerCacheTestP, PutReplace) {
+  EXPECT_TRUE(Put(body_request_, body_response_));
+  EXPECT_TRUE(Match(body_request_));
+  EXPECT_STREQ("http://example.com/body.html",
+               callback_response_->url.spec().c_str());
+
+  EXPECT_TRUE(Put(no_body_request_, no_body_response_));
+  EXPECT_TRUE(Match(no_body_request_));
+  EXPECT_STREQ("http://example.com/no_body.html",
+               callback_response_->url.spec().c_str());
+
+  EXPECT_TRUE(Put(body_request_, body_response_));
+  EXPECT_TRUE(Match(body_request_));
+  EXPECT_STREQ("http://example.com/body.html",
+               callback_response_->url.spec().c_str());

[falken, 2014/10/27 04:09:45]

I'm a bit confused how this tests the new behavior. I'd expect you Put things
with the same request and different responses to see it was replaced?

[jkarlin, 2014/10/27 12:15:28]

Done. Thank you for catching this!

+}
+
 TEST_P(ServiceWorkerCacheTestP, MatchNoBody) {
   EXPECT_TRUE(Put(no_body_request_, no_body_response_));
   EXPECT_TRUE(Match(no_body_request_));