Add DCHECKs to StatefulExternalExtensionProvider

chrome/browser/extensions/external_policy_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/external_policy_extension_provider.h"
 
 #include "base/logging.h"
 #include "base/values.h"
 #include "chrome/common/pref_names.h"
 #include "chrome/browser/extensions/stateful_external_extension_provider.h"
@@ skipping 12 matching lines @@
   if (!Extension::IdIsValid(id)) {
     LOG(WARNING) << "Policy specifies invalid ID for external "
                  << "extension: " << id;
     return false;
   }
   return true;
 }
 
 }
 
-ExternalPolicyExtensionProvider::ExternalPolicyExtensionProvider()
-  : StatefulExternalExtensionProvider(Extension::INVALID,
-                                      Extension::EXTERNAL_POLICY_DOWNLOAD) {
+ExternalPolicyExtensionProvider::ExternalPolicyExtensionProvider(
+    const ListValue* forcelist)
+        : StatefulExternalExtensionProvider(

[jochen (gone - plz use gerrit), 2010/12/14 13:40:43]

only indent 4 spaces

[gfeher, 2010/12/14 14:23:15]

Done.

+              Extension::INVALID,
+              Extension::EXTERNAL_POLICY_DOWNLOAD) {
+  SetPreferences(forcelist);
 }
 
 ExternalPolicyExtensionProvider::~ExternalPolicyExtensionProvider() {
 }
 
 void ExternalPolicyExtensionProvider::SetPreferences(
     const ListValue* forcelist) {
   DictionaryValue* result = new DictionaryValue();
   if (forcelist != NULL) {
     std::string extension_desc;
     for (ListValue::const_iterator it = forcelist->begin();
          it != forcelist->end(); ++it) {
       if (!(*it)->GetAsString(&extension_desc)) {
         LOG(WARNING) << "Failed to read forcelist string.";
       } else {
         // Each string item of the list has the following form:
         // extension_id_code;extension_update_url
         // The update URL might also contain semicolons.
         size_t pos = extension_desc.find(';');
         std::string id = extension_desc.substr(0, pos);
         std::string update_url = extension_desc.substr(pos+1);
         if (CheckExtension(id, update_url)) {
           result->SetString(id + ".external_update_url", update_url);
         }
       }
     }
   }
-  prefs_.reset(result);
+  set_prefs(result);
 }