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

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

Issue 2838503002: Remove redundant WebLocalFrame* parameter from WillReleaseScriptContext (Closed)
Patch Set: Rebase Created 3 years, 7 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 4541 matching lines...) Expand 10 before | Expand all | Expand 10 after
4552 4552
4553 void RenderFrameImpl::DidCreateScriptContext(blink::WebLocalFrame* frame, 4553 void RenderFrameImpl::DidCreateScriptContext(blink::WebLocalFrame* frame,
4554 v8::Local<v8::Context> context, 4554 v8::Local<v8::Context> context,
4555 int world_id) { 4555 int world_id) {
4556 DCHECK_EQ(frame_, frame); 4556 DCHECK_EQ(frame_, frame);
4557 4557
4558 for (auto& observer : observers_) 4558 for (auto& observer : observers_)
4559 observer.DidCreateScriptContext(context, world_id); 4559 observer.DidCreateScriptContext(context, world_id);
4560 } 4560 }
4561 4561
4562 void RenderFrameImpl::WillReleaseScriptContext(blink::WebLocalFrame* frame, 4562 void RenderFrameImpl::WillReleaseScriptContext(v8::Local<v8::Context> context,
4563 v8::Local<v8::Context> context,
4564 int world_id) { 4563 int world_id) {
4565 DCHECK_EQ(frame_, frame);
4566
4567 for (auto& observer : observers_) 4564 for (auto& observer : observers_)
4568 observer.WillReleaseScriptContext(context, world_id); 4565 observer.WillReleaseScriptContext(context, world_id);
4569 } 4566 }
4570 4567
4571 void RenderFrameImpl::DidChangeScrollOffset() { 4568 void RenderFrameImpl::DidChangeScrollOffset() {
4572 render_view_->StartNavStateSyncTimerIfNecessary(this); 4569 render_view_->StartNavStateSyncTimerIfNecessary(this);
4573 4570
4574 for (auto& observer : observers_) 4571 for (auto& observer : observers_)
4575 observer.DidChangeScrollOffset(); 4572 observer.DidChangeScrollOffset();
4576 } 4573 }
(...skipping 2420 matching lines...) Expand 10 before | Expand all | Expand 10 after
6997 policy(info.default_policy), 6994 policy(info.default_policy),
6998 replaces_current_history_item(info.replaces_current_history_item), 6995 replaces_current_history_item(info.replaces_current_history_item),
6999 history_navigation_in_new_child_frame( 6996 history_navigation_in_new_child_frame(
7000 info.is_history_navigation_in_new_child_frame), 6997 info.is_history_navigation_in_new_child_frame),
7001 client_redirect(info.is_client_redirect), 6998 client_redirect(info.is_client_redirect),
7002 cache_disabled(info.is_cache_disabled), 6999 cache_disabled(info.is_cache_disabled),
7003 form(info.form), 7000 form(info.form),
7004 source_location(info.source_location) {} 7001 source_location(info.source_location) {}
7005 7002
7006 } // namespace content 7003 } // 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