commit 79b50e7e0e57bc815c694afd20d8e9477a4fbb24
author Vitaly Buka <vitalybuka@chromium.org> Tue Jun 23 15:39:58 2015 -0700
committer ChromeOS Commit Bot <chromeos-commit-bot@chromium.org> Wed Jun 24 01:46:34 2015 +0000
tree c561a7fc1f29b9214ef6485f887c4ee0ca38408b
parent 2c4178e1bc291b79565282cd85bc1055e7b424bc

buffet: Fixed listing of cloud commands on local API

By mistake CloudDelegateImpl sign-up OnCommandAdded and OnCommandRemoved
for the same event. As result command was added to the list and
removed from immediately.

BUG=brillo:1219
TEST=Add command using cloud, list command using local API

Change-Id: I5164017c3ee6004ca5d3a72f5c050beacf15a74f
Reviewed-on: https://chromium-review.googlesource.com/281340
Commit-Queue: Vitaly Buka <vitalybuka@chromium.org>
Trybot-Ready: Vitaly Buka <vitalybuka@chromium.org>
Tested-by: Vitaly Buka <vitalybuka@chromium.org>
Reviewed-by: Alex Vakulenko <avakulenko@chromium.org>

buffet/privet/cloud_delegate.cc

diff --git a/buffet/privet/cloud_delegate.cc b/buffet/privet/cloud_delegate.cc
index 3c0449c..bc23f5e 100644
--- a/buffet/privet/cloud_delegate.cc
+++ b/buffet/privet/cloud_delegate.cc
@@ -60,7 +60,7 @@
         &CloudDelegateImpl::OnCommandDefChanged, weak_factory_.GetWeakPtr()));
     command_manager_->AddOnCommandAddedCallback(base::Bind(
         &CloudDelegateImpl::OnCommandAdded, weak_factory_.GetWeakPtr()));
-    command_manager_->AddOnCommandAddedCallback(base::Bind(
+    command_manager_->AddOnCommandRemovedCallback(base::Bind(
         &CloudDelegateImpl::OnCommandRemoved, weak_factory_.GetWeakPtr()));
 
     state_manager_->AddOnChangedCallback(base::Bind(