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

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: Disable new test on android/chrome, like others in the same file Created 6 years, 2 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 1264 matching lines...) Expand 10 before | Expand all | Expand 10 after
1275 params.pending_history_list_offset = -1; 1275 params.pending_history_list_offset = -1;
1276 params.current_history_list_offset = -1; 1276 params.current_history_list_offset = -1;
1277 params.current_history_list_length = 0; 1277 params.current_history_list_length = 0;
1278 params.url = url; 1278 params.url = url;
1279 params.transition = ui::PAGE_TRANSITION_LINK; 1279 params.transition = ui::PAGE_TRANSITION_LINK;
1280 params.navigation_type = FrameMsg_Navigate_Type::NORMAL; 1280 params.navigation_type = FrameMsg_Navigate_Type::NORMAL;
1281 params.browser_navigation_start = base::TimeTicks::Now(); 1281 params.browser_navigation_start = base::TimeTicks::Now();
1282 Navigate(params); 1282 Navigate(params);
1283 } 1283 }
1284 1284
1285 void RenderFrameHostImpl::OpenURL(const FrameHostMsg_OpenURL_Params& params) {
1286 OnOpenURL(params);
1287 }
1288
1285 void RenderFrameHostImpl::Stop() { 1289 void RenderFrameHostImpl::Stop() {
1286 Send(new FrameMsg_Stop(routing_id_)); 1290 Send(new FrameMsg_Stop(routing_id_));
1287 } 1291 }
1288 1292
1289 void RenderFrameHostImpl::DispatchBeforeUnload(bool for_cross_site_transition) { 1293 void RenderFrameHostImpl::DispatchBeforeUnload(bool for_cross_site_transition) {
1290 TRACE_EVENT_ASYNC_BEGIN0( 1294 TRACE_EVENT_ASYNC_BEGIN0(
1291 "navigation", "RenderFrameHostImpl::BeforeUnload", this); 1295 "navigation", "RenderFrameHostImpl::BeforeUnload", this);
1292 // TODO(creis): Support subframes. 1296 // TODO(creis): Support subframes.
1293 if (GetParent() || !IsRenderFrameLive()) { 1297 if (GetParent() || !IsRenderFrameLive()) {
1294 // We don't have a live renderer, so just skip running beforeunload. 1298 // We don't have a live renderer, so just skip running beforeunload.
(...skipping 225 matching lines...) Expand 10 before | Expand all | Expand 10 after
1520 // Clear any state if a pending navigation is canceled or preempted. 1524 // Clear any state if a pending navigation is canceled or preempted.
1521 if (suspended_nav_params_) 1525 if (suspended_nav_params_)
1522 suspended_nav_params_.reset(); 1526 suspended_nav_params_.reset();
1523 1527
1524 TRACE_EVENT_ASYNC_END0("navigation", 1528 TRACE_EVENT_ASYNC_END0("navigation",
1525 "RenderFrameHostImpl navigation suspended", this); 1529 "RenderFrameHostImpl navigation suspended", this);
1526 navigations_suspended_ = false; 1530 navigations_suspended_ = false;
1527 } 1531 }
1528 1532
1529 } // namespace content 1533 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/frame_host/render_frame_host_impl.h ('k') | content/browser/frame_host/render_frame_proxy_host.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698