PM: Fix two comments.
Removes an old TODO and revises another comment.
BUG=None
TEST=None
Change-Id: I2342ae2a8af225246c57ae5c26b58be3985cc95a
Reviewed-on: https://chromium-review.googlesource.com/201108
Tested-by: Gilad Arnold <garnold@chromium.org>
Reviewed-by: Alex Vakulenko <avakulenko@chromium.org>
Commit-Queue: Gilad Arnold <garnold@chromium.org>
diff --git a/policy_manager/chromeos_policy.cc b/policy_manager/chromeos_policy.cc
index f0d6e3d..2ce8108 100644
--- a/policy_manager/chromeos_policy.cc
+++ b/policy_manager/chromeos_policy.cc
@@ -253,9 +253,6 @@
//
// If no wait period was previously determined, or it no longer fits in the
// scatter factor, then generate a new one. Otherwise, keep the one we have.
- // TODO(garnold) Current code (UpdateAttempter::GenerateNewWaitingPeriod())
- // always generates a non-zero value, which seems to imply that *some*
- // scattering always happens. Yet to validate whether this is intentional.
TimeDelta wait_period = update_state.scatter_wait_period;
if (wait_period == kZeroInterval || wait_period > *scatter_factor_p) {
wait_period = TimeDelta::FromSeconds(
diff --git a/policy_manager/policy.h b/policy_manager/policy.h
index 22d651e..f5fd257 100644
--- a/policy_manager/policy.h
+++ b/policy_manager/policy.h
@@ -43,7 +43,7 @@
int scatter_check_threshold;
// Minimum/maximum check threshold values.
// TODO(garnold) These appear to not be related to the current update and so
- // should probably be obtained vs variables ia UpdaterProvider.
+ // should probably be obtained as variables via UpdaterProvider.
int scatter_check_threshold_min;
int scatter_check_threshold_max;
};