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

chrome/browser/safe_browsing/download_protection_service.cc

Index: chrome/browser/safe_browsing/download_protection_service.cc
diff --git a/chrome/browser/safe_browsing/download_protection_service.cc b/chrome/browser/safe_browsing/download_protection_service.cc
index bcd1643e02abb684895a3c0e7a49a5af8ee4f53f..8d60dda0b190cc3665ad261eccb3b2c16751984d 100644
--- a/chrome/browser/safe_browsing/download_protection_service.cc
+++ b/chrome/browser/safe_browsing/download_protection_service.cc
@@ -298,12 +298,12 @@ class DownloadUrlSBClient : public DownloadSBClient {
       : 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.