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

chrome/browser/safe_browsing/safe_browsing_service.h

Index: chrome/browser/safe_browsing/safe_browsing_service.h
===================================================================
--- chrome/browser/safe_browsing/safe_browsing_service.h	(revision 66642)
+++ chrome/browser/safe_browsing/safe_browsing_service.h	(working copy)
@@ -25,6 +25,7 @@
 
 class PrefService;
 class SafeBrowsingDatabase;
+class SafeBrowsingMalwareReport;
 class SafeBrowsingProtocolManager;
 class URLRequestContextGetter;
 
@@ -174,6 +175,13 @@
   // 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;
@@ -192,8 +200,6 @@
   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,