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

Side by Side Diff: remoting/host/policy_hack/policy_watcher_linux.cc

Issue 722743003: Reporting of policy errors via host-offline-reason: part 1 (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebased and reopened to try memory trybots. Created 6 years 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 // Most of this code is copied from various classes in 5 // Most of this code is copied from various classes in
6 // src/chrome/browser/policy. In particular, look at 6 // src/chrome/browser/policy. In particular, look at
7 // 7 //
8 // file_based_policy_loader.{h,cc} 8 // file_based_policy_loader.{h,cc}
9 // config_dir_policy_provider.{h,cc} 9 // config_dir_policy_provider.{h,cc}
10 // 10 //
(...skipping 168 matching lines...) Expand 10 before | Expand all | Expand 10 after
179 } 179 }
180 180
181 // Load the policy definitions. 181 // Load the policy definitions.
182 scoped_ptr<base::DictionaryValue> new_policy = Load(); 182 scoped_ptr<base::DictionaryValue> new_policy = Load();
183 if (new_policy.get()) { 183 if (new_policy.get()) {
184 UpdatePolicies(new_policy.get()); 184 UpdatePolicies(new_policy.get());
185 ScheduleFallbackReloadTask(); 185 ScheduleFallbackReloadTask();
186 } else { 186 } else {
187 // A failure to load policy definitions is probably temporary, so try 187 // A failure to load policy definitions is probably temporary, so try
188 // again soon. 188 // again soon.
189 SignalTransientPolicyError();
189 ScheduleReloadTask(base::TimeDelta::FromSeconds(kSettleIntervalSeconds)); 190 ScheduleReloadTask(base::TimeDelta::FromSeconds(kSettleIntervalSeconds));
190 } 191 }
191 } 192 }
192 193
193 bool IsSafeToReloadPolicy(const base::Time& now, base::TimeDelta* delay) { 194 bool IsSafeToReloadPolicy(const base::Time& now, base::TimeDelta* delay) {
194 DCHECK(OnPolicyWatcherThread()); 195 DCHECK(OnPolicyWatcherThread());
195 DCHECK(delay); 196 DCHECK(delay);
196 const base::TimeDelta kSettleInterval = 197 const base::TimeDelta kSettleInterval =
197 base::TimeDelta::FromSeconds(kSettleIntervalSeconds); 198 base::TimeDelta::FromSeconds(kSettleIntervalSeconds);
198 199
(...skipping 47 matching lines...) Expand 10 before | Expand all | Expand 10 after
246 247
247 scoped_ptr<PolicyWatcher> PolicyWatcher::Create( 248 scoped_ptr<PolicyWatcher> PolicyWatcher::Create(
248 policy::PolicyService* policy_service, 249 policy::PolicyService* policy_service,
249 scoped_refptr<base::SingleThreadTaskRunner> task_runner) { 250 scoped_refptr<base::SingleThreadTaskRunner> task_runner) {
250 base::FilePath policy_dir(kPolicyDir); 251 base::FilePath policy_dir(kPolicyDir);
251 return make_scoped_ptr(new PolicyWatcherLinux(task_runner, policy_dir)); 252 return make_scoped_ptr(new PolicyWatcherLinux(task_runner, policy_dir));
252 } 253 }
253 254
254 } // namespace policy_hack 255 } // namespace policy_hack
255 } // namespace remoting 256 } // namespace remoting
OLDNEW
« no previous file with comments | « remoting/host/policy_hack/policy_watcher.cc ('k') | remoting/host/policy_hack/policy_watcher_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698