Intergration of the client-side phishing detection.

chrome/browser/safe_browsing/client_side_detection_service.h

Index: chrome/browser/safe_browsing/client_side_detection_service.h
diff --git a/chrome/browser/safe_browsing/client_side_detection_service.h b/chrome/browser/safe_browsing/client_side_detection_service.h
index a9bc7f6966ddfb82ccc5d9ce970795b6ddc8ea58..51312b657e41c7f0bffea4c8f9672fd89d8e2afb 100644
--- a/chrome/browser/safe_browsing/client_side_detection_service.h
+++ b/chrome/browser/safe_browsing/client_side_detection_service.h
@@ -31,7 +31,9 @@
 #include "base/scoped_ptr.h"
 #include "base/task.h"
 #include "base/time.h"
+#include "chrome/common/render_messages_params.h"
 #include "chrome/browser/safe_browsing/csd.pb.h"
+#include "chrome/browser/tab_contents/navigation_controller.h"

[lzheng, 2011/02/11 19:39:42]

From the changes, it is not obvious to me hy these headers are added either.
Double check?

[noelutz, 2011/02/15 23:00:55]

Done.

 #include "chrome/common/net/url_fetcher.h"
 #include "googleurl/src/gurl.h"
 
@@ -82,11 +84,15 @@ class ClientSideDetectionService : public URLFetcher::Delegate {
   // fetch.  If an error occurs the phishing verdict will always be false.  The
   // callback is always called after SendClientReportPhishingRequest() returns
   // and on the same thread as SendClientReportPhishingRequest() was called.
-  void SendClientReportPhishingRequest(
+  virtual void SendClientReportPhishingRequest(
       const GURL& phishing_url,
       double score,
       ClientReportPhishingRequestCallback* callback);
 
+ protected:
+  // Used for testing only.
+  ClientSideDetectionService();
+
  private:
   friend class ClientSideDetectionServiceTest;