Minor revisions to incognito hit report code

components/safe_browsing/base_ui_manager.h

 // Copyright 2016 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.
 
 #ifndef COMPONENTS_SAFE_BROWSING_BASE_UI_MANAGER_H_
 #define COMPONENTS_SAFE_BROWSING_BASE_UI_MANAGER_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 43 matching lines @@
   // the current page is 'safe'.
   virtual void LogPauseDelay(base::TimeDelta time);
 
   // This is a no-op in the base class, but should be overridden to send threat
   // details. Called on the IO thread by the ThreatDetails with the serialized
   // protocol buffer.
   virtual void SendSerializedThreatDetails(const std::string& serialized);
 
   // This is a no-op in the base class, but should be overridden to report hits
   // to the unsafe contents (malware, phishing, unsafe download URL)
-  // to the server. Can only be called on UI thread.
+  // to the server. Can only be called on UI thread. Will only upload a hit
+  // report if the user has enabled SBER and is not currently in incognito mode.
   virtual void MaybeReportSafeBrowsingHit(
       const safe_browsing::HitReport& hit_report,
-      content::WebContents* web_contents);
+      const content::WebContents* web_contents);
 
   // A convenience wrapper method for IsUrlWhitelistedOrPendingForWebContents.
   virtual bool IsWhitelisted(const UnsafeResource& resource);
 
   // Checks if we already displayed or are displaying an interstitial
   // for the top-level site |url| in a given WebContents. If
   // |whitelist_only|, it returns true only if the user chose to ignore
   // the interstitial. Otherwise, it returns true if an interstitial for
   // |url| is already displaying *or* if the user has seen an
   // interstitial for |url| before in this WebContents and proceeded
@@ skipping 69 matching lines @@
 
  private:
   friend class base::RefCountedThreadSafe<BaseUIManager>;
 
   DISALLOW_COPY_AND_ASSIGN(BaseUIManager);
 };
 
 }  // namespace safe_browsing
 
 #endif  // COMPONENTS_SAFE_BROWSING_BASE_UI_MANAGER_H_