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

Unified Diff: chrome/browser/policy/cloud/user_cloud_policy_manager_factory.h

Issue 2606773002: Setup Chromad user policy plumbing (Closed)
Patch Set: Rebase Created 4 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/policy/cloud/user_cloud_policy_manager_factory.h
diff --git a/chrome/browser/policy/cloud/user_cloud_policy_manager_factory.h b/chrome/browser/policy/cloud/user_cloud_policy_manager_factory.h
index 49c7126987cd3cb6e3c3e5442de394a5d9c430d6..f96617c128635c46787bde7ed0675ffbc8d6bb23 100644
--- a/chrome/browser/policy/cloud/user_cloud_policy_manager_factory.h
+++ b/chrome/browser/policy/cloud/user_cloud_policy_manager_factory.h
@@ -29,16 +29,16 @@ class UserCloudPolicyManager;
// instances that initialize per-profile cloud policy settings on the desktop
// platforms.
//
-// UserCloudPolicyManager is handled different than other
-// KeyedServices because it is a dependency of PrefService.
-// Therefore, lifetime of instances is managed by Profile, Profile startup code
-// invokes CreateForBrowserContext() explicitly, takes ownership, and the
-// instance is only deleted after PrefService destruction.
+// UserCloudPolicyManager is handled different than other KeyedServices because
+// it is a dependency of PrefService. Therefore, lifetime of instances is
+// managed by Profile, Profile startup code invokes CreateForBrowserContext()
+// explicitly, takes ownership, and the instance is only deleted after
+// PrefService destruction.
//
// TODO(mnissler): Remove the special lifetime management in favor of
// PrefService directly depending on UserCloudPolicyManager once the former has
-// been converted to a KeyedService.
-// See also http://crbug.com/131843 and http://crbug.com/131844.
+// been converted to a KeyedService. See also https://crbug.com/131843 and
+// https://crbug.com/131844.
class UserCloudPolicyManagerFactory : public BrowserContextKeyedBaseFactory {
public:
// Returns an instance of the UserCloudPolicyManagerFactory singleton.
@@ -51,8 +51,8 @@ class UserCloudPolicyManagerFactory : public BrowserContextKeyedBaseFactory {
// Creates an instance for |context|. Note that the caller is responsible for
// managing the lifetime of the instance. Subsequent calls to
// GetForBrowserContext() will return the created instance as long as it
- // lives. If RegisterTestingFactory() has been called, then calls to
- // this method will return null.
+ // lives. If RegisterTestingFactory() has been called, then calls to this
+ // method will return null.
//
// If |force_immediate_load| is true, policy is loaded synchronously from
// UserCloudPolicyStore at startup.
« no previous file with comments | « chrome/browser/policy/cloud/user_cloud_policy_invalidator_factory.cc ('k') | chrome/browser/policy/profile_policy_connector.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698