Componentize SafeBrowsingBlockingPage for WebView use

components/security_interstitials/content/security_interstitial_page.h

Index: components/security_interstitials/content/security_interstitial_page.h
diff --git a/components/security_interstitials/content/security_interstitial_page.h b/components/security_interstitials/content/security_interstitial_page.h
index 4554e5e41a3e9c547d54c42652a3abc9fa6d16ff..7c5f0a48599736524e3321bd95cc527b7b771509 100644
--- a/components/security_interstitials/content/security_interstitial_page.h
+++ b/components/security_interstitials/content/security_interstitial_page.h
@@ -27,9 +27,11 @@ class SecurityInterstitialControllerClient;
 
 class SecurityInterstitialPage : public content::InterstitialPageDelegate {
  public:
+  // |request_url| refers to the URL that triggers the interstitial page. it
+  // can be a main frame URL or subresource URL.

[meacer, 2017/01/11 01:33:31]

The comment might not be accurate for non-SafeBrowsing interstitials. For
example, there is no subresource URL for SSL interstitials. Not sure how to make
the comment apply for both types of interstitials. How about:

"|request_url| is the URL which triggered the interstitial page. For
SafeBrowsing interstitials, it can be a main frame or a subresource URL. For SSL
interstitials, it's always the subresource URL"

[Jialiu Lin, 2017/01/11 02:53:16]

much better. Thanks!
(I think you meant "For SSL interstitials, it's always the main frame url.")

   SecurityInterstitialPage(
       content::WebContents* web_contents,
-      const GURL& url,
+      const GURL& request_url,
       std::unique_ptr<SecurityInterstitialControllerClient> controller);
   ~SecurityInterstitialPage() override;