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

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

Issue 2411693003: Move blocking of top-level navigations to nested URLs with extension origins from non-extension pro… (Closed)
Patch Set: Created 4 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/interstitial_page_navigator_impl.h" 5 #include "content/browser/frame_host/interstitial_page_navigator_impl.h"
6 6
7 #include "content/browser/frame_host/interstitial_page_impl.h" 7 #include "content/browser/frame_host/interstitial_page_impl.h"
8 #include "content/browser/frame_host/navigation_controller_impl.h" 8 #include "content/browser/frame_host/navigation_controller_impl.h"
9 #include "content/browser/frame_host/navigator_delegate.h" 9 #include "content/browser/frame_host/navigator_delegate.h"
10 #include "content/browser/renderer_host/render_view_host_impl.h" 10 #include "content/browser/renderer_host/render_view_host_impl.h"
(...skipping 16 matching lines...) Expand all
27 return controller_; 27 return controller_;
28 } 28 }
29 29
30 void InterstitialPageNavigatorImpl::DidStartProvisionalLoad( 30 void InterstitialPageNavigatorImpl::DidStartProvisionalLoad(
31 RenderFrameHostImpl* render_frame_host, 31 RenderFrameHostImpl* render_frame_host,
32 const GURL& url, 32 const GURL& url,
33 const base::TimeTicks& navigation_start) { 33 const base::TimeTicks& navigation_start) {
34 // The interstitial page should only navigate once. 34 // The interstitial page should only navigate once.
35 DCHECK(!navigation_handle_.get()); 35 DCHECK(!navigation_handle_.get());
36 navigation_handle_ = 36 navigation_handle_ =
37 NavigationHandleImpl::Create(url, render_frame_host->frame_tree_node(), 37 NavigationHandleImpl::Create(url,
38 false, // is_renderer_initiated 38 render_frame_host->GetSiteInstance()->GetSiteURL(),
39 false, // is_synchronous 39 render_frame_host->frame_tree_node(),
40 false, // is_srcdoc 40 false, // is_renderer_initiated
41 navigation_start, 41 false, // is_synchronous
42 0, // pending_nav_entry_id 42 false, // is_srcdoc
43 false); // started_in_context_menu 43 navigation_start,
44 0, // pending_nav_entry_id
45 false); // started_in_context_menu
44 } 46 }
45 47
46 void InterstitialPageNavigatorImpl::DidNavigate( 48 void InterstitialPageNavigatorImpl::DidNavigate(
47 RenderFrameHostImpl* render_frame_host, 49 RenderFrameHostImpl* render_frame_host,
48 const FrameHostMsg_DidCommitProvisionalLoad_Params& input_params) { 50 const FrameHostMsg_DidCommitProvisionalLoad_Params& input_params) {
49 if (navigation_handle_) { 51 if (navigation_handle_) {
50 navigation_handle_->DidCommitNavigation(input_params, false, 52 navigation_handle_->DidCommitNavigation(input_params, false,
51 render_frame_host); 53 render_frame_host);
52 navigation_handle_.reset(); 54 navigation_handle_.reset();
53 } 55 }
54 56
55 // TODO(nasko): Move implementation here, but for the time being call out 57 // TODO(nasko): Move implementation here, but for the time being call out
56 // to the interstitial page code. 58 // to the interstitial page code.
57 interstitial_->DidNavigate(render_frame_host->render_view_host(), 59 interstitial_->DidNavigate(render_frame_host->render_view_host(),
58 input_params); 60 input_params);
59 } 61 }
60 62
61 NavigationHandleImpl* 63 NavigationHandleImpl*
62 InterstitialPageNavigatorImpl::GetNavigationHandleForFrameHost( 64 InterstitialPageNavigatorImpl::GetNavigationHandleForFrameHost(
63 RenderFrameHostImpl* render_frame_host) { 65 RenderFrameHostImpl* render_frame_host) {
64 return navigation_handle_.get(); 66 return navigation_handle_.get();
65 } 67 }
66 68
67 } // namespace content 69 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698