Remove thumbnails from the ClientSideDetectionService.

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 15 matching lines @@
 #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 "chrome/browser/safe_browsing/csd.pb.h"
 #include "chrome/common/net/url_fetcher.h"
 #include "googleurl/src/gurl.h"
 
-class SkBitmap;
 class URLRequestContextGetter;
 
 namespace net {
 class URLRequestStatus;
 }  // namespace net
 
 namespace safe_browsing {
 
 class ClientSideDetectionService : public URLFetcher::Delegate {
  public:
@@ skipping 21 matching lines @@
 
   // Gets the model file descriptor once the model is ready and stored
   // on disk.  If there was an error the callback is called and the
   // platform file is set to kInvalidPlatformFileValue. The
   // ClientSideDetectionService takes ownership of the |callback|.
   // The callback is always called after GetModelFile() returns and on the
   // same thread as GetModelFile() was called.
   void GetModelFile(OpenModelDoneCallback* callback);
 
   // Sends a request to the SafeBrowsing servers with the potentially phishing
-  // URL, the client-side phishing score, and a low resolution thumbnail.  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.
+  // 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(
       const GURL& phishing_url,
       double score,
-      SkBitmap thumbnail,
       ClientReportPhishingRequestCallback* callback);
 
  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,
@@ skipping 36 matching lines @@
                           int bytes_written);
 
   // Helper function which closes the |model_file_| if necessary.
   void CloseModelFile();
 
   // Starts preparing the request to be sent to the client-side detection
   // frontends.
   void StartClientReportPhishingRequest(
       const GURL& phishing_url,
       double score,
-      SkBitmap thumbnail,
       ClientReportPhishingRequestCallback* callback);
 
   // Starts getting the model file.
   void StartGetModelFile(OpenModelDoneCallback* callback);
 
   // Called by OnURLFetchComplete to handle the response from fetching the
   // model.
   void HandleModelResponse(const URLFetcher* source,
                            const GURL& url,
                            const net::URLRequestStatus& status,
@@ skipping 34 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_