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

Unified Diff: chrome/browser/safe_browsing/safe_browsing_blocking_page.h

Issue 330753002: Browser and unit tests for the SB interstitial V3 (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixed rebase-induced bugs Created 6 years, 6 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/safe_browsing/safe_browsing_blocking_page.h
diff --git a/chrome/browser/safe_browsing/safe_browsing_blocking_page.h b/chrome/browser/safe_browsing/safe_browsing_blocking_page.h
index eccd1a020b50c0fc395a455af3a8a49e10a73bb4..51ec776e849d6b332f71e320af78f2aaf063fdf4 100644
--- a/chrome/browser/safe_browsing/safe_browsing_blocking_page.h
+++ b/chrome/browser/safe_browsing/safe_browsing_blocking_page.h
@@ -82,12 +82,11 @@ class SafeBrowsingBlockingPage : public content::InterstitialPageDelegate {
virtual void OnDontProceed() OVERRIDE;
protected:
+ template <class TestSBInterstitialPage>
friend class SafeBrowsingBlockingPageTest;
+ template <class TestSBInterstitialPage>
FRIEND_TEST_ALL_PREFIXES(SafeBrowsingBlockingPageTest,
ProceedThenDontProceed);
- friend class SafeBrowsingBlockingPageV2Test;
- FRIEND_TEST_ALL_PREFIXES(SafeBrowsingBlockingPageV2Test,
- ProceedThenDontProceed);
void SetReportingPreference(bool report);
void UpdateReportingPref(); // Used for the transition from old to new pref.
@@ -106,8 +105,10 @@ class SafeBrowsingBlockingPage : public content::InterstitialPageDelegate {
return interstitial_page_;
}
+ template <class TestSBInterstitialPage>
FRIEND_TEST_ALL_PREFIXES(SafeBrowsingBlockingPageTest,
MalwareReportsTransitionDisabled);
+ template <class TestSBInterstitialPage>
FRIEND_TEST_ALL_PREFIXES(SafeBrowsingBlockingPageTest,
MalwareReportsToggling);
@@ -286,6 +287,24 @@ class SafeBrowsingBlockingPageV2 : public SafeBrowsingBlockingPage {
DISALLOW_COPY_AND_ASSIGN(SafeBrowsingBlockingPageV2);
};
+class SafeBrowsingBlockingPageV3 : public SafeBrowsingBlockingPage {
+ public:
+ SafeBrowsingBlockingPageV3(SafeBrowsingUIManager* ui_manager,
+ content::WebContents* web_contents,
+ const UnsafeResourceList& unsafe_resources);
+
+ // InterstitialPageDelegate method:
+ virtual std::string GetHTMLContents() OVERRIDE;
+
+ private:
+ // Fills the passed dictionary with the values to be passed to the template
+ // when creating the HTML.
+ void PopulateMalwareLoadTimeData(base::DictionaryValue* load_time_data);
+ void PopulatePhishingLoadTimeData(base::DictionaryValue* load_time_data);
+
+ DISALLOW_COPY_AND_ASSIGN(SafeBrowsingBlockingPageV3);
+};
+
// Factory for creating SafeBrowsingBlockingPage. Useful for tests.
class SafeBrowsingBlockingPageFactory {
public:

Powered by Google App Engine
This is Rietveld 408576698