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/command_queue.cc

diff --git a/buffet/commands/command_queue.cc b/buffet/commands/command_queue.cc
index 557a303..8583921 100644
--- a/buffet/commands/command_queue.cc
+++ b/buffet/commands/command_queue.cc
@@ -10,6 +10,7 @@
 std::string CommandQueue::Add(std::unique_ptr<CommandInstance> instance) {
   std::string id = std::to_string(++next_id_);
   instance->SetID(id);
+  instance->SetCommandQueue(this);
   auto pair = map_.insert(std::make_pair(id, std::move(instance)));
   LOG_IF(FATAL, !pair.second) << "Command with ID '" << id
                               << "' is already in the queue";
@@ -25,6 +26,7 @@
   auto p = map_.find(id);
   if (p != map_.end()) {
     instance = std::move(p->second);
+    instance->SetCommandQueue(nullptr);
     map_.erase(p);
     if (dispatch_interface_)
       dispatch_interface_->OnCommandRemoved(instance.get());