Only run the phishing classifier on HTML and XHTML documents.

This also adds support to TestRenderViewHost for changing the MIME type that
it reports in the ViewHostMsg_FrameNavigate_Params.

BUG=none
TEST=ClientSideDetectionHostTest.ShouldClassifyUrl

Review URL: http://codereview.chromium.org/6691036

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80505 0039d316-1c4b-4281-b951-d872f2087c98

2 files changed, 14 insertions, 2 deletions

diff --git a/content/browser/renderer_host/test_render_view_host.cc b/content/browser/renderer_host/test_render_view_host.cc
index 06e884c..6eba441 100644
--- a/content/browser/renderer_host/test_render_view_host.cc
+++ b/content/browser/renderer_host/test_render_view_host.cc
@@ -46,7 +46,8 @@ TestRenderViewHost::TestRenderViewHost(SiteInstance* instance,
                      kInvalidSessionStorageNamespaceId),
       render_view_created_(false),
       delete_counter_(NULL),
-      simulate_fetch_via_proxy_(false) {
+      simulate_fetch_via_proxy_(false),
+      contents_mime_type_("text/html") {
   // For normal RenderViewHosts, this is freed when |Shutdown()| is called.
   // For TestRenderViewHost, the view is explicitly deleted in the destructor
   // below, because TestRenderWidgetHostView::Destroy() doesn't |delete this|.
@@ -95,7 +96,7 @@ void TestRenderViewHost::SendNavigateWithTransition(
   params.password_form = PasswordForm();
   params.security_info = std::string();
   params.gesture = NavigationGestureUser;
-  params.contents_mime_type = std::string();
+  params.contents_mime_type = contents_mime_type_;
   params.is_post = false;
   params.was_within_same_page = false;
   params.http_status_code = 0;
@@ -112,6 +113,10 @@ void TestRenderViewHost::set_simulate_fetch_via_proxy(bool proxy) {
   simulate_fetch_via_proxy_ = proxy;
 }
 
+void TestRenderViewHost::set_contents_mime_type(const std::string& mime_type) {
+  contents_mime_type_ = mime_type;
+}
+
 TestRenderWidgetHostView::TestRenderWidgetHostView(RenderWidgetHost* rwh)
     : rwh_(rwh),
       is_showing_(false) {
diff --git a/content/browser/renderer_host/test_render_view_host.h b/content/browser/renderer_host/test_render_view_host.h
index f25c8c7..4326529 100644
--- a/content/browser/renderer_host/test_render_view_host.h
+++ b/content/browser/renderer_host/test_render_view_host.h
@@ -206,6 +206,10 @@ class TestRenderViewHost : public RenderViewHost {
   // False by default.
   void set_simulate_fetch_via_proxy(bool proxy);
 
+  // If set, future loads will have |mime_type| set as the mime type.
+  // If not set, the mime type will default to "text/html".
+  void set_contents_mime_type(const std::string& mime_type);
+
   // RenderViewHost overrides --------------------------------------------------
 
   virtual bool CreateRenderView(const string16& frame_name);
@@ -224,6 +228,9 @@ class TestRenderViewHost : public RenderViewHost {
   // See set_simulate_fetch_via_proxy() above.
   bool simulate_fetch_via_proxy_;
 
+  // See set_contents_mime_type() above.
+  std::string contents_mime_type_;
+
   DISALLOW_COPY_AND_ASSIGN(TestRenderViewHost);
 };