Reflect the status of a request for accessing a blacklisted url.

content/public/browser/interstitial_page.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_PUBLIC_BROWSER_INTERSTITIAL_PAGE_H_
 #define CONTENT_PUBLIC_BROWSER_INTERSTITIAL_PAGE_H_
 
 #include "content/common/content_export.h"
 
 class GURL;
@@ skipping 61 matching lines @@
   // the target URL.
   // Warning: 'this' has been deleted when this method returns.
   virtual void Proceed() = 0;
 
   // Sizes the RenderViewHost showing the actual interstitial page contents.
   virtual void SetSize(const gfx::Size& size) = 0;
 
   // Sets the focus to the interstitial.
   virtual void Focus() = 0;
 
-  virtual RenderViewHost* GetRenderViewHostForTesting() const = 0;
+  // Gets the RenderViewHost associated with the Interstitial Page.
+  // Used for routing gestures to the interstitial in Android as it shares a

[Bernhard Bauer, 2014/12/22 14:50:05]

Nit: I would probably rephrase this slightly, to not create the impression that
this is *only* used on Android. Or maybe we can even delete that sentence
completely, now that this method is not Android-only anymore.

[knn, 2015/01/07 23:50:29]

Done.

+  // single platform window for all tabs.
+  virtual RenderViewHost* GetRenderViewHost() const = 0;

[Charlie Reis, 2015/01/07 20:55:38]

Can you expose RenderFrameHost* GetMainFrame() instead?  (That would be
frame_tree_.root()->current_frame_host() inside InterstitialPageImpl.)

The RenderViewHost class is going away as part of the out-of-process iframes /
Site Isolation work, and we shouldn't be adding new dependencies on it.  It
looks like you just need the main frame anyway.

[knn, 2015/01/07 23:50:29]

Done.

+
   virtual InterstitialPageDelegate* GetDelegateForTesting() = 0;
   virtual void DontCreateViewForTesting() = 0;
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_INTERSTITIAL_PAGE_H_