Cleanup ServiceWorkerCacheWriter

Service worker code base no longer use net::HttpResponseInfo directly.
Update comments and remove an unused method.

Bug: N/A

Change-Id: I151b758fe0555af7c6a204e15c440a6ca2a8e038
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2537500Reviewed-by: Matt Falkenhagen <falken@chromium.org>
Commit-Queue: Kenichi Ishibashi <bashi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#827216}

content/browser/service_worker/service_worker_cache_writer.cc

@@ -65,9 +65,9 @@ class ServiceWorkerCacheWriter::ReadResponseHeadCallbackAdapter
     DCHECK(owner_);
   }
 
-  void DidReadResponseInfo(int result,
+  void DidReadResponseHead(int result,
                            network::mojom::URLResponseHeadPtr response_head,
-                           scoped_refptr<net::IOBufferWithSize> /*metadata*/) {
+                           base::Optional<mojo_base::BigBuffer>) {
     result_ = result;
     if (!owner_)
       return;
@@ -76,12 +76,6 @@ class ServiceWorkerCacheWriter::ReadResponseHeadCallbackAdapter
       owner_->AsyncDoLoop(result);
   }
 
-  void DidReadResponseHead(int result,
-                           network::mojom::URLResponseHeadPtr response_head,
-                           base::Optional<mojo_base::BigBuffer>) {
-    DidReadResponseInfo(result, std::move(response_head), nullptr);
-  }
-
   void SetAsync() { async_ = true; }
   int result() { return result_; }
 

content/browser/service_worker/service_worker_cache_writer.h

@@ -39,7 +39,7 @@ class CONTENT_EXPORT ServiceWorkerCacheWriter {
   using OnWriteCompleteCallback = base::OnceCallback<void(net::Error)>;
 
   // This class defines the interfaces of observer that observes write
-  // operations. The observer is notified when response info or data
+  // operations. The observer is notified when response head or data
   // will be written to storage.
   class WriteObserver {
    public:
@@ -258,7 +258,7 @@ class CONTENT_EXPORT ServiceWorkerCacheWriter {
   // WriteResponseHead() operates the same as
   // WriteResponseHeadToResponseWriter() and WriteData() operates the same as
   // WriteDataToResponseWriter().
-  // If observer is set, the argument |response_info| or |data| is first sent
+  // If observer is set, the argument |response_head| or |data| is first sent
   // to observer then WriteResponseHeadToResponseWriter() or
   // WriteDataToResponseWriter() is called.
   int WriteResponseHead(network::mojom::URLResponseHeadPtr response_head);

content/browser/service_worker/service_worker_cache_writer_unittest.cc

@@ -22,7 +22,7 @@ namespace content {
 namespace {
 
 // A test implementation of ServiceWorkerCacheWriter::WriteObserver.
-// This observer captures the response info or data sent to the observer
+// This observer captures the response head or data sent to the observer
 // for further checking.
 class MockServiceWorkerCacheWriterObserver
     : public ServiceWorkerCacheWriter::WriteObserver {