Refrain from sending hit report when user is in incognito mode

components/safe_browsing/base_ui_manager.cc

Index: components/safe_browsing/base_ui_manager.cc
diff --git a/components/safe_browsing/base_ui_manager.cc b/components/safe_browsing/base_ui_manager.cc
index cd8940d9e1c7ecd423f80e290a6704881c332224..c6b3de970f9d08515120b683a794f318e23c2726 100644
--- a/components/safe_browsing/base_ui_manager.cc
+++ b/components/safe_browsing/base_ui_manager.cc
@@ -248,7 +248,8 @@ void BaseUIManager::ShowBlockingPageForResource(
 // or after the warning dialog for download urls, only for
 // UMA || extended_reporting users.

[Jialiu Lin, 2017/05/18 22:32:12]

This comment is no longer accurate. Could you update it? 
Should be ".., only for extended reporting users who are not in incognito mode."

[mortonm, 2017/05/19 20:21:03]

Done.

 void BaseUIManager::MaybeReportSafeBrowsingHit(
-    const HitReport& hit_report) {
+    const HitReport& hit_report,
+    content::WebContents* web_contents) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   return;
 }