Decouple killbit testing from external extension providers

chrome/browser/extensions/stateful_external_extension_provider.cc

 // Copyright (c) 2010 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/extensions/stateful_external_extension_provider.h"
 
 #include "app/app_paths.h"
 #include "base/file_path.h"
 #include "base/logging.h"
 #include "base/path_service.h"
@@ skipping 15 matching lines @@
     Extension::Location crx_location,
     Extension::Location download_location)
   : crx_location_(crx_location),
     download_location_(download_location) {
 }
 
 StatefulExternalExtensionProvider::~StatefulExternalExtensionProvider() {
 }
 
 void StatefulExternalExtensionProvider::VisitRegisteredExtension(
-    Visitor* visitor, const std::set<std::string>& ids_to_ignore) const {
+    Visitor* visitor) const {
   for (DictionaryValue::key_iterator i = prefs_->begin_keys();
        i != prefs_->end_keys(); ++i) {
     const std::string& extension_id = *i;
-    if (ids_to_ignore.find(extension_id) != ids_to_ignore.end())
-      continue;
-
     DictionaryValue* extension;
     if (!prefs_->GetDictionaryWithoutPathExpansion(extension_id, &extension))
       continue;
 
     FilePath::StringType external_crx;
     std::string external_version;
     std::string external_update_url;
 
     bool has_external_crx = extension->GetString(kExternalCrx, &external_crx);
     bool has_external_version = extension->GetString(kExternalVersion,
@@ skipping 97 matching lines @@
   } else {
     NOTREACHED();  // Chrome should not allow prefs to get into this state.
     return false;
   }
 
   if (location)
     *location = loc;
 
   return true;
 }