http-cache: remove double-store of resources

servo · bors-servo · Jun 22, 2019 · Jun 1, 2019 · Jun 19, 2019 · Jun 20, 2019
commit 049817c5a71c0d0a05badba85fbdde8db3b070fa
diff --git a/components/net/http_loader.rs b/components/net/http_loader.rs
@@ -1382,12 +1382,7 @@ fn http_network_fetch(
     // Step 13
     // TODO this step isn't possible yet (CSP)
 
-    // Step 14
-    if !response.is_network_error() && request.cache_mode != CacheMode::NoStore {
-        if let Ok(mut http_cache) = context.state.http_cache.write() {
-            http_cache.store(&request, &response);
-        }
-    }
+    // Step 14, update the cached response, done via the shared response body.
 
     // TODO this step isn't possible yet
     // Step 15