commit 5191e81a2a81aaf57acea001a9c45b5186604bbc
author Anton Muhin <antonm@google.com> Thu Oct 30 17:49:48 2014 +0400
committer chrome-internal-fetch <chrome-internal-fetch@google.com> Fri Nov 07 06:34:01 2014 +0000
tree 0ad36854415e650efafb83d23096699ef48e341b
parent 5c7bf01d22935ebf669aa85a615923ebc420c1a9

buffet: Separate command IDs from DBus paths.

As we're starting to support cloud and local cases, we need proper
support for cloud command IDs.  Therefore now we require each
command instance to have some command id, but generate dbus path
independently.

BUG=None
TEST=cros_workon_make buffet --test&&manual

Change-Id: I83ae92872d920c8e0d6e15324ad11feebfd1f540
Reviewed-on: https://chromium-review.googlesource.com/226532
Tested-by: Anton Muhin <antonm@chromium.org>
Reviewed-by: Alex Vakulenko <avakulenko@chromium.org>
Commit-Queue: Anton Muhin <antonm@chromium.org>

buffet/manager.cc

diff --git a/buffet/manager.cc b/buffet/manager.cc
index 7deb36f..9c1cd94 100644
--- a/buffet/manager.cc
+++ b/buffet/manager.cc
@@ -167,6 +167,7 @@
 
 void Manager::HandleAddCommand(scoped_ptr<DBusMethodResponse> response,
                                const std::string& json_command) {
+  static int next_id = 0;
   std::string error_message;
   std::unique_ptr<base::Value> value(base::JSONReader::ReadAndReturnError(
       json_command, base::JSON_PARSE_RFC, nullptr, &error_message));
@@ -183,6 +184,7 @@
     response->ReplyWithError(error.get());
     return;
   }
+  command_instance->SetID(std::to_string(++next_id));
   command_manager_->AddCommand(std::move(command_instance));
   response->Return();
 }