PlzNavigate: Release StreamHandle.

content/renderer/render_frame_impl.cc

 // Copyright 2013 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.
 
 #include "content/renderer/render_frame_impl.h"
 
 #include <map>
 #include <string>
 #include <utility>
 #include <vector>
@@ skipping 5116 matching lines @@
   std::unique_ptr<StreamOverrideParameters> stream_override(
       new StreamOverrideParameters());
   stream_override->stream_url = stream_url;
   stream_override->consumer_handle =
       mojo::ScopedDataPipeConsumerHandle(commit_data.handle);
   stream_override->response = response;
   stream_override->redirects = request_params.redirects;
   stream_override->redirect_responses = request_params.redirect_response;
   stream_override->redirect_infos = request_params.redirect_infos;
 
+  // PlzNavigate: Notify the browser that it can released its |stream_handle_|

[clamy, 2017/06/12 15:15:47]

nit: no PlzNavigate needed (it's a full PlzNavigate function).
I would prefer the comment as:
// Used to notify the browser that it can release its... object isn't used
anymore.
TODO(clamy): Remove this when we switch to Mojo streams.

[arthursonzogni, 2017/06/13 09:21:48]

Done.

+  // when the |stream_override| object is no more used. This is a temporary
+  // solution until mojo is used.
+  stream_override->on_delete = base::BindOnce(
+      [](base::WeakPtr<RenderFrameImpl> weak_self, const GURL& url) {
+        if (RenderFrameImpl* self = weak_self.get()) {
+          self->Send(
+              new FrameHostMsg_StreamHandleConsumed(self->routing_id_, url));
+        }
+      },
+      weak_factory_.GetWeakPtr());
+
   if (commit_data.url_loader_factory.is_valid()) {
     // Chrome doesn't use interface versioning.
     url_loader_factory_.Bind(mojom::URLLoaderFactoryPtrInfo(
         mojo::ScopedMessagePipeHandle(commit_data.url_loader_factory), 0u));
   }
 
   // If the request was initiated in the context of a user gesture then make
   // sure that the navigation also executes in the context of a user gesture.
   std::unique_ptr<blink::WebScopedUserGesture> gesture(
       request_params.has_user_gesture ? new blink::WebScopedUserGesture(frame_)
@@ skipping 1708 matching lines @@
       policy(info.default_policy),
       replaces_current_history_item(info.replaces_current_history_item),
       history_navigation_in_new_child_frame(
           info.is_history_navigation_in_new_child_frame),
       client_redirect(info.is_client_redirect),
       cache_disabled(info.is_cache_disabled),
       form(info.form),
       source_location(info.source_location) {}
 
 }  // namespace content