Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3523)

Unified Diff: chrome/browser/policy/configuration_policy_pref_store_unittest.cc

Issue 22812002: Refactored ConfigurationPolicyPrefStore to not depend on chrome/. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/policy/configuration_policy_pref_store_unittest.cc
diff --git a/chrome/browser/policy/configuration_policy_pref_store_unittest.cc b/chrome/browser/policy/configuration_policy_pref_store_unittest.cc
index dce901a904533cb5fffb4efd25c2300cb7107760..400d53b6fff06e3b08524fcf35187ef3b4997ac8 100644
--- a/chrome/browser/policy/configuration_policy_pref_store_unittest.cc
+++ b/chrome/browser/policy/configuration_policy_pref_store_unittest.cc
@@ -11,6 +11,7 @@
#include "base/prefs/pref_store_observer_mock.h"
#include "base/run_loop.h"
#include "chrome/browser/policy/configuration_policy_handler.h"
+#include "chrome/browser/policy/configuration_policy_handler_list.h"
#include "chrome/browser/policy/configuration_policy_pref_store.h"
#include "chrome/browser/policy/external_data_fetcher.h"
#include "chrome/browser/policy/mock_configuration_policy_provider.h"
@@ -55,6 +56,7 @@ class ConfigurationPolicyPrefStoreTest : public testing::Test {
providers.push_back(&provider_);
policy_service_.reset(new PolicyServiceImpl(providers));
store_ = new ConfigurationPolicyPrefStore(policy_service_.get(),
+ &handler_list_,
Mattias Nissler (ping if slow) 2013/08/12 13:38:28 I'm surprised the tests in these file pass with an
Joao da Silva 2013/08/12 13:41:48 It's not empty, the default ctor adds all the hand
Mattias Nissler (ping if slow) 2013/08/12 15:14:43 Ah, correct. All good then.
POLICY_LEVEL_MANDATORY);
}
@@ -68,6 +70,7 @@ class ConfigurationPolicyPrefStoreTest : public testing::Test {
loop.RunUntilIdle();
}
+ ConfigurationPolicyHandlerList handler_list_;
MockConfigurationPolicyProvider provider_;
scoped_ptr<PolicyServiceImpl> policy_service_;
scoped_refptr<ConfigurationPolicyPrefStore> store_;

Powered by Google App Engine
This is Rietveld 408576698