Fixes and improvements to cloud policy for extensions.

chrome/browser/policy/forwarding_policy_provider.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/policy/forwarding_policy_provider.h"
 
 #include "components/policy/core/common/schema_map.h"
 #include "components/policy/core/common/schema_registry.h"
 
 namespace policy {
@@ skipping 42 matching lines @@
     state_ = READY;
     OnUpdatePolicy(delegate_);
     return;
   }
 
   state_ = WAITING_FOR_REFRESH;
   RefreshPolicies();
 }
 
 void ForwardingPolicyProvider::OnSchemaRegistryUpdated(bool has_new_schemas) {
-  if (!has_new_schemas || state_ != READY)
+  if (state_ != READY)
     return;
-  RefreshPolicies();
+  if (has_new_schemas) {
+    RefreshPolicies();
+  } else {
+    // Remove the policies that were being served for the component that have
+    // been removed. This is important so that update notifications are also
+    // sent in case those component are reinstalled during the current session.
+    OnUpdatePolicy(delegate_);
+  }
 }
 
 void ForwardingPolicyProvider::OnUpdatePolicy(
     ConfigurationPolicyProvider* provider) {
   DCHECK_EQ(delegate_, provider);
 
   if (state_ == WAITING_FOR_REFRESH)
     state_ = READY;
 
   scoped_ptr<PolicyBundle> bundle(new PolicyBundle());
   if (state_ == READY) {
     bundle->CopyFrom(delegate_->policies());
     schema_map()->FilterBundle(bundle.get());
   } else {
     // Always forward the Chrome policy, even if the components are not ready
     // yet.
     const PolicyNamespace chrome_ns(POLICY_DOMAIN_CHROME, "");
     bundle->Get(chrome_ns).CopyFrom(delegate_->policies().Get(chrome_ns));
   }
 
   UpdatePolicy(bundle.Pass());
 }
 
 }  // namespace policy