Replaced some callbacks with commend SuccessCallback and ErrorCallback

BUG: 24267885
Change-Id: Id2f928d3571ad620ddeef6e98dbc06d8ade514fe
Reviewed-on: https://weave-review.googlesource.com/1157
Reviewed-by: Vitaly Buka <vitalybuka@google.com>
diff --git a/libweave/examples/ubuntu/network_manager.cc b/libweave/examples/ubuntu/network_manager.cc
index c14dac5..fc033b5 100644
--- a/libweave/examples/ubuntu/network_manager.cc
+++ b/libweave/examples/ubuntu/network_manager.cc
@@ -60,8 +60,8 @@
     const std::string& passphrase,
     int pid,
     base::Time until,
-    const base::Closure& success_callback,
-    const base::Callback<void(const Error*)>& error_callback) {
+    const SuccessCallback& success_callback,
+    const ErrorCallback& error_callback) {
   if (pid) {
     int status = 0;
     if (pid == waitpid(pid, &status, WNOWAIT)) {
diff --git a/libweave/examples/ubuntu/network_manager.h b/libweave/examples/ubuntu/network_manager.h
index 10b413a..e59bc96 100644
--- a/libweave/examples/ubuntu/network_manager.h
+++ b/libweave/examples/ubuntu/network_manager.h
@@ -50,8 +50,8 @@
                     const std::string& passphrase,
                     int pid,
                     base::Time until,
-                    const base::Closure& success_callback,
-                    const base::Callback<void(const Error*)>& error_callback);
+                    const SuccessCallback& success_callback,
+                    const ErrorCallback& error_callback);
   void UpdateNetworkState();
 
   bool force_bootstrapping_{false};
diff --git a/libweave/examples/ubuntu/ssl_stream.cc b/libweave/examples/ubuntu/ssl_stream.cc
index 71d5148..a54578d 100644
--- a/libweave/examples/ubuntu/ssl_stream.cc
+++ b/libweave/examples/ubuntu/ssl_stream.cc
@@ -16,8 +16,8 @@
   CancelPendingOperations();
 }
 
-void SSLStream::RunDelayedTask(const base::Closure& success_callback) {
-  success_callback.Run();
+void SSLStream::RunDelayedTask(const base::Closure& task) {
+  task.Run();
 }
 
 void SSLStream::Read(void* buffer,
diff --git a/libweave/include/weave/error.h b/libweave/include/weave/error.h
index b9f07a5..67f923f 100644
--- a/libweave/include/weave/error.h
+++ b/libweave/include/weave/error.h
@@ -127,7 +127,7 @@
 };
 
 using SuccessCallback = base::Closure;
-using ErrorCallback = base::Callback<void(const Error*)>;
+using ErrorCallback = base::Callback<void(const Error* error)>;
 
 }  // namespace weave
 
diff --git a/libweave/include/weave/stream.h b/libweave/include/weave/stream.h
index 99385a7..f36e113 100644
--- a/libweave/include/weave/stream.h
+++ b/libweave/include/weave/stream.h
@@ -18,7 +18,7 @@
   virtual ~InputStream() = default;
 
   // Callbacks types for Read.
-  using ReadSuccessCallback = base::Callback<void(size_t)>;
+  using ReadSuccessCallback = base::Callback<void(size_t size)>;
 
   // Implementation should return immediately and post either success_callback
   // or error_callback. Caller guarantees that buffet is alive until either of
diff --git a/libweave/include/weave/test/mock_wifi_provider.h b/libweave/include/weave/test/mock_wifi_provider.h
index 1bea0ed..408e0ad 100644
--- a/libweave/include/weave/test/mock_wifi_provider.h
+++ b/libweave/include/weave/test/mock_wifi_provider.h
@@ -20,7 +20,7 @@
                void(const std::string&,
                     const std::string&,
                     const base::Closure&,
-                    const base::Callback<void(const Error*)>&));
+                    const ErrorCallback&));
   MOCK_METHOD1(StartAccessPoint, void(const std::string&));
   MOCK_METHOD0(StopAccessPoint, void());
 };
