Bug 1057764: Remove references to appRepUrl completely (r=gcp)

toolkit/components/downloads/ApplicationReputation.cpp

@@ -421,31 +421,8 @@ PendingLookup::LookupNext()
     nsRefPtr<PendingDBLookup> lookup(new PendingDBLookup(this));
     return lookup->LookupSpec(spec, true);
   }
-#ifdef XP_WIN
-  // If there is no service URL for querying application reputation, abort.
-  nsCString serviceUrl;
-  NS_ENSURE_SUCCESS(Preferences::GetCString(PREF_SB_APP_REP_URL, &serviceUrl),
-                    NS_ERROR_NOT_AVAILABLE);
-  if (serviceUrl.EqualsLiteral("")) {
-    return OnComplete(false, NS_ERROR_NOT_AVAILABLE);
-  }
-
-  // There are no more URIs to check against local list. If the file is not
-  // eligible for remote lookup, bail.
-  if (!IsBinaryFile()) {
-    LOG(("Not eligible for remote lookups [this=%x]", this));
-    return OnComplete(false, NS_OK);
-  }
-  // Send the remote query if we are on Windows.
-  nsresult rv = SendRemoteQuery();
-  if (NS_FAILED(rv)) {
-    return OnComplete(false, rv);
-  }
-  return NS_OK;
-#else
   LOG(("PendingLookup: Nothing left to check [this=%p]", this));
   return OnComplete(false, NS_OK);
-#endif
 }
 
 nsCString

toolkit/components/downloads/test/unit/xpcshell.ini

@@ -10,8 +10,9 @@ support-files =
   data/signed_win.exe
 
 [test_app_rep.js]
-[test_app_rep_windows.js]
-run-if = os == "win"
+# Bug 1057764: disable remote lookups completely
+# [test_app_rep_windows.js]
+# run-if = os == "win"
 [test_bug_382825.js]
 [test_bug_384744.js]
 [test_bug_395092.js]