Mostly integrate new malware IP blacklist with the csd client. When

chrome/browser/safe_browsing/client_side_detection_host.h

 // Copyright (c) 2011 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 CHROME_BROWSER_SAFE_BROWSING_CLIENT_SIDE_DETECTION_HOST_H_
 #define CHROME_BROWSER_SAFE_BROWSING_CLIENT_SIDE_DETECTION_HOST_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 34 matching lines @@
   // we should classify the new URL.
   virtual void DidNavigateMainFrame(
       const content::LoadCommittedDetails& details,
       const content::FrameNavigateParams& params) OVERRIDE;
 
   // Called when the SafeBrowsingService found a hit with one of the
   // SafeBrowsing lists.  This method is called on the UI thread.
   virtual void OnSafeBrowsingHit(
       const SafeBrowsingUIManager::UnsafeResource& resource) OVERRIDE;
 
+  virtual scoped_refptr<SafeBrowsingDatabaseManager> database_manager();
+
  protected:
+  // Used for testing only when we mock out the host.  The host object takes
+  // ownership of the manager object.

[mattm, 2013/10/29 01:11:47]

Not sure it makes sense to say "takes ownership" of a refcounted object. You
could say it takes a reference or holds a reference.

[noé, 2013/10/31 02:41:12]

Done.

+  explicit ClientSideDetectionHost(
+      SafeBrowsingDatabaseManager* database_manager);

[mattm, 2013/10/29 01:11:47]

Was it not possible to use the other constructor along with
set_safe_browsing_managers()?

[noé, 2013/10/31 02:41:12]

Done.

+
   // From content::WebContentsObserver.
   virtual void WebContentsDestroyed(content::WebContents* tab) OVERRIDE;
 
  private:
   friend class ClientSideDetectionHostTest;
   class ShouldClassifyUrlRequest;
   friend class ShouldClassifyUrlRequest;
 
   explicit ClientSideDetectionHost(content::WebContents* tab);
 
   // Verdict is an encoded ClientPhishingRequest protocol message.
   void OnPhishingDetectionDone(const std::string& verdict);
 
   // Callback that is called when the server ping back is
   // done. Display an interstitial if |is_phishing| is true.
   // Otherwise, we do nothing.  Called in UI thread.
   void MaybeShowPhishingWarning(GURL phishing_url, bool is_phishing);
 
   // Callback that is called when the malware IP server ping back is
   // done. Display an interstitial if |is_malware| is true.
   // Otherwise, we do nothing.  Called in UI thread.
   void MaybeShowMalwareWarning(GURL original_url, GURL malware_url,
                                bool is_malware);
 
   // Callback that is called when the browser feature extractor is done.
   // This method is responsible for deleting the request object.  Called on
   // the UI thread.
   void FeatureExtractionDone(bool success, ClientPhishingRequest* request);
+
   // Function to be called when the browser malware feature extractor is done.
-  void MalwareFeatureExtractionDone(scoped_ptr<ClientMalwareRequest> request);
+  // Called on the UI thread.
+  void MalwareFeatureExtractionDone(bool success,
+                                    scoped_ptr<ClientMalwareRequest> request);
 
   // Update the entries in browse_info_->ips map.
   void UpdateIPUrlMap(const std::string& ip, const std::string& url);
 
   // From NotificationObserver.  Called when a notification comes in.  This
   // method is called in the UI thread.
   virtual void Observe(int type,
                        const content::NotificationSource& source,
                        const content::NotificationDetails& details) OVERRIDE;
 
@@ skipping 58 matching lines @@
   // Whether the malware bad ip matching and report feature is enabled.
   // This should be accessed from UI thread.
   bool malware_report_enabled_;
 
   DISALLOW_COPY_AND_ASSIGN(ClientSideDetectionHost);
 };
 
 }  // namespace safe_browsing
 
 #endif  // CHROME_BROWSER_SAFE_BROWSING_CLIENT_SIDE_DETECTION_HOST_H_