Create a browser feature extractor that runs after the renderer has

chrome/browser/safe_browsing/client_side_detection_host.h

Index: chrome/browser/safe_browsing/client_side_detection_host.h
diff --git a/chrome/browser/safe_browsing/client_side_detection_host.h b/chrome/browser/safe_browsing/client_side_detection_host.h
index d22931086a5cc4114f9ef9cc7427913fce8e91f0..20100924e5748c4df21b13d5977e5584839bd803 100644
--- a/chrome/browser/safe_browsing/client_side_detection_host.h
+++ b/chrome/browser/safe_browsing/client_side_detection_host.h
@@ -17,7 +17,8 @@
 class TabContents;
 
 namespace safe_browsing {
-
+class BrowserFeatureExtractor;
+class ClientPhishingRequest;
 class ClientSideDetectionService;
 
 // This class is used to receive the IPC from the renderer which
@@ -57,6 +58,8 @@ class ClientSideDetectionHost : public TabContentsObserver {
   // Otherwise, we do nothing.  Called in UI thread.
   void MaybeShowPhishingWarning(GURL phishing_url, bool is_phishing);
 
+  void FeatureExtractionDone(bool success, ClientPhishingRequest* request);

[Brian Ryner, 2011/06/10 04:45:10]

Add a brief comment for this method?

[noelutz, 2011/06/14 01:10:10]

Done.

+
   // Used for testing.  This function does not take ownership of the service
   // class.
   void set_client_side_detection_service(ClientSideDetectionService* service);
@@ -72,6 +75,8 @@ class ClientSideDetectionHost : public TabContentsObserver {
   // Keep a handle to the latest classification request so that we can cancel
   // it if necessary.
   scoped_refptr<ShouldClassifyUrlRequest> classification_request_;
+  // Browser-side feature extractor.
+  scoped_ptr<BrowserFeatureExtractor> feature_extractor_;
 
   base::ScopedCallbackFactory<ClientSideDetectionHost> cb_factory_;