Intergration of the client-side phishing detection.

chrome/browser/safe_browsing/client_side_detection_service.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.
 //
 // Helper class which handles communication with the SafeBrowsing backends for
 // client-side phishing detection.  This class can be used to get a file
 // descriptor to the client-side phishing model and also to send a ping back to
 // Google to verify if a particular site is really phishing or not.
 //
 // This class is not thread-safe and expects all calls to GetModelFile() and
@@ skipping 13 matching lines @@
 #include "base/basictypes.h"
 #include "base/callback.h"
 #include "base/file_path.h"
 #include "base/gtest_prod_util.h"
 #include "base/platform_file.h"
 #include "base/ref_counted.h"
 #include "base/scoped_callback_factory.h"
 #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"
 
 class URLRequestContextGetter;
 
 namespace net {
 class URLRequestStatus;
 }  // namespace net
 
 namespace safe_browsing {
@@ skipping 30 matching lines @@
   // same thread as GetModelFile() was called.
   void GetModelFile(OpenModelDoneCallback* callback);
 
   // Sends a request to the SafeBrowsing servers with the potentially phishing
   // URL and the client-side phishing score.  The |phishing_url| scheme should
   // be HTTP.  This method takes ownership of the |callback| and calls it once
   // the result has come back from the server or if an error occurs during the
   // 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;
 
   enum ModelStatus {
     // It's unclear whether or not the model was already fetched.
     UNKNOWN_STATUS,
     // Model is fetched and is stored on disk.
     READY_STATUS,
     // Error occured during fetching or writing.
     ERROR_STATUS,
@@ skipping 98 matching lines @@
 
   // The context we use to issue network requests.
   scoped_refptr<URLRequestContextGetter> request_context_getter_;
 
   DISALLOW_COPY_AND_ASSIGN(ClientSideDetectionService);
 };
 
 }  // namepsace safe_browsing
 
 #endif  // CHROME_BROWSER_SAFE_BROWSING_CLIENT_SIDE_DETECTION_SERVICE_H_