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

Unified Diff: chrome/browser/policy/async_policy_provider.cc

Issue 15061007: Added a PolicyDomainDescriptor. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: addressed comments, rebase on new PolicySchema Created 7 years, 7 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
Index: chrome/browser/policy/async_policy_provider.cc
diff --git a/chrome/browser/policy/async_policy_provider.cc b/chrome/browser/policy/async_policy_provider.cc
index 26b61b3cee3186155c49c0e73046e51db56cbce3..ae93d562a4bf4c47308ec0f86fe272acbbe6560b 100644
--- a/chrome/browser/policy/async_policy_provider.cc
+++ b/chrome/browser/policy/async_policy_provider.cc
@@ -79,13 +79,24 @@ void AsyncPolicyProvider::RefreshPolicies() {
// refresh task has been posted, it is invalidated now.
refresh_callback_.Reset(
base::Bind(&AsyncPolicyProvider::ReloadAfterRefreshSync,
- base::Unretained(this)));
+ weak_factory_.GetWeakPtr()));
BrowserThread::PostTaskAndReply(
BrowserThread::FILE, FROM_HERE,
base::Bind(base::DoNothing),
refresh_callback_.callback());
}
+void AsyncPolicyProvider::RegisterPolicyDomain(
+ scoped_refptr<const PolicyDomainDescriptor> descriptor) {
+ if (loader_) {
+ BrowserThread::PostTask(BrowserThread::FILE,
+ FROM_HERE,
+ base::Bind(&AsyncPolicyLoader::RegisterPolicyDomain,
+ base::Unretained(loader_),
+ descriptor));
+ }
+}
+
void AsyncPolicyProvider::ReloadAfterRefreshSync() {
DCHECK(CalledOnValidThread());
// This task can only enter if it was posted from RefreshPolicies(), and it

Powered by Google App Engine
This is Rietveld 408576698