Cleanup and style guideline conformance for policy implementation

chrome/browser/policy/file_based_policy_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/policy/file_based_policy_provider.h"
 
 #include <set>
 
 #include "base/logging.h"
 #include "base/message_loop.h"
@@ skipping 32 matching lines @@
                                       kSettleIntervalSeconds,
                                       kReloadIntervalMinutes);
   watcher_ = new FileBasedPolicyWatcher;
   watcher_->Init(loader_.get());
 }
 
 FileBasedPolicyProvider::~FileBasedPolicyProvider() {
   loader_->Stop();
 }
 
-bool FileBasedPolicyProvider::Provide(ConfigurationPolicyStore* store) {
+bool FileBasedPolicyProvider::Provide(
+    ConfigurationPolicyStoreInterface* store) {
   scoped_ptr<DictionaryValue> policy(loader_->GetPolicy());
   DCHECK(policy.get());
   DecodePolicyValueTree(policy.get(), store);
   return true;
 }
 
 void FileBasedPolicyProvider::DecodePolicyValueTree(
     DictionaryValue* policies,
-    ConfigurationPolicyStore* store) {
+    ConfigurationPolicyStoreInterface* store) {
   const PolicyDefinitionList* policy_list(policy_definition_list());
   for (const PolicyDefinitionList::Entry* i = policy_list->begin;
        i != policy_list->end; ++i) {
     Value* value;
     if (policies->Get(i->name, &value) && value->IsType(i->value_type))
       store->Apply(i->policy_type, value->DeepCopy());
   }
 
   // TODO(mnissler): Handle preference overrides once |ConfigurationPolicyStore|
   // supports it.
@@ skipping 176 matching lines @@
       !watcher_.Watch(loader->config_file_path(), loader.get()))
     loader->OnError();
 
   // There might have been changes to the directory in the time between
   // construction of the loader and initialization of the watcher. Call reload
   // to detect if that is the case.
   loader->Reload();
 }
 
 }  // namespace policy