Remove the remaining OnResponsesLegacyBlob code

MojoBlob is default and most other code has been already removed.

Bug: 776642
Change-Id: Ibf11da14a3dd46c7f4613d0617e0ee0c3b87b999
Reviewed-on: https://chromium-review.googlesource.com/1114564Reviewed-by: Matt Falkenhagen <falken@chromium.org>
Commit-Queue: Kinuko Yasuda <kinuko@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570725}

content/browser/service_worker/service_worker_fetch_dispatcher.cc

@@ -391,14 +391,6 @@ class ServiceWorkerFetchDispatcher::ResponseCallback
                    nullptr /* body_as_stream */, std::move(body_as_blob),
                    FetchEventResult::kGotResponse, dispatch_event_time);
   }
-  void OnResponseLegacyBlob(const ServiceWorkerResponse& response,
-                            base::Time dispatch_event_time,
-                            OnResponseLegacyBlobCallback callback) override {
-    HandleResponse(fetch_dispatcher_, version_, fetch_event_id_, response,
-                   nullptr /* body_as_stream */, nullptr /* body_as_blob */,
-                   FetchEventResult::kGotResponse, dispatch_event_time);
-    std::move(callback).Run();
-  }
   void OnResponseStream(
       const ServiceWorkerResponse& response,
       blink::mojom::ServiceWorkerStreamHandlePtr body_as_stream,

content/common/service_worker/service_worker_fetch_response_callback.mojom

@@ -25,12 +25,6 @@ interface ServiceWorkerFetchResponseCallback {
   OnResponseBlob(ServiceWorkerResponse response,
                  blink.mojom.Blob body_as_blob,
                  mojo_base.mojom.Time dispatch_event_time);
-   // Responds to the request with |response|. The body is provided as a
-   // non-Mojo Blob via |response.blob_uuid|. The callback is useful for Blob
-   // lifetime management purposes and is called once the caller is done with
-   // the Blob. TODO(kinuko): Remove this once MojoBlob is fully shipped.
-  OnResponseLegacyBlob(ServiceWorkerResponse response,
-                       mojo_base.mojom.Time dispatch_event_time) => ();
   // Responds to the request with |response|. The body is returned as a stream.
   OnResponseStream(ServiceWorkerResponse response,
                    blink.mojom.ServiceWorkerStreamHandle body_as_stream,

content/renderer/service_worker/service_worker_subresource_loader.cc

@@ -333,13 +333,6 @@ void ServiceWorkerSubresourceLoader::OnResponseBlob(
                 nullptr /* body_as_stream */);
 }
 
-void ServiceWorkerSubresourceLoader::OnResponseLegacyBlob(
-    const ServiceWorkerResponse& response,
-    base::Time dispatch_event_time,
-    OnResponseLegacyBlobCallback callback) {
-  NOTREACHED();
-}
-
 void ServiceWorkerSubresourceLoader::OnResponseStream(
     const ServiceWorkerResponse& response,
     blink::mojom::ServiceWorkerStreamHandlePtr body_as_stream,

content/renderer/service_worker/service_worker_subresource_loader.h

@@ -78,9 +78,6 @@ class CONTENT_EXPORT ServiceWorkerSubresourceLoader
   void OnResponseBlob(const ServiceWorkerResponse& response,
                       blink::mojom::BlobPtr blob,
                       base::Time dispatch_event_time) override;
-  void OnResponseLegacyBlob(const ServiceWorkerResponse& response,
-                            base::Time dispatch_event_time,
-                            OnResponseLegacyBlobCallback callback) override;
   void OnResponseStream(
       const ServiceWorkerResponse& response,
       blink::mojom::ServiceWorkerStreamHandlePtr body_as_stream,