Remove redundant WebLocalFrame* parameter from runScriptsAtDocumentReady

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 3882 matching lines @@
 
   for (auto& observer : render_view_->observers())
     observer.DidFinishDocumentLoad(frame);
   for (auto& observer : observers_)
     observer.DidFinishDocumentLoad();
 
   // Check whether we have new encoding name.
   UpdateEncoding(frame, frame->view()->pageEncoding().utf8());
 }
 
-void RenderFrameImpl::runScriptsAtDocumentReady(blink::WebLocalFrame* frame,
-                                                bool document_is_empty) {
-  DCHECK_EQ(frame_, frame);
+void RenderFrameImpl::runScriptsAtDocumentReady(bool document_is_empty) {
   base::WeakPtr<RenderFrameImpl> weak_self = weak_factory_.GetWeakPtr();
 
   MojoBindingsController* mojo_bindings_controller =
       MojoBindingsController::Get(this);
   if (mojo_bindings_controller)
     mojo_bindings_controller->RunScriptsAtDocumentReady();
 
   if (!weak_self.get())
     return;
 
   GetContentClient()->renderer()->RunScriptsAtDocumentEnd(this);
 
-  // ContentClient might have deleted |frame| and |this| by now!
+  // ContentClient might have deleted |frame_| and |this| by now!
   if (!weak_self.get())
     return;
 
   // If this is an empty document with an http status code indicating an error,
   // we may want to display our own error page, so the user doesn't end up
   // with an unexplained blank page.
   if (!document_is_empty)
     return;
 
   // Do not show error page when DevTools is attached.
   const RenderFrameImpl* localRoot = GetLocalRoot();
   if (localRoot->devtools_agent_ && localRoot->devtools_agent_->IsAttached())
     return;
 
   // Display error page instead of a blank page, if appropriate.
   std::string error_domain = "http";
   InternalDocumentStateData* internal_data =
-      InternalDocumentStateData::FromDataSource(frame->dataSource());
+      InternalDocumentStateData::FromDataSource(frame_->dataSource());
   int http_status_code = internal_data->http_status_code();
   if (GetContentClient()->renderer()->HasErrorPage(http_status_code,
                                                    &error_domain)) {
     WebURLError error;
-    error.unreachableURL = frame->document().url();
+    error.unreachableURL = frame_->document().url();
     error.domain = WebString::fromUTF8(error_domain);
     error.reason = http_status_code;
     // This call may run scripts, e.g. via the beforeunload event.
-    LoadNavigationErrorPage(frame->dataSource()->getRequest(), error, true,
+    LoadNavigationErrorPage(frame_->dataSource()->getRequest(), error, true,
                             nullptr);
   }
-  // Do not use |this| or |frame| here without checking |weak_self|.
+  // Do not use |this| or |frame_| here without checking |weak_self|.
 }
 
 void RenderFrameImpl::runScriptsAtDocumentIdle(blink::WebLocalFrame* frame) {
   DCHECK_EQ(frame_, frame);
   GetContentClient()->renderer()->RunScriptsAtDocumentIdle(this);
   // ContentClient might have deleted |frame| and |this| by now!
 }
 
 void RenderFrameImpl::didHandleOnloadEvents(blink::WebLocalFrame* frame) {
   DCHECK_EQ(frame_, frame);
@@ skipping 3010 matching lines @@
       policy(info.defaultPolicy),
       replaces_current_history_item(info.replacesCurrentHistoryItem),
       history_navigation_in_new_child_frame(
           info.isHistoryNavigationInNewChildFrame),
       client_redirect(info.isClientRedirect),
       cache_disabled(info.isCacheDisabled),
       form(info.form),
       source_location(info.sourceLocation) {}
 
 }  // namespace content