Remove redundant condition in policy setup

chrome/browser/chromeos/policy/user_cloud_policy_manager_chromeos.h

 // Copyright (c) 2012 The Chromium 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 CHROME_BROWSER_CHROMEOS_POLICY_USER_CLOUD_POLICY_MANAGER_CHROMEOS_H_
 #define CHROME_BROWSER_CHROMEOS_POLICY_USER_CLOUD_POLICY_MANAGER_CHROMEOS_H_
 
 #include <memory>
 #include <string>
 
@@ skipping 26 matching lines @@
 class DeviceManagementService;
 class PolicyOAuth2TokenFetcher;
 class WildcardLoginChecker;
 
 // Implements logic for initializing user policy on Chrome OS.
 class UserCloudPolicyManagerChromeOS : public CloudPolicyManager,
                                        public CloudPolicyClient::Observer,
                                        public CloudPolicyService::Observer,
                                        public KeyedService {
  public:
-  // If |wait_for_policy_fetch| is true, IsInitializationComplete() will return
-  // false as long as there hasn't been a successful policy fetch.
+  // If |wait_for_policy_fetch| is true, IsInitializationComplete() is forced to
+  // false until either there has been a successful policy fetch from the server
+  // or |initial_policy_fetch_timeout| has expired. (The timeout may be set to
+  // TimeDelta::Max() to block permanently.)
   // |task_runner| is the runner for policy refresh tasks.
   // |file_task_runner| is used for file operations. Currently this must be the
   // FILE BrowserThread.
   // |io_task_runner| is used for network IO. Currently this must be the IO
   // BrowserThread.
   UserCloudPolicyManagerChromeOS(
       std::unique_ptr<CloudPolicyStore> store,
       std::unique_ptr<CloudExternalDataManager> external_data_manager,
       const base::FilePath& component_policy_cache_path,
       bool wait_for_policy_fetch,
@@ skipping 98 matching lines @@
   // IsInitializationComplete().
   bool wait_for_policy_fetch_;
 
   // Whether we should allow policy fetches to fail, or wait forever until they
   // succeed (typically we won't allow them to fail until we have loaded policy
   // at least once).
   bool allow_failed_policy_fetches_;
 
   // A timer that puts a hard limit on the maximum time to wait for the initial
   // policy fetch.
-  base::Timer policy_fetch_timeout_;
+  base::Timer policy_fetch_timeout_{false, false};
 
   // The pref service to pass to the refresh scheduler on initialization.
   PrefService* local_state_;
 
   // Used to fetch the policy OAuth token, when necessary. This object holds
   // a callback with an unretained reference to the manager, when it exists.
   std::unique_ptr<PolicyOAuth2TokenFetcher> token_fetcher_;
 
   // Keeps alive the wildcard checker while its running.
   std::unique_ptr<WildcardLoginChecker> wildcard_login_checker_;
 
   // The access token passed to OnAccessTokenAvailable. It is stored here so
   // that it can be used if OnInitializationCompleted is called later.
   std::string access_token_;
 
   // Timestamps for collecting timing UMA stats.
   base::Time time_init_started_;
   base::Time time_init_completed_;
   base::Time time_token_available_;
   base::Time time_client_registered_;
 
   DISALLOW_COPY_AND_ASSIGN(UserCloudPolicyManagerChromeOS);
 };
 
 }  // namespace policy
 
 #endif  // CHROME_BROWSER_CHROMEOS_POLICY_USER_CLOUD_POLICY_MANAGER_CHROMEOS_H_