Send malware reports when a user opts-in from the safe browsing interstitial ...

chrome/browser/safe_browsing/safe_browsing_service.h

 // Copyright (c) 2010 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.
 //
 // The Safe Browsing service is responsible for downloading anti-phishing and
 // anti-malware tables and checking urls against them.
 
 #ifndef CHROME_BROWSER_SAFE_BROWSING_SAFE_BROWSING_SERVICE_H_
 #define CHROME_BROWSER_SAFE_BROWSING_SAFE_BROWSING_SERVICE_H_
 #pragma once
 
 #include <deque>
 #include <set>
 #include <string>
 #include <vector>
 
 #include "base/hash_tables.h"
 #include "base/lock.h"
 #include "base/ref_counted.h"
 #include "base/scoped_ptr.h"
 #include "base/time.h"
 #include "chrome/browser/safe_browsing/safe_browsing_util.h"
 #include "googleurl/src/gurl.h"
 #include "webkit/glue/resource_type.h"
 
 class PrefService;
 class SafeBrowsingDatabase;
+class SafeBrowsingMalwareReport;
 class SafeBrowsingProtocolManager;
 class URLRequestContextGetter;
 
 namespace base {
 class Thread;
 }
 
 // Construction needs to happen on the main thread.
 class SafeBrowsingService
     : public base::RefCountedThreadSafe<SafeBrowsingService> {
@@ skipping 129 matching lines @@
 
   // Called on the IO thread to reset the database.
   void ResetDatabase();
 
   // Log the user perceived delay caused by SafeBrowsing. This delay is the time
   // delta starting from when we would have started reading data from the
   // network, and ending when the SafeBrowsing check completes indicating that
   // the current page is 'safe'.
   void LogPauseDelay(base::TimeDelta time);
 
+  // Send the given malware report.
+  virtual void SendMalwareReport(

[lzheng, 2010/11/30 23:14:49]

Call this ReprotMalwareDetails and explain the difference between this one and
"ReportSafeBrowsingHit"? I found it is hard to tell the difference between the
two functions by reading function names and comments.

[panayiotis, 2010/12/01 22:34:52]

Done.

+      scoped_refptr<SafeBrowsingMalwareReport> report);
+
+ protected:
+  virtual ~SafeBrowsingService();  // For the unittest.
+
  private:
   typedef std::set<SafeBrowsingCheck*> CurrentChecks;
   typedef std::vector<SafeBrowsingCheck*> GetHashRequestors;
   typedef base::hash_map<SBPrefix, GetHashRequestors> GetHashRequests;
 
   // Used for whitelisting a render view when the user ignores our warning.
   struct WhiteListedEntry;
 
   // Clients that we've queued up for checking later once the database is ready.
   struct QueuedCheck {
     Client* client;
     GURL url;
     base::Time start;
   };
 
   friend class base::RefCountedThreadSafe<SafeBrowsingService>;
   friend class SafeBrowsingServiceTest;
 
-  ~SafeBrowsingService();
-
   // Called to initialize objects that are used on the io_thread.
   void OnIOInitialize(const std::string& client_key,
                       const std::string& wrapped_key,
                       URLRequestContextGetter* request_context_getter);
 
   // Called to shutdown operations on the io_thread.
   void OnIOShutdown();
 
   // Returns whether |database_| exists and is accessible.
   bool DatabaseAvailable() const;
@@ skipping 112 matching lines @@
   // Indicates if we're in the midst of trying to close the database.  If this
   // is true, nothing on the IO thread should access the database.
   bool closing_database_;
 
   std::deque<QueuedCheck> queued_checks_;
 
   DISALLOW_COPY_AND_ASSIGN(SafeBrowsingService);
 };
 
 #endif  // CHROME_BROWSER_SAFE_BROWSING_SAFE_BROWSING_SERVICE_H_