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

Unified Diff: content/browser/frame_host/render_frame_host_impl.h

Issue 2469163002: Clean up source_site_instance usage in Navigator::RequestOpenURL. (Closed)
Patch Set: Charlie's comments Created 4 years, 1 month 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « content/browser/frame_host/navigator_impl.cc ('k') | content/browser/frame_host/render_frame_host_impl.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/frame_host/render_frame_host_impl.h
diff --git a/content/browser/frame_host/render_frame_host_impl.h b/content/browser/frame_host/render_frame_host_impl.h
index f0d8d06376d69286e96bfca5db50aa2f39ee9ff3..e3972d6f2c01da04b1fce6300f320978b3e61a12 100644
--- a/content/browser/frame_host/render_frame_host_impl.h
+++ b/content/browser/frame_host/render_frame_host_impl.h
@@ -375,15 +375,6 @@ class CONTENT_EXPORT RenderFrameHostImpl
// Navigates to an interstitial page represented by the provided data URL.
void NavigateToInterstitialURL(const GURL& data_url);
- // Treat this prospective navigation as though it originated from the frame.
- // Used, e.g., for a navigation request that originated from a RemoteFrame.
- // |source_site_instance| is the SiteInstance of the frame that initiated the
- // navigation.
- // TODO(creis): Remove this method and have RenderFrameProxyHost call
- // RequestOpenURL with its FrameTreeNode.
- void OpenURL(const FrameHostMsg_OpenURL_Params& params,
- SiteInstance* source_site_instance);
-
// Stop the load in progress.
void Stop();
« no previous file with comments | « content/browser/frame_host/navigator_impl.cc ('k') | content/browser/frame_host/render_frame_host_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698