Sanity tweaks to the extension blacklist: check all extensions at once on

chrome/browser/extensions/blacklist.cc

Index: chrome/browser/extensions/blacklist.cc
diff --git a/chrome/browser/extensions/blacklist.cc b/chrome/browser/extensions/blacklist.cc
index 12e5a7a524dfc6c129851dd65215c279da9b2e95..f8eaec3b56195b2195d940b7be9d7f9383a09c73 100644
--- a/chrome/browser/extensions/blacklist.cc
+++ b/chrome/browser/extensions/blacklist.cc
@@ -181,6 +181,20 @@ void Blacklist::GetBlacklistedIDs(const std::set<std::string>& ids,
                  callback));
 }
 
+namespace {
+void IsNotEmpty(const Blacklist::IsBlacklistedCallback& callback,
+                const std::set<std::string>& set) {
+  callback.Run(set.empty());
+}
+}  // namespace

[Matt Perry, 2013/02/07 02:16:33]

nit: I think this might be against the style guide. I would move it up to the
top of the file with the rest of the anon namespace stuff.

[not at google - send to devlin, 2013/02/07 02:26:58]

Done.

+
+void Blacklist::IsBlacklisted(const std::string& extension_id,
+                              const IsBlacklistedCallback& callback) {
+  std::set<std::string> check;
+  check.insert(extension_id);
+  GetBlacklistedIDs(check, base::Bind(&IsNotEmpty, callback));
+}
+
 void Blacklist::SetFromUpdater(const std::vector<std::string>& ids,
                                const std::string& version) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));