Send policy blobs to session_manager

chrome/browser/chromeos/login/signed_settings_helper.cc

 // Copyright (c) 2010 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/chromeos/login/signed_settings_helper.h"
 
 #include <string>
 #include <vector>
 
 #include "base/lazy_instance.h"
@@ skipping 36 matching lines @@
 
   // Cancels the callback.
   void CancelCallback() {
     callback_ = NULL;
   }
 
   // Cancels the callback and cancels the op if it is not executing.
   void Cancel() {
     CancelCallback();
 
-   if (!executing_)
+    if (!executing_)
       OnOpCompleted();
   }
 
   // Accessors.
   SignedSettings* op() const {
     return op_.get();
   }
 
   SignedSettingsHelper::Callback* callback() const {
     return callback_;
@@ skipping 154 matching lines @@
   virtual void CreateOp() {
     op_ = SignedSettings::CreateRetrievePropertyOp(name_, this);
   }
 
  private:
   std::string name_;
 
   DISALLOW_COPY_AND_ASSIGN(RetrievePropertyOpContext);
 };
 
+class StorePolicyOpContext : public SignedSettings::Delegate<bool>,
+                             public OpContext {
+ public:
+  StorePolicyOpContext(const std::string& value,
+                       SignedSettingsHelper::Callback* callback,
+                       Delegate* delegate)
+      : OpContext(callback, delegate),
+        value_(value) {
+  }
+
+  // chromeos::SignedSettings::Delegate implementation
+  virtual void OnSettingsCompleted(SignedSettings::ReturnCode code,

[Chris Masone, 2011/03/23 23:03:53]

On SettingsOpCompleted, no?

[Mattias Nissler (ping if slow), 2011/03/24 18:55:57]

Maybe this could use an OVERRIDE declaration then ;)

[Jakob Kummerow, 2011/03/28 13:53:53]

Most definitely. Good catch. Done.

[Jakob Kummerow, 2011/03/28 13:53:53]

Done.

+                                   bool unused) {
+    VLOG(2) << "OnSettingsOpCompleted, code = " << code;
+    if (callback_)
+      callback_->OnStorePolicyCompleted(code);
+    OnOpCompleted();
+  }
+
+ protected:
+  // OpContext implementation
+  virtual void CreateOp() {
+    op_ = SignedSettings::CreateStorePolicyOp(value_, this);
+  }
+
+ private:
+  std::string value_;
+  DISALLOW_COPY_AND_ASSIGN(StorePolicyOpContext);
+};
+
+class RetrievePolicyOpContext : public SignedSettings::Delegate<std::string>,
+                                public OpContext {
+ public:
+  RetrievePolicyOpContext(SignedSettingsHelper::Callback* callback,
+                          Delegate* delegate)
+      : OpContext(callback, delegate) {
+  }
+
+  // chromeos::SignedSettings::Delegate implementation
+  virtual void OnSettingsCompleted(SignedSettings::ReturnCode code,

[Chris Masone, 2011/03/23 23:03:53]

Again

[Jakob Kummerow, 2011/03/28 13:53:53]

Done.

+                                   std::string value) {
+    if (callback_)
+      callback_->OnRetrievePolicyCompleted(code, value);
+    OnOpCompleted();
+  }
+
+ protected:
+  // OpContext implementation
+  virtual void CreateOp() {
+    op_ = SignedSettings::CreateRetrievePolicyOp(this);
+  }
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(RetrievePolicyOpContext);
+};
+
 }  // namespace
 
 
 class SignedSettingsHelperImpl : public SignedSettingsHelper,
                                  public OpContext::Delegate {
  public:
   // SignedSettingsHelper implementation
   virtual void StartCheckWhitelistOp(const std::string& email,
                                      Callback* callback);
   virtual void StartWhitelistOp(const std::string& email,
                                 bool add_to_whitelist,
                                 Callback* callback);
   virtual void StartStorePropertyOp(const std::string& name,
                                     const std::string& value,
                                     Callback* callback);
   virtual void StartRetrieveProperty(const std::string& name,
                                      Callback* callback);
+  virtual void StartStorePolicyOp(const std::string& value,
+                                  Callback* callback);
+  virtual void StartRetrievePolicyOp(Callback* callback);
   virtual void CancelCallback(Callback* callback);
 
   // OpContext::Delegate implementation
   virtual void OnOpCreated(OpContext* context);
   virtual void OnOpStarted(OpContext* context);
   virtual void OnOpCompleted(OpContext* context);
 
  private:
   SignedSettingsHelperImpl();
   ~SignedSettingsHelperImpl();
@@ skipping 38 matching lines @@
   AddOpContext(new WhitelistOpContext(
       add_to_whitelist ? WhitelistOpContext::ADD : WhitelistOpContext::REMOVE,
       email,
       callback,
       this));
 }
 
 void SignedSettingsHelperImpl::StartStorePropertyOp(
     const std::string& name,
     const std::string& value,
-    SignedSettingsHelper::SignedSettingsHelper::Callback* callback) {
+    SignedSettingsHelper::Callback* callback) {
   AddOpContext(new StorePropertyOpContext(
       name,
       value,
       callback,
       this));
 }
 
 void SignedSettingsHelperImpl::StartRetrieveProperty(
     const std::string& name,
     SignedSettingsHelper::Callback* callback) {
   AddOpContext(new RetrievePropertyOpContext(
       name,
       callback,
       this));
 }
 
+void SignedSettingsHelperImpl::StartStorePolicyOp(
+    const std::string& value,
+    SignedSettingsHelper::Callback* callback) {
+  AddOpContext(new StorePolicyOpContext(value, callback, this));
+}
+
+void SignedSettingsHelperImpl::StartRetrievePolicyOp(
+    SignedSettingsHelper::Callback* callback) {
+  AddOpContext(new RetrievePolicyOpContext(callback, this));
+}
+
 void SignedSettingsHelperImpl::CancelCallback(
     SignedSettingsHelper::Callback* callback) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
 
   for (size_t i = 0; i < pending_contexts_.size(); ++i) {
     if (pending_contexts_[i]->callback() == callback) {
       pending_contexts_[i]->CancelCallback();
     }
   }
 }
@@ skipping 37 matching lines @@
 
   if (test_delegate_)
     test_delegate_->OnOpCompleted(context->op());
 }
 
 SignedSettingsHelper* SignedSettingsHelper::Get() {
   return g_signed_settings_helper_impl.Pointer();
 }
 
 }  // namespace chromeos