Refrain from sending hit report when user is in incognito mode

components/safe_browsing/base_ui_manager.cc

 // 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.
 
 #include "components/safe_browsing/base_ui_manager.h"
 
 #include "base/bind.h"
 #include "base/callback.h"
 #include "base/i18n/rtl.h"
 #include "base/memory/ptr_util.h"
@@ skipping 226 matching lines @@
   return;
 }
 
 void BaseUIManager::CreateAndSendHitReport(const UnsafeResource& resource) {}
 
 void BaseUIManager::ShowBlockingPageForResource(
     const UnsafeResource& resource) {
   BaseBlockingPage::ShowBlockingPage(this, resource);
 }
 
 // A safebrowsing hit is sent after a blocking page for malware/phishing

[vakh (use Gerrit instead), 2017/05/19 22:41:13]

nit: SafeBrowsing

[mortonm, 2017/05/22 17:03:59]

Done.

-// or after the warning dialog for download urls, only for
-// UMA || extended_reporting users.
+// or after the warning dialog for download urls, only for extended_reporting
+// users who are not in incognito mode.
 void BaseUIManager::MaybeReportSafeBrowsingHit(
-    const HitReport& hit_report) {
+    const HitReport& hit_report,
+    content::WebContents* web_contents) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   return;
 }
 
 void BaseUIManager::ReportSafeBrowsingHitOnIOThread(
     const HitReport& hit_report) {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
   return;
 }
 
@@ skipping 90 matching lines @@
   if (resource.is_subresource) {
     NavigationEntry* entry = resource.GetNavigationEntryForResource();
     if (!entry)
       return GURL();
     return entry->GetURL().GetWithEmptyPath();
   }
   return resource.url.GetWithEmptyPath();
 }
 
 }  // namespace safe_browsing