Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(396)

Side by Side Diff: content/renderer/render_frame_impl.cc

Issue 2830003002: Remove redundant WebLocalFrame* parameter from DidChangeScrollOffset (Closed)
Patch Set: Rebase Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/renderer/render_frame_impl.h" 5 #include "content/renderer/render_frame_impl.h"
6 6
7 #include <map> 7 #include <map>
8 #include <string> 8 #include <string>
9 #include <utility> 9 #include <utility>
10 #include <vector> 10 #include <vector>
(...skipping 4550 matching lines...) Expand 10 before | Expand all | Expand 10 after
4561 4561
4562 void RenderFrameImpl::WillReleaseScriptContext(blink::WebLocalFrame* frame, 4562 void RenderFrameImpl::WillReleaseScriptContext(blink::WebLocalFrame* frame,
4563 v8::Local<v8::Context> context, 4563 v8::Local<v8::Context> context,
4564 int world_id) { 4564 int world_id) {
4565 DCHECK_EQ(frame_, frame); 4565 DCHECK_EQ(frame_, frame);
4566 4566
4567 for (auto& observer : observers_) 4567 for (auto& observer : observers_)
4568 observer.WillReleaseScriptContext(context, world_id); 4568 observer.WillReleaseScriptContext(context, world_id);
4569 } 4569 }
4570 4570
4571 void RenderFrameImpl::DidChangeScrollOffset(blink::WebLocalFrame* frame) { 4571 void RenderFrameImpl::DidChangeScrollOffset() {
4572 DCHECK_EQ(frame_, frame);
4573 render_view_->StartNavStateSyncTimerIfNecessary(this); 4572 render_view_->StartNavStateSyncTimerIfNecessary(this);
4574 4573
4575 for (auto& observer : observers_) 4574 for (auto& observer : observers_)
4576 observer.DidChangeScrollOffset(); 4575 observer.DidChangeScrollOffset();
4577 } 4576 }
4578 4577
4579 void RenderFrameImpl::WillInsertBody(blink::WebLocalFrame* frame) { 4578 void RenderFrameImpl::WillInsertBody(blink::WebLocalFrame* frame) {
4580 DCHECK(!frame_ || frame_ == frame); 4579 DCHECK(!frame_ || frame_ == frame);
4581 Send(new FrameHostMsg_WillInsertBody(routing_id_, 4580 Send(new FrameHostMsg_WillInsertBody(routing_id_,
4582 render_view_->GetRoutingID())); 4581 render_view_->GetRoutingID()));
(...skipping 2415 matching lines...) Expand 10 before | Expand all | Expand 10 after
6998 policy(info.default_policy), 6997 policy(info.default_policy),
6999 replaces_current_history_item(info.replaces_current_history_item), 6998 replaces_current_history_item(info.replaces_current_history_item),
7000 history_navigation_in_new_child_frame( 6999 history_navigation_in_new_child_frame(
7001 info.is_history_navigation_in_new_child_frame), 7000 info.is_history_navigation_in_new_child_frame),
7002 client_redirect(info.is_client_redirect), 7001 client_redirect(info.is_client_redirect),
7003 cache_disabled(info.is_cache_disabled), 7002 cache_disabled(info.is_cache_disabled),
7004 form(info.form), 7003 form(info.form),
7005 source_location(info.source_location) {} 7004 source_location(info.source_location) {}
7006 7005
7007 } // namespace content 7006 } // namespace content
OLDNEW
« no previous file with comments | « content/renderer/render_frame_impl.h ('k') | third_party/WebKit/Source/web/LocalFrameClientImpl.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698