Set drive as the default download folder

components/policy/core/browser/configuration_policy_pref_store_test.h

 // Copyright 2014 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.
 
 #ifndef COMPONENTS_POLICY_CORE_BROWSER_CONFIGURATION_POLICY_PREF_STORE_TEST_H_
 #define COMPONENTS_POLICY_CORE_BROWSER_CONFIGURATION_POLICY_PREF_STORE_TEST_H_
 
 #include "base/basictypes.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/message_loop/message_loop.h"
 #include "components/policy/core/browser/configuration_policy_handler_list.h"
 #include "components/policy/core/common/mock_configuration_policy_provider.h"
 #include "components/policy/core/common/policy_service_impl.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace policy {
 
 class PolicyMap;
 class PolicyService;
 class ConfigurationPolicyPrefStore;
+struct PolicyHandlerParameters;

[bartfab (slow), 2014/03/14 13:43:45]

Nit: Arrange in alphabetic order.

 
 class ConfigurationPolicyPrefStoreTest : public testing::Test {
  protected:
   ConfigurationPolicyPrefStoreTest();
   virtual ~ConfigurationPolicyPrefStoreTest();
   virtual void TearDown() OVERRIDE;
+  virtual void PopulatePolicyHandlerParameters(PolicyHandlerParameters* perf);

[bartfab (slow), 2014/03/14 13:43:45]

Nit 1: Move this down a few lines so that instead of being lumped into this
block of methods, it stands on its own, and document that this can be overridden
to pass specific parameters to handlers.

Nit 2: s/perf/parameters/

   void UpdateProviderPolicy(const PolicyMap& policy);
 
   PolicyServiceImpl::Providers providers_;
   ConfigurationPolicyHandlerList handler_list_;
   MockConfigurationPolicyProvider provider_;
   scoped_ptr<PolicyService> policy_service_;
   scoped_refptr<ConfigurationPolicyPrefStore> store_;
   base::MessageLoop loop_;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(ConfigurationPolicyPrefStoreTest);
 };
 
 }  // namespace policy
 
 #endif  // COMPONENTS_POLICY_CORE_BROWSER_CONFIGURATION_POLICY_PREF_STORE_TEST_H_