Remove redundant WebLocalFrame* parameter from didCreateDataSource

content/renderer/render_frame_impl.cc

Index: content/renderer/render_frame_impl.cc
diff --git a/content/renderer/render_frame_impl.cc b/content/renderer/render_frame_impl.cc
index 679773f82aca83ced3397903a82c2e4dd7c899e6..a1df88dd227e830c8fb631501eb3b10a5d19e538 100644
--- a/content/renderer/render_frame_impl.cc
+++ b/content/renderer/render_frame_impl.cc
@@ -3384,10 +3384,7 @@ void RenderFrameImpl::willSubmitForm(const blink::WebFormElement& form) {
     observer.WillSubmitForm(form);
 }
 
-void RenderFrameImpl::didCreateDataSource(blink::WebLocalFrame* frame,
-                                          blink::WebDataSource* datasource) {
-  DCHECK(!frame_ || frame_ == frame);

[jochen (gone - plz use gerrit), 2017/03/30 11:43:43]

this doesn't look like it's good enough to proof redundancy (and I think the
crashes you get on the bots are due to this)

-
+void RenderFrameImpl::didCreateDataSource(blink::WebDataSource* datasource) {
   bool content_initiated = !pending_navigation_params_.get();
 
   // Make sure any previous redirect URLs end up in our new data source.
@@ -3479,7 +3476,7 @@ void RenderFrameImpl::didCreateDataSource(blink::WebLocalFrame* frame,
 
   datasource->setServiceWorkerNetworkProvider(
       ServiceWorkerNetworkProvider::CreateForNavigation(
-          routing_id_, navigation_state->request_params(), frame,
+          routing_id_, navigation_state->request_params(), frame_,
           content_initiated));
 }