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 228 matching lines @@
 
 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
 // 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;
 }
 
 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