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

Unified Diff: chrome/browser/policy/mock_configuration_policy_provider.h

Issue 10236003: Added RefreshPolicies to PolicyService. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebased Created 8 years, 8 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/mock_configuration_policy_provider.h
diff --git a/chrome/browser/policy/mock_configuration_policy_provider.h b/chrome/browser/policy/mock_configuration_policy_provider.h
index 016f5199a2bed968603eaa875e4db26d74ddad6c..8134f078160d2f8918f67aaf6dae684ecf2a5d86 100644
--- a/chrome/browser/policy/mock_configuration_policy_provider.h
+++ b/chrome/browser/policy/mock_configuration_policy_provider.h
@@ -19,26 +19,21 @@ class MockConfigurationPolicyProvider : public ConfigurationPolicyProvider {
MockConfigurationPolicyProvider();
virtual ~MockConfigurationPolicyProvider();
- void AddMandatoryPolicy(const std::string& policy, Value* value);
- void AddRecommendedPolicy(const std::string& policy, Value* value);
- void RemovePolicy(const std::string& policy);
-
- void SetInitializationComplete(bool initialization_complete);
-
- // ConfigurationPolicyProvider method overrides.
- virtual bool ProvideInternal(PolicyMap* policies) OVERRIDE;
- virtual bool IsInitializationComplete() const OVERRIDE;
- virtual void RefreshPolicies() OVERRIDE;
+ MOCK_METHOD1(ProvideInternal, bool(PolicyMap*));
+ MOCK_CONST_METHOD0(IsInitializationComplete, bool());
+ MOCK_METHOD0(RefreshPolicies, void());
// Make public for tests.
using ConfigurationPolicyProvider::NotifyPolicyUpdated;
-
- private:
-
- PolicyMap policy_map_;
- bool initialization_complete_;
};
+// A gmock action that copies |policy_map| into the first argument of the mock
+// method, as expected by ProvideInternal().
+ACTION_P(CopyPolicyMap, policy_map) {
+ arg0->CopyFrom(*policy_map);
+ return true;
+}
+
class MockConfigurationPolicyObserver
: public ConfigurationPolicyProvider::Observer {
public:

Powered by Google App Engine
This is Rietveld 408576698