PolicyManager: Convert EvalStatus to a enum class.
This simple patch converts EvalStatus to a enum class to have better
type checking over the enum and group the constants on a namespace.
BUG=chromium:340871
TEST=Build and run unit tests.
Change-Id: I56e8d52325c5de578a6c78c7c85c9dccf61d48a1
Reviewed-on: https://chromium-review.googlesource.com/189635
Reviewed-by: Alex Deymo <deymo@chromium.org>
Commit-Queue: Alex Deymo <deymo@chromium.org>
Tested-by: Alex Deymo <deymo@chromium.org>
diff --git a/policy_manager/policy_manager.h b/policy_manager/policy_manager.h
index d0be4e2..b84f493 100644
--- a/policy_manager/policy_manager.h
+++ b/policy_manager/policy_manager.h
@@ -30,10 +30,11 @@
// |policy_method|.
//
// When the policy request succeeds, the |result| is set and the method
- // returns EvalStatusSucceeded, otherwise, the |result| may not be set. Also,
- // if the policy implementation should block, this method returns immediately
- // with EvalStatusAskMeAgainLater. In case of failure EvalStatusFailed is
- // returned and the |error| message is set, which must not be NULL.
+ // returns EvalStatus::kSucceeded, otherwise, the |result| may not be set.
+ // Also, if the policy implementation should block, this method returns
+ // immediately with EvalStatus::kAskMeAgainLater. In case of failure
+ // EvalStatus::kFailed is returned and the |error| message is set, which must
+ // not be NULL.
//
// An example call to this method is:
// pm.PolicyRequest(&Policy::SomePolicyMethod, &bool_result, arg1, arg2);
@@ -51,7 +52,7 @@
scoped_ptr<const Policy> policy_;
// A safe default value to the current policy. This policy is used whenever
- // a policy implementation fails with EvalStatusFailed.
+ // a policy implementation fails with EvalStatus::kFailed.
const DefaultPolicy default_policy_;
// State Providers.