Switch RenderViewContextMenu to use RequestOpenURL

content/browser/frame_host/interstitial_page_impl.cc

Index: content/browser/frame_host/interstitial_page_impl.cc
diff --git a/content/browser/frame_host/interstitial_page_impl.cc b/content/browser/frame_host/interstitial_page_impl.cc
index c3b0abb150231ff7bde25a8a1a4165a1c402f724..b3a7a0b42e15c9cd6291fbba70d002d4d67bdbe8 100644
--- a/content/browser/frame_host/interstitial_page_impl.cc
+++ b/content/browser/frame_host/interstitial_page_impl.cc
@@ -526,6 +526,10 @@ void InterstitialPageImpl::DidNavigate(
   }
 }
 
+WebContents* InterstitialPageImpl::OpenURL(const OpenURLParams& params) {
+  return nullptr;

[nasko, 2017/02/14 00:10:23]

Let's add a NOTREACHED to ensure we don't accidentally hit that.

[Patrick Noland, 2017/02/14 01:16:16]

Done.

+}
+
 RendererPreferences InterstitialPageImpl::GetRendererPrefs(
     BrowserContext* browser_context) const {
   delegate_->OverrideRendererPrefs(&renderer_preferences_);