diff --git a/libweave/src/privet/cloud_delegate.cc b/libweave/src/privet/cloud_delegate.cc
index 7d01088..03e74c7 100644
--- a/libweave/src/privet/cloud_delegate.cc
+++ b/libweave/src/privet/cloud_delegate.cc
@@ -83,7 +83,7 @@
   void UpdateDeviceInfo(const std::string& name,
                         const std::string& description,
                         const std::string& location,
-                        const base::Closure& success_callback,
+                        const SuccessCallback& success_callback,
                         const ErrorCallback& error_callback) override {
     ErrorPtr error;
     if (!device_->UpdateDeviceInfo(name, description, location, &error))
@@ -159,7 +159,7 @@
 
   void AddCommand(const base::DictionaryValue& command,
                   const UserInfo& user_info,
-                  const SuccessCallback& success_callback,
+                  const CommandSuccessCallback& success_callback,
                   const ErrorCallback& error_callback) override {
     CHECK(user_info.scope() != AuthScope::kNone);
     CHECK_NE(user_info.user_id(), 0u);
@@ -184,7 +184,7 @@
 
   void GetCommand(const std::string& id,
                   const UserInfo& user_info,
-                  const SuccessCallback& success_callback,
+                  const CommandSuccessCallback& success_callback,
                   const ErrorCallback& error_callback) override {
     CHECK(user_info.scope() != AuthScope::kNone);
     ErrorPtr error;
@@ -196,7 +196,7 @@
 
   void CancelCommand(const std::string& id,
                      const UserInfo& user_info,
-                     const SuccessCallback& success_callback,
+                     const CommandSuccessCallback& success_callback,
                      const ErrorCallback& error_callback) override {
     CHECK(user_info.scope() != AuthScope::kNone);
     ErrorPtr error;
@@ -209,7 +209,7 @@
   }
 
   void ListCommands(const UserInfo& user_info,
-                    const SuccessCallback& success_callback,
+                    const CommandSuccessCallback& success_callback,
                     const ErrorCallback& error_callback) override {
     CHECK(user_info.scope() != AuthScope::kNone);
 
diff --git a/libweave/src/privet/cloud_delegate.h b/libweave/src/privet/cloud_delegate.h
index f96da19..0b72a44 100644
--- a/libweave/src/privet/cloud_delegate.h
+++ b/libweave/src/privet/cloud_delegate.h
@@ -36,8 +36,8 @@
   CloudDelegate();
   virtual ~CloudDelegate();
 
-  using SuccessCallback = base::Callback<void(const base::DictionaryValue&)>;
-  using ErrorCallback = base::Callback<void(Error*)>;
+  using CommandSuccessCallback =
+      base::Callback<void(const base::DictionaryValue& commands)>;
 
   class Observer {
    public:
@@ -64,7 +64,7 @@
   virtual void UpdateDeviceInfo(const std::string& name,
                                 const std::string& description,
                                 const std::string& location,
-                                const base::Closure& success_callback,
+                                const SuccessCallback& success_callback,
                                 const ErrorCallback& error_callback) = 0;
 
   // Returns the name of the maker.
@@ -103,24 +103,24 @@
   // Adds command created from the given JSON representation.
   virtual void AddCommand(const base::DictionaryValue& command,
                           const UserInfo& user_info,
-                          const SuccessCallback& success_callback,
+                          const CommandSuccessCallback& success_callback,
                           const ErrorCallback& error_callback) = 0;
 
   // Returns command with the given ID.
   virtual void GetCommand(const std::string& id,
                           const UserInfo& user_info,
-                          const SuccessCallback& success_callback,
+                          const CommandSuccessCallback& success_callback,
                           const ErrorCallback& error_callback) = 0;
 
   // Cancels command with the given ID.
   virtual void CancelCommand(const std::string& id,
                              const UserInfo& user_info,
-                             const SuccessCallback& success_callback,
+                             const CommandSuccessCallback& success_callback,
                              const ErrorCallback& error_callback) = 0;
 
   // Lists commands.
   virtual void ListCommands(const UserInfo& user_info,
-                            const SuccessCallback& success_callback,
+                            const CommandSuccessCallback& success_callback,
                             const ErrorCallback& error_callback) = 0;
 
   void AddObserver(Observer* observer) { observer_list_.AddObserver(observer); }
diff --git a/libweave/src/privet/mock_delegates.h b/libweave/src/privet/mock_delegates.h
index 930d838..8d29a1e 100644
--- a/libweave/src/privet/mock_delegates.h
+++ b/libweave/src/privet/mock_delegates.h
@@ -145,7 +145,7 @@
                void(const std::string&,
                     const std::string&,
                     const std::string&,
-                    const base::Closure&,
+                    const SuccessCallback&,
                     const ErrorCallback&));
   MOCK_CONST_METHOD0(GetOemName, std::string());
   MOCK_CONST_METHOD0(GetModelName, std::string());
@@ -160,21 +160,21 @@
   MOCK_METHOD4(AddCommand,
                void(const base::DictionaryValue&,
                     const UserInfo&,
-                    const SuccessCallback&,
+                    const CommandSuccessCallback&,
                     const ErrorCallback&));
   MOCK_METHOD4(GetCommand,
                void(const std::string&,
                     const UserInfo&,
-                    const SuccessCallback&,
+                    const CommandSuccessCallback&,
                     const ErrorCallback&));
   MOCK_METHOD4(CancelCommand,
                void(const std::string&,
                     const UserInfo&,
-                    const SuccessCallback&,
+                    const CommandSuccessCallback&,
                     const ErrorCallback&));
   MOCK_METHOD3(ListCommands,
                void(const UserInfo&,
-                    const SuccessCallback&,
+                    const CommandSuccessCallback&,
                     const ErrorCallback&));
 
   MockCloudDelegate() {
diff --git a/libweave/src/privet/privet_handler.cc b/libweave/src/privet/privet_handler.cc
index 63cb9b0..b91d8e3 100644
--- a/libweave/src/privet/privet_handler.cc
+++ b/libweave/src/privet/privet_handler.cc
@@ -212,7 +212,7 @@
 }
 
 void OnCommandRequestFailed(const PrivetHandler::RequestCallback& callback,
-                            Error* error) {
+                            const Error* error) {
   if (error->HasError("gcd", "unknown_command")) {
     ErrorPtr new_error = error->Clone();
     Error::AddTo(&new_error, FROM_HERE, errors::kDomain, errors::kNotFound,