PlzNavigate: Release StreamHandle.

content/child/web_url_loader_impl.h

 // Copyright 2014 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.
 
 #ifndef CONTENT_CHILD_WEB_URL_LOADER_IMPL_H_
 #define CONTENT_CHILD_WEB_URL_LOADER_IMPL_H_
 
+#include "base/callback_forward.h"
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
 #include "content/common/content_export.h"
 #include "content/common/url_loader_factory.mojom.h"
 #include "content/public/common/resource_response.h"
 #include "mojo/public/cpp/system/data_pipe.h"
 #include "net/url_request/redirect_info.h"
 #include "third_party/WebKit/public/platform/WebURLLoader.h"
 #include "url/gurl.h"
 
 namespace content {
 
 class ResourceDispatcher;
 struct ResourceResponseInfo;
 
 // PlzNavigate: Used to override parameters of the navigation request.
 struct CONTENT_EXPORT StreamOverrideParameters {
  public:
   StreamOverrideParameters();
   ~StreamOverrideParameters();
-  // TODO(clamy): The browser should be made aware on destruction of this struct
-  // that it can release its associated stream handle.
+
   GURL stream_url;
   mojo::ScopedDataPipeConsumerHandle consumer_handle;
   ResourceResponseHead response;
   std::vector<GURL> redirects;
   std::vector<ResourceResponseInfo> redirect_responses;
   std::vector<net::RedirectInfo> redirect_infos;
 
   // The delta between the actual transfer size and the one reported by the
   // AsyncResourceLoader due to not having the ResourceResponse.
   int total_transfer_size_delta;
 
   int total_transferred = 0;
+
+  // Called when this struct is deleted. Used to notify the browser that it can
+  // release its associated StreamHandle.
+  base::OnceCallback<void(const GURL&)> on_delete;
 };
 
 class CONTENT_EXPORT WebURLLoaderImpl
     : public NON_EXPORTED_BASE(blink::WebURLLoader) {
  public:
 
   // Takes ownership of |web_task_runner|.
   WebURLLoaderImpl(ResourceDispatcher* resource_dispatcher,
                    mojom::URLLoaderFactory* url_loader_factory);
   ~WebURLLoaderImpl() override;
@@ skipping 27 matching lines @@
   class Context;
   class RequestPeerImpl;
   scoped_refptr<Context> context_;
 
   DISALLOW_COPY_AND_ASSIGN(WebURLLoaderImpl);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_CHILD_WEB_URL_LOADER_IMPL_H_