Update the malware interstitial to have the new layout

chrome/browser/safe_browsing/safe_browsing_blocking_page.h

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 43ac464b57d8d2d4d8769e630e575e4fd5cf2d53..c3e4b23ad5732c2db14d7e99bebba3ed3a1f7c4e 100644
--- a/chrome/browser/safe_browsing/safe_browsing_blocking_page.h
+++ b/chrome/browser/safe_browsing/safe_browsing_blocking_page.h
@@ -280,6 +280,25 @@ class SafeBrowsingBlockingPageV2 : public SafeBrowsingBlockingPage {
   DISALLOW_COPY_AND_ASSIGN(SafeBrowsingBlockingPageV2);
 };
 
+class SafeBrowsingBlockingPageV3 : public SafeBrowsingBlockingPage {

[Bernhard Bauer, 2014/06/09 10:04:03]

If this class isn't meant to be directly instantiated, you could move it to an
anonymous namespace in the .cc file.

[felt, 2014/06/09 14:24:09]

I copied the pattern the pattern that's already in place in this file (if you
look above, V1 and V2 are handled in precisely the same way). Do you mind if I
leave as-is for consistency? Or should I refactor all three?

[Bernhard Bauer, 2014/06/09 14:45:41]

If you don't mind shaving some yak, it would be a nice cleanup. If you do mind,
it would be nice to leave at least a TODO here, so you (or some other helpful
contributor) can do it in a followup.

[felt, 2014/06/09 15:24:02]

I moved the V3 one into an anonymous namespace. I'll send another CL that does
the same for V1 and V2, to avoid confusion about what this CL touches.

[Bernhard Bauer, 2014/06/09 15:43:00]

Awesome, thanks!

+ public:
+  // Don't instantiate this class directly, use ShowBlockingPage instead.
+  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: