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

Side by Side Diff: content/browser/frame_host/render_frame_host_impl.cc

Issue 574403002: If a RemoteFrame asks to navigate, send an OpenURL IPC (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: +plumbing and test Created 6 years, 3 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/browser/frame_host/render_frame_host_impl.h" 5 #include "content/browser/frame_host/render_frame_host_impl.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/command_line.h" 8 #include "base/command_line.h"
9 #include "base/containers/hash_tables.h" 9 #include "base/containers/hash_tables.h"
10 #include "base/lazy_instance.h" 10 #include "base/lazy_instance.h"
(...skipping 1190 matching lines...) Expand 10 before | Expand all | Expand 10 after
1201 params.pending_history_list_offset = -1; 1201 params.pending_history_list_offset = -1;
1202 params.current_history_list_offset = -1; 1202 params.current_history_list_offset = -1;
1203 params.current_history_list_length = 0; 1203 params.current_history_list_length = 0;
1204 params.url = url; 1204 params.url = url;
1205 params.transition = PAGE_TRANSITION_LINK; 1205 params.transition = PAGE_TRANSITION_LINK;
1206 params.navigation_type = FrameMsg_Navigate_Type::NORMAL; 1206 params.navigation_type = FrameMsg_Navigate_Type::NORMAL;
1207 params.browser_navigation_start = base::TimeTicks::Now(); 1207 params.browser_navigation_start = base::TimeTicks::Now();
1208 Navigate(params); 1208 Navigate(params);
1209 } 1209 }
1210 1210
1211 void RenderFrameHostImpl::OpenURL(const FrameHostMsg_OpenURL_Params& params) {
1212 OnOpenURL(params);
1213 }
1214
1211 void RenderFrameHostImpl::Stop() { 1215 void RenderFrameHostImpl::Stop() {
1212 Send(new FrameMsg_Stop(routing_id_)); 1216 Send(new FrameMsg_Stop(routing_id_));
1213 } 1217 }
1214 1218
1215 void RenderFrameHostImpl::DispatchBeforeUnload(bool for_cross_site_transition) { 1219 void RenderFrameHostImpl::DispatchBeforeUnload(bool for_cross_site_transition) {
1216 TRACE_EVENT_ASYNC_BEGIN0( 1220 TRACE_EVENT_ASYNC_BEGIN0(
1217 "navigation", "RenderFrameHostImpl::BeforeUnload", this); 1221 "navigation", "RenderFrameHostImpl::BeforeUnload", this);
1218 // TODO(creis): Support subframes. 1222 // TODO(creis): Support subframes.
1219 if (!render_view_host_->IsRenderViewLive() || GetParent()) { 1223 if (!render_view_host_->IsRenderViewLive() || GetParent()) {
1220 // We don't have a live renderer, so just skip running beforeunload. 1224 // We don't have a live renderer, so just skip running beforeunload.
(...skipping 232 matching lines...) Expand 10 before | Expand all | Expand 10 after
1453 // Clear any state if a pending navigation is canceled or preempted. 1457 // Clear any state if a pending navigation is canceled or preempted.
1454 if (suspended_nav_params_) 1458 if (suspended_nav_params_)
1455 suspended_nav_params_.reset(); 1459 suspended_nav_params_.reset();
1456 1460
1457 TRACE_EVENT_ASYNC_END0("navigation", 1461 TRACE_EVENT_ASYNC_END0("navigation",
1458 "RenderFrameHostImpl navigation suspended", this); 1462 "RenderFrameHostImpl navigation suspended", this);
1459 navigations_suspended_ = false; 1463 navigations_suspended_ = false;
1460 } 1464 }
1461 1465
1462 } // namespace content 1466 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698