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

Side by Side Diff: content/browser/frame_host/navigator_delegate.h

Issue 2377833002: Block navigations in extensions via ShouldTransferNavigation (not via OpenURL). (Closed)
Patch Set: Add missing files under chrome/test/data/extensions/api_test/browser_action/popup_with_form/ 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 #ifndef CONTENT_BROWSER_FRAME_HOST_NAVIGATOR_DELEGATE_H_ 5 #ifndef CONTENT_BROWSER_FRAME_HOST_NAVIGATOR_DELEGATE_H_
6 #define CONTENT_BROWSER_FRAME_HOST_NAVIGATOR_DELEGATE_H_ 6 #define CONTENT_BROWSER_FRAME_HOST_NAVIGATOR_DELEGATE_H_
7 7
8 #include "base/memory/scoped_vector.h" 8 #include "base/memory/scoped_vector.h"
9 #include "base/strings/string16.h" 9 #include "base/strings/string16.h"
10 #include "content/public/browser/invalidate_type.h" 10 #include "content/public/browser/invalidate_type.h"
(...skipping 97 matching lines...) Expand 10 before | Expand all | Expand 10 after
108 virtual void DidStartNavigationToPendingEntry(const GURL& url, 108 virtual void DidStartNavigationToPendingEntry(const GURL& url,
109 ReloadType reload_type) {} 109 ReloadType reload_type) {}
110 110
111 // Opens a URL with the given parameters. See PageNavigator::OpenURL, which 111 // Opens a URL with the given parameters. See PageNavigator::OpenURL, which
112 // this forwards to. 112 // this forwards to.
113 virtual void RequestOpenURL(RenderFrameHostImpl* render_frame_host, 113 virtual void RequestOpenURL(RenderFrameHostImpl* render_frame_host,
114 const OpenURLParams& params) {} 114 const OpenURLParams& params) {}
115 115
116 // Returns whether to continue a navigation that needs to transfer to a 116 // Returns whether to continue a navigation that needs to transfer to a
117 // different process between the load start and commit. 117 // different process between the load start and commit.
118 virtual bool ShouldTransferNavigation(); 118 virtual bool ShouldTransferNavigation(bool is_main_frame_navigation);
119 119
120 // Returns whether URLs for aborted browser-initiated navigations should be 120 // Returns whether URLs for aborted browser-initiated navigations should be
121 // preserved in the omnibox. Defaults to false. 121 // preserved in the omnibox. Defaults to false.
122 virtual bool ShouldPreserveAbortedURLs(); 122 virtual bool ShouldPreserveAbortedURLs();
123 123
124 // A RenderFrameHost in the specified |frame_tree_node| started loading a new 124 // A RenderFrameHost in the specified |frame_tree_node| started loading a new
125 // document. This correponds to Blink's notion of the throbber starting. 125 // document. This correponds to Blink's notion of the throbber starting.
126 // |to_different_document| will be true unless the load is a fragment 126 // |to_different_document| will be true unless the load is a fragment
127 // navigation, or triggered by history.pushState/replaceState. 127 // navigation, or triggered by history.pushState/replaceState.
128 virtual void DidStartLoading(FrameTreeNode* frame_tree_node, 128 virtual void DidStartLoading(FrameTreeNode* frame_tree_node,
129 bool to_different_document) {} 129 bool to_different_document) {}
130 130
131 // A document stopped loading. This corresponds to Blink's notion of the 131 // A document stopped loading. This corresponds to Blink's notion of the
132 // throbber stopping. 132 // throbber stopping.
133 virtual void DidStopLoading() {} 133 virtual void DidStopLoading() {}
134 134
135 // The load progress was changed. 135 // The load progress was changed.
136 virtual void DidChangeLoadProgress() {} 136 virtual void DidChangeLoadProgress() {}
137 137
138 // Returns the NavigationThrottles to add to this navigation. Normally these 138 // Returns the NavigationThrottles to add to this navigation. Normally these
139 // are defined by the content/ embedder, except in the case of interstitials 139 // are defined by the content/ embedder, except in the case of interstitials
140 // where no NavigationThrottles are added to the navigation. 140 // where no NavigationThrottles are added to the navigation.
141 virtual ScopedVector<NavigationThrottle> CreateThrottlesForNavigation( 141 virtual ScopedVector<NavigationThrottle> CreateThrottlesForNavigation(
142 NavigationHandle* navigation_handle); 142 NavigationHandle* navigation_handle);
143 }; 143 };
144 144
145 } // namspace content 145 } // namspace content
146 146
147 #endif // CONTENT_BROWSER_FRAME_HOST_NAVIGATOR_DELEGATE_H_ 147 #endif // CONTENT_BROWSER_FRAME_HOST_NAVIGATOR_DELEGATE_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698