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

Unified Diff: chrome/browser/extensions/api/settings_private/prefs_util.cc

Issue 1342033002: Add tooltips to cr-policy-indicator (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@issue_521791_settings_policy_1
Patch Set: . Created 5 years, 3 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/extensions/api/settings_private/prefs_util.cc
diff --git a/chrome/browser/extensions/api/settings_private/prefs_util.cc b/chrome/browser/extensions/api/settings_private/prefs_util.cc
index 2b7c747f89984d119d1d377852fb6a93b2506a4c..22329b6e384eec2812fc54824a0ee0f202c48b3a 100644
--- a/chrome/browser/extensions/api/settings_private/prefs_util.cc
+++ b/chrome/browser/extensions/api/settings_private/prefs_util.cc
@@ -14,6 +14,7 @@
#include "extensions/browser/extension_pref_value_map.h"
#include "extensions/browser/extension_pref_value_map_factory.h"
#include "extensions/browser/extension_registry.h"
+#include "extensions/common/extension.h"
#if defined(OS_CHROMEOS)
#include "chrome/browser/chromeos/ownership/owner_settings_service_chromeos.h"
@@ -189,6 +190,8 @@ scoped_ptr<settings_private::PrefObject> PrefsUtil::GetPref(
settings_private::PolicySource::POLICY_SOURCE_PRIMARY_USER;
pref_object->policy_enforcement =
settings_private::PolicyEnforcement::POLICY_ENFORCEMENT_ENFORCED;
+ pref_object->policy_source_name.reset(new std::string(
+ user_manager::UserManager::Get()->GetPrimaryUser()->email()));
return pref_object.Pass();
}
if (IsPrefEnterpriseManaged(name)) {
@@ -229,20 +232,27 @@ scoped_ptr<settings_private::PrefObject> PrefsUtil::GetPref(
settings_private::PolicySource::POLICY_SOURCE_OWNER;
pref_object->policy_enforcement =
settings_private::PolicyEnforcement::POLICY_ENFORCEMENT_ENFORCED;
+ pref_object->policy_source_name.reset(
+ new std::string(user_manager::UserManager::Get()->GetOwnerEmail()));
return pref_object.Pass();
}
#endif
if (pref && pref->IsExtensionControlled()) {
- pref_object->policy_source =
- settings_private::PolicySource::POLICY_SOURCE_EXTENSION;
- pref_object->policy_enforcement =
- settings_private::PolicyEnforcement::POLICY_ENFORCEMENT_ENFORCED;
std::string extension_id =
ExtensionPrefValueMapFactory::GetForBrowserContext(profile_)
->GetExtensionControllingPref(pref->name());
- pref_object->extension_id.reset(new std::string(extension_id));
- return pref_object.Pass();
+ const Extension* extension = ExtensionRegistry::Get(profile_)->
+ GetExtensionById(extension_id, ExtensionRegistry::ENABLED);
+ if (extension) {
+ pref_object->policy_source =
+ settings_private::PolicySource::POLICY_SOURCE_EXTENSION;
+ pref_object->policy_enforcement =
+ settings_private::PolicyEnforcement::POLICY_ENFORCEMENT_ENFORCED;
+ pref_object->extension_id.reset(new std::string(extension_id));
+ pref_object->policy_source_name.reset(new std::string(extension->name()));
+ return pref_object.Pass();
+ }
}
if (pref && (!pref->IsUserModifiable() || IsPrefSupervisorControlled(name))) {
// TODO(stevenjb): Investigate whether either of these should be badged.

Powered by Google App Engine
This is Rietveld 408576698