PM: Make State an abstract interface.
Similar to CL:196656, move getters/variables to concrete
subclasses. In this case, also introduce a RealState type to achieve
this.
BUG=chromium:364763
TEST=Unit tests pass.
Change-Id: Ic12cd872e576c9690d32359474f3528cf706633d
Reviewed-on: https://chromium-review.googlesource.com/196885
Reviewed-by: David Zeuthen <zeuthen@chromium.org>
Commit-Queue: David Zeuthen <zeuthen@chromium.org>
Tested-by: David Zeuthen <zeuthen@chromium.org>
diff --git a/policy_manager/real_state.h b/policy_manager/real_state.h
new file mode 100644
index 0000000..7349d58
--- /dev/null
+++ b/policy_manager/real_state.h
@@ -0,0 +1,70 @@
+// Copyright (c) 2014 The Chromium OS 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 CHROMEOS_PLATFORM_UPDATE_ENGINE_POLICY_MANAGER_REAL_STATE_H_
+#define CHROMEOS_PLATFORM_UPDATE_ENGINE_POLICY_MANAGER_REAL_STATE_H_
+
+#include <base/memory/scoped_ptr.h>
+
+#include "update_engine/policy_manager/state.h"
+
+namespace chromeos_policy_manager {
+
+// State concrete implementation.
+class RealState : public State {
+ public:
+ virtual ~RealState() {}
+
+ RealState(ConfigProvider* config_provider,
+ DevicePolicyProvider* device_policy_provider,
+ RandomProvider* random_provider,
+ ShillProvider* shill_provider,
+ SystemProvider* system_provider,
+ TimeProvider* time_provider,
+ UpdaterProvider* updater_provider) :
+ config_provider_(config_provider),
+ device_policy_provider_(device_policy_provider),
+ random_provider_(random_provider),
+ shill_provider_(shill_provider),
+ system_provider_(system_provider),
+ time_provider_(time_provider),
+ updater_provider_(updater_provider) {}
+
+ // These methods return the given provider.
+ virtual ConfigProvider* config_provider() override {
+ return config_provider_.get();
+ }
+ virtual DevicePolicyProvider* device_policy_provider() override {
+ return device_policy_provider_.get();
+ }
+ virtual RandomProvider* random_provider() override {
+ return random_provider_.get();
+ }
+ virtual ShillProvider* shill_provider() override {
+ return shill_provider_.get();
+ }
+ virtual SystemProvider* system_provider() override {
+ return system_provider_.get();
+ }
+ virtual TimeProvider* time_provider() override {
+ return time_provider_.get();
+ }
+ virtual UpdaterProvider* updater_provider() override {
+ return updater_provider_.get();
+ }
+
+ private:
+ // Instances of the providers.
+ scoped_ptr<ConfigProvider> config_provider_;
+ scoped_ptr<DevicePolicyProvider> device_policy_provider_;
+ scoped_ptr<RandomProvider> random_provider_;
+ scoped_ptr<ShillProvider> shill_provider_;
+ scoped_ptr<SystemProvider> system_provider_;
+ scoped_ptr<TimeProvider> time_provider_;
+ scoped_ptr<UpdaterProvider> updater_provider_;
+};
+
+} // namespace chromeos_policy_manager
+
+#endif // CHROMEOS_PLATFORM_UPDATE_ENGINE_POLICY_MANAGER_REAL_STATE_H_