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

Unified Diff: chrome/browser/chromeos/policy/user_cloud_policy_manager_chromeos.cc

Issue 2940883002: Remove redundant condition in policy setup (Closed)
Patch Set: Improve constructor comment Created 3 years, 6 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
« no previous file with comments | « chrome/browser/chromeos/policy/user_cloud_policy_manager_chromeos.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/chromeos/policy/user_cloud_policy_manager_chromeos.cc
diff --git a/chrome/browser/chromeos/policy/user_cloud_policy_manager_chromeos.cc b/chrome/browser/chromeos/policy/user_cloud_policy_manager_chromeos.cc
index dcf21cddbb0d65bb8975f005dd235e2e056e528e..f974f240ade365cc8adf3a99fbe5f8f91cc9cfb2 100644
--- a/chrome/browser/chromeos/policy/user_cloud_policy_manager_chromeos.cc
+++ b/chrome/browser/chromeos/policy/user_cloud_policy_manager_chromeos.cc
@@ -99,8 +99,7 @@ UserCloudPolicyManagerChromeOS::UserCloudPolicyManagerChromeOS(
store_(std::move(store)),
external_data_manager_(std::move(external_data_manager)),
component_policy_cache_path_(component_policy_cache_path),
- wait_for_policy_fetch_(wait_for_policy_fetch),
- policy_fetch_timeout_(false, false) {
+ wait_for_policy_fetch_(wait_for_policy_fetch) {
time_init_started_ = base::Time::Now();
// Caller must pass a non-zero policy_fetch_timeout iff
@@ -111,8 +110,7 @@ UserCloudPolicyManagerChromeOS::UserCloudPolicyManagerChromeOS(
chromeos::switches::kAllowFailedPolicyFetchForTest) ||
!initial_policy_fetch_timeout.is_max();
// No need to set the timer when the timeout is infinite.
- if (wait_for_policy_fetch_ && allow_failed_policy_fetches_ &&
- !initial_policy_fetch_timeout.is_max()) {
+ if (wait_for_policy_fetch_ && !initial_policy_fetch_timeout.is_max()) {
policy_fetch_timeout_.Start(
FROM_HERE,
initial_policy_fetch_timeout,
« no previous file with comments | « chrome/browser/chromeos/policy/user_cloud_policy_manager_chromeos.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698