Remove redundant WebLocalFrame* parameter from didCreateNewDocument

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..44d1f38659bd3eb3d92385b9a9c3a7c86e474cbf 100644
--- a/content/renderer/render_frame_impl.cc
+++ b/content/renderer/render_frame_impl.cc
@@ -3780,13 +3780,11 @@ void RenderFrameImpl::didCommitProvisionalLoad(
   UpdateEncoding(frame, frame->view()->pageEncoding().utf8());
 }
 
-void RenderFrameImpl::didCreateNewDocument(blink::WebLocalFrame* frame) {
-  DCHECK(!frame_ || frame_ == frame);

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

maybe keep DCHECK(frame_) for a while?

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

on second thought, please first verify a while with DCHECK_EQ(frame_, frame).
You deref frame_ below and if it can be nullptr while frame is not, this change
is not safe (so not lgtm for now)

-
+void RenderFrameImpl::didCreateNewDocument() {
   for (auto& observer : observers_)
     observer.DidCreateNewDocument();
   for (auto& observer : render_view_->observers())
-    observer.DidCreateNewDocument(frame);
+    observer.DidCreateNewDocument(frame_);
 }
 
 void RenderFrameImpl::didClearWindowObject(blink::WebLocalFrame* frame) {