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

Unified Diff: chrome/browser/extensions/api/storage/policy_value_store.cc

Issue 60823003: Introduced a ForwardingPolicyProvider. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fixed ios Created 7 years, 1 month 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/storage/policy_value_store.cc
diff --git a/chrome/browser/extensions/api/storage/policy_value_store.cc b/chrome/browser/extensions/api/storage/policy_value_store.cc
index 08833fd3c3f1c5aac74eff83c17f43684b925a46..e2fc1cc1bb6f407f102cbf571634fb01e7e6cda5 100644
--- a/chrome/browser/extensions/api/storage/policy_value_store.cc
+++ b/chrome/browser/extensions/api/storage/policy_value_store.cc
@@ -38,8 +38,7 @@ PolicyValueStore::PolicyValueStore(
PolicyValueStore::~PolicyValueStore() {}
-void PolicyValueStore::SetCurrentPolicy(const policy::PolicyMap& policy,
- bool notify_if_changed) {
+void PolicyValueStore::SetCurrentPolicy(const policy::PolicyMap& policy) {
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE));
// Convert |policy| to a dictionary value. Only include mandatory policies
// for now.
@@ -94,7 +93,7 @@ void PolicyValueStore::SetCurrentPolicy(const policy::PolicyMap& policy,
changes.end(), result->changes().begin(), result->changes().end());
}
- if (!changes.empty() && notify_if_changed) {
+ if (!changes.empty()) {
observers_->Notify(
&SettingsObserver::OnSettingsChanged,
extension_id_,

Powered by Google App Engine
This is Rietveld 408576698