Small modifications to safebrowsing code to make it simpler to add the extension

chrome/browser/safe_browsing/download_protection_service.cc

 // Copyright (c) 2012 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/download_protection_service.h"
 
 #include "base/bind.h"
 #include "base/compiler_specific.h"
 #include "base/format_macros.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 280 matching lines @@
 class DownloadUrlSBClient : public DownloadSBClient {
  public:
   DownloadUrlSBClient(
       const DownloadProtectionService::DownloadInfo& info,
       const DownloadProtectionService::CheckDownloadCallback& callback,
       const scoped_refptr<SafeBrowsingUIManager>& ui_manager,
       const scoped_refptr<SafeBrowsingDatabaseManager>& database_manager)
       : DownloadSBClient(info, callback, ui_manager,
                          DOWNLOAD_URL_CHECKS_TOTAL,
                          DOWNLOAD_URL_CHECKS_MALWARE),
-        database_manager_(database_manager) { }
+        database_manager_(database_manager) {
+  }
 
   virtual void StartCheck() OVERRIDE {
     DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
-    if (!database_manager_ || database_manager_->CheckDownloadUrl(
-            info_.download_url_chain, this)) {
+    if (database_manager_->CheckDownloadUrl(info_.download_url_chain, this)) {

[Scott Hess - ex-Googler, 2013/01/17 19:24:42]

Per earlier discussion, I think you need to put back the NULL test for
database_manager_.

Actually, at that point the whitespace change becomes egregious, so maybe just
revert this entire file.

[not at google - send to devlin, 2013/01/18 01:02:42]

Done.

       CheckDone(SB_THREAT_TYPE_SAFE);
     } else {
       AddRef();  // SafeBrowsingService takes a pointer not a scoped_refptr.
     }
   }
 
   virtual bool IsDangerous(SBThreatType threat_type) const OVERRIDE {
     return threat_type == SB_THREAT_TYPE_BINARY_MALWARE_URL;
   }
 
@@ skipping 666 matching lines @@
   std::string url = kDownloadRequestUrl;
   std::string api_key = google_apis::GetAPIKey();
   if (!api_key.empty()) {
     base::StringAppendF(&url, "?key=%s",
                         net::EscapeQueryParamValue(api_key, true).c_str());
   }
   return url;
 }
 
 }  // namespace safe_browsing