If we show a SafeBrowsing warning we always send the client-side detection

chrome/browser/safe_browsing/client_side_detection_service.cc

 // Copyright (c) 2011 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 "chrome/browser/safe_browsing/client_side_detection_service.h"
 
 #include "base/command_line.h"
 #include "base/file_util_proxy.h"
 #include "base/logging.h"
 #include "base/time.h"
@@ skipping 246 matching lines @@
     ClientPhishingRequest* verdict,
     ClientReportPhishingRequestCallback* callback) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   scoped_ptr<ClientPhishingRequest> request(verdict);
   scoped_ptr<ClientReportPhishingRequestCallback> cb(callback);
 
   std::string request_data;
   if (!request->SerializeToString(&request_data)) {
     UMA_HISTOGRAM_COUNTS("SBClientPhishing.RequestNotSerialized", 1);
     VLOG(1) << "Unable to serialize the CSD request. Proto file changed?";
-    cb->Run(GURL(request->url()), false);
+    if (cb.get()) {
+      cb->Run(GURL(request->url()), false);
+    }
     return;
   }
 
   URLFetcher* fetcher = URLFetcher::Create(0 /* ID is not used */,
                                            GURL(kClientReportPhishingUrl),
                                            URLFetcher::POST,
                                            this);
 
   // Remember which callback and URL correspond to the current fetcher object.
   ClientReportInfo* info = new ClientReportInfo;
@@ skipping 51 matching lines @@
 
 void ClientSideDetectionService::HandlePhishingVerdict(
     const URLFetcher* source,
     const GURL& url,
     const net::URLRequestStatus& status,
     int response_code,
     const net::ResponseCookies& cookies,
     const std::string& data) {
   ClientPhishingResponse response;
   scoped_ptr<ClientReportInfo> info(client_phishing_reports_[source]);
+  bool is_phishing = false;
   if (status.is_success() && RC_REQUEST_OK == response_code &&
       response.ParseFromString(data)) {
     // Cache response, possibly flushing an old one.
     cache_[info->phishing_url] =
         make_linked_ptr(new CacheState(response.phishy(), base::Time::Now()));
-    info->callback->Run(info->phishing_url, response.phishy());
+    is_phishing = response.phishy();
   } else {
     DLOG(ERROR) << "Unable to get the server verdict for URL: "
                 << info->phishing_url << " status: " << status.status() << " "
                 << "response_code:" << response_code;
-    info->callback->Run(info->phishing_url, false);
+  }
+  if (info->callback.get()) {
+    info->callback->Run(info->phishing_url, is_phishing);
   }
   client_phishing_reports_.erase(source);
   delete source;
 }
 
 bool ClientSideDetectionService::IsInCache(const GURL& url) {
   UpdateCache();
 
   return cache_.find(url) != cache_.end();
 }
@@ skipping 104 matching lines @@
     // |size| bits sets to one.
     std::string mask(net::kIPv6AddressSize, '\x00');
     mask.replace(0, size / 8, size / 8, '\xFF');
     if (size % 8) {
       mask[size / 8] = 0xFF << (8 - (size % 8));
     }
     (*bad_subnets)[mask].insert(model.bad_subnet(i).prefix());
   }
 }
 }  // namespace safe_browsing