commit f6b387120252a1b89e7afd04af6a05475d99bd79
author Alex Vakulenko <avakulenko@chromium.org> Wed Sep 03 16:23:38 2014 -0700
committer chrome-internal-fetch <chrome-internal-fetch@google.com> Fri Sep 05 01:17:32 2014 +0000
tree 5f52d30c5cbc5284a2576e9be5ac5dc1ef23ffa3
parent f8f10fa8a436ee9c61138cc577cee8e0d3679088

buffet: Move command state into CommandInstance

The command state is now stored in CommandInstance and not in
DBusCommandProxy. CommandInstance can now notify the proxy of
command state changes via CommandProxyInterface.

Moved command status strings from dbus_constants.h into the
CommandInstance class, as members.

Added a property on DBusCommandProxy to expose the command
parameters to command handlers, so they can get the parameter
values over D-Bus.

BUG=chromium:374864
TEST=FEATURES=test emerge-link buffet

Change-Id: Ief3397ef09644772ffc3b1b01ed972a8b6779df4
Reviewed-on: https://chromium-review.googlesource.com/216296
Tested-by: Alex Vakulenko <avakulenko@chromium.org>
Reviewed-by: Christopher Wiley <wiley@chromium.org>
Commit-Queue: Alex Vakulenko <avakulenko@chromium.org>

buffet/commands/dbus_command_proxy.h

diff --git a/buffet/commands/dbus_command_proxy.h b/buffet/commands/dbus_command_proxy.h
index a4b2a53..1221099 100644
--- a/buffet/commands/dbus_command_proxy.h
+++ b/buffet/commands/dbus_command_proxy.h
@@ -9,8 +9,11 @@
 #include <string>
 
 #include <base/basictypes.h>
+#include <chromeos/dbus/data_serialization.h>
 #include <chromeos/dbus/dbus_object.h>
 
+#include "buffet/commands/command_proxy_interface.h"
+
 namespace chromeos {
 namespace dbus_utils {
 class ExportedObjectManager;
@@ -21,17 +24,21 @@
 
 class CommandInstance;
 
-class DBusCommandProxy {
+class DBusCommandProxy : public CommandProxyInterface {
  public:
   DBusCommandProxy(chromeos::dbus_utils::ExportedObjectManager* object_manager,
                    const scoped_refptr<dbus::Bus>& bus,
                    CommandInstance* command_instance);
-  virtual ~DBusCommandProxy() = default;
+  ~DBusCommandProxy() override = default;
 
   void RegisterAsync(
       const chromeos::dbus_utils::AsyncEventSequencer::CompletionAction&
           completion_callback);
 
+  // CommandProxyInterface implementation/overloads.
+  void OnStatusChanged(const std::string& status) override;
+  void OnProgressChanged(int progress) override;
+
  private:
   // DBus properties for org.chromium.Buffet.Command interface.
   chromeos::dbus_utils::ExportedProperty<std::string> name_;
@@ -39,6 +46,8 @@
   chromeos::dbus_utils::ExportedProperty<std::string> id_;
   chromeos::dbus_utils::ExportedProperty<std::string> status_;
   chromeos::dbus_utils::ExportedProperty<int32_t> progress_;
+  chromeos::dbus_utils::ExportedProperty<chromeos::dbus_utils::Dictionary>
+      parameters_;
 
   // Handles calls to org.chromium.Buffet.Command.SetProgress(progress).
   void HandleSetProgress(chromeos::ErrorPtr* error, int32_t progress);
@@ -55,6 +64,7 @@
   chromeos::dbus_utils::DBusObject dbus_object_;
 
   friend class DBusCommandProxyTest;
+  friend class DBusCommandDispacherTest;
   DISALLOW_COPY_AND_ASSIGN(DBusCommandProxy);
 };