Added code to persist downloaded cloud policy to disk.

chrome/browser/policy/user_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_USER_CLOUD_POLICY_STORE_H_
 #define CHROME_BROWSER_POLICY_USER_CLOUD_POLICY_STORE_H_
 
 #include <string>
 
 #include "base/basictypes.h"
 #include "base/compiler_specific.h"
+#include "base/memory/ref_counted.h"
 #include "base/memory/weak_ptr.h"
 #include "chrome/browser/policy/user_cloud_policy_store_base.h"
+#include "chrome/browser/policy/user_policy_disk_cache.h"

[Mattias Nissler (ping if slow), 2012/08/20 15:29:30]

This is a part of the chrome OS user policy legacy that should eventually go
away. It uses a wrapper protobuf that is no longer need: Just storing
PolicyFetchResponse is enough as it contains everything we need, and has the
additional benefit that all data contained in it can be protected by a
signature. Thus, you may want to reconsider using UserPolicyDiskCache.

[Andrew T Wilson (Slow), 2012/08/20 18:04:37]

OK, I'll roll my own version that doesn't use the wrapper protobuf.

 
 namespace policy {
 
 // Implements a cloud policy store that is stored in a simple file in the user's
 // profile directory. This is used on (non-chromeos) platforms that do not have
 // a secure storage implementation.
-class UserCloudPolicyStore : public UserCloudPolicyStoreBase {
+class UserCloudPolicyStore : public UserCloudPolicyStoreBase,
+                             public UserPolicyDiskCache::Delegate {
  public:
   // Creates a policy store associated with the user signed in to this
   // |profile|.
-  explicit UserCloudPolicyStore(Profile* profile);
+  explicit UserCloudPolicyStore(Profile* profile,
+                                const FilePath& policy_file);
   virtual ~UserCloudPolicyStore();
 
   // CloudPolicyStore implementation.
   virtual void Load() OVERRIDE;
   virtual void Store(
       const enterprise_management::PolicyFetchResponse& policy) OVERRIDE;
 
+  // UserPolicyDiskCache::Delegate implementation.
+  virtual void OnDiskCacheLoaded(
+      UserPolicyDiskCache::LoadResult result,
+      const enterprise_management::CachedCloudPolicyResponse& policy) OVERRIDE;
+

[Mattias Nissler (ping if slow), 2012/08/20 15:29:30]

remove extra blank line

[Andrew T Wilson (Slow), 2012/08/21 23:18:19]

Done.

+
  private:
   // Starts policy blob validation. |callback| is invoked once validation is
   // complete.
   void Validate(
       scoped_ptr<enterprise_management::PolicyFetchResponse> policy,
       const UserCloudPolicyValidator::CompletionCallback& callback);
 
+  // Callback invoked to install a just-loaded policy after validation has
+  // finished.
+  void InstallLoadedPolicyAfterValidation(UserCloudPolicyValidator* validator);
+
   // Callback invoked to store the policy after validation has finished.
   void StorePolicyAfterValidation(UserCloudPolicyValidator* validator);
 
+  // WeakPtrFactory used to create callbacks for validating and storing policy.
   base::WeakPtrFactory<UserCloudPolicyStore> weak_factory_;
 
+  // Helper object used to read/write policy to disk.
+  scoped_refptr<UserPolicyDiskCache> policy_cache_;
+
   // Weak pointer to the profile associated with this store.
   Profile* profile_;
 
   DISALLOW_COPY_AND_ASSIGN(UserCloudPolicyStore);
 };
 
 }  // namespace policy
 
 #endif  // CHROME_BROWSER_POLICY_USER_CLOUD_POLICY_STORE_H_