Implement cloud policy invalidations using the invalidation service framework.

chrome/browser/policy/cloud/cloud_policy_store.h

 // Copyright (c) 2012 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.
 
 #ifndef CHROME_BROWSER_POLICY_CLOUD_CLOUD_POLICY_STORE_H_
 #define CHROME_BROWSER_POLICY_CLOUD_CLOUD_POLICY_STORE_H_
 
 #include "base/basictypes.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/observer_list.h"
@@ skipping 55 matching lines @@
   }
   bool is_managed() const {
     return policy_.get() &&
            policy_->state() == enterprise_management::PolicyData::ACTIVE;
   }
   Status status() const { return status_; }
   CloudPolicyValidatorBase::Status validation_status() const {
     return validation_status_;
   }
 
+  // Returns true if the latest policy loaded was different from the previous
+  // policy.
+  bool policy_changed() const {

[Mattias Nissler (ping if slow), 2013/08/28 14:20:40]

AFAICS, the only user of this is CloudPolicyInvalidator::GetRefreshMetric. Thus,
it'd be preferable to do change detection there instead of complicating the
public interface of CloudPolicyStore.

[Steve Condie, 2013/09/07 01:11:20]

Opened bug 286613 to track refactoring the policy change code as suggested by
this comment.

I believe all the remaining comments from Mattias have been addressed in
https://codereview.chromium.org/23592017/

+    return policy_changed_;
+  }
+
   // Store a new policy blob. Pending load/store operations will be canceled.
   // The store operation may proceed asynchronously and observers are notified
   // once the operation finishes. If successful, OnStoreLoaded() will be invoked
   // on the observers and the updated policy can be read through policy().
   // Errors generate OnStoreError() notifications.
+  // |invalidation_version| is the invalidation version of the policy to be
+  // stored.
+  void Store(
+      const enterprise_management::PolicyFetchResponse& policy,
+      int64 invalidation_version);
+
   virtual void Store(
       const enterprise_management::PolicyFetchResponse& policy) = 0;
 
   // Load the current policy blob from persistent storage. Pending load/store
   // operations will be canceled. This may trigger asynchronous operations.
   // Upon success, OnStoreLoaded() will be called on the registered observers.
   // Otherwise, OnStoreError() reports the reason for failure.
   virtual void Load() = 0;
 
   // Registers an observer to be notified when policy changes.
   void AddObserver(Observer* observer);
 
   // Removes the specified observer.
   void RemoveObserver(Observer* observer);
 
+  // The invalidation version of the last policy stored. This value can be read
+  // by observers to determine which version of the policy is now available.
+  int64 invalidation_version() {
+    return invalidation_version_;
+  }
+
  protected:
   // Invokes the corresponding callback on all registered observers.
   void NotifyStoreLoaded();
   void NotifyStoreError();
 
   // Decoded version of the currently effective policy.
   PolicyMap policy_map_;
 
   // Currently effective policy.
   scoped_ptr<enterprise_management::PolicyData> policy_;
 
   // Latest status code.
   Status status_;
 
   // Latest validation status.
   CloudPolicyValidatorBase::Status validation_status_;
 
+  // The invalidation version of the last policy stored.
+  int64 invalidation_version_;
+
  private:
   // Whether the store has completed asynchronous initialization, which is
   // triggered by calling Load().
   bool is_initialized_;
 
+  // Whether latest policy loaded was different from the previous policy.
+  bool policy_changed_;
+
+  // The hash value of the current policy. This is used to determine when the
+  // policy changes.
+  uint32 hash_value_;
+
   ObserverList<Observer, true> observers_;
 
   DISALLOW_COPY_AND_ASSIGN(CloudPolicyStore);
 };
 
 }  // namespace policy
 
 #endif  // CHROME_BROWSER_POLICY_CLOUD_CLOUD_POLICY_STORE_H_