Remove weave::Cloud::GetDeviceInfo and OnCloudRequestErrorCallback

BUG:24267885
Change-Id: Ic13d0ca757c8ec9c9bde332e430288f1e45e28ba
Reviewed-on: https://weave-review.googlesource.com/1217
Reviewed-by: Alex Vakulenko <avakulenko@google.com>
diff --git a/libweave/include/weave/cloud.h b/libweave/include/weave/cloud.h
index d3c9500..0ef4c14 100644
--- a/libweave/include/weave/cloud.h
+++ b/libweave/include/weave/cloud.h
@@ -26,21 +26,11 @@
  public:
   using OnRegistrationChangedCallback =
       base::Callback<void(RegistrationStatus satus)>;
-  using OnCloudRequestCallback =
-      base::Callback<void(const base::DictionaryValue& response)>;
-  using OnCloudRequestErrorCallback = base::Callback<void(const Error* error)>;
 
   // Sets callback which is called when registration state is changed.
   virtual void AddOnRegistrationChangedCallback(
       const OnRegistrationChangedCallback& callback) = 0;
 
-  // Gets the full device description JSON object asynchronously.
-  // Passes the device info as the first argument to |callback|, or nullptr if
-  // the device is not registered or in case of communication failure.
-  virtual void GetDeviceInfo(
-      const OnCloudRequestCallback& success_callback,
-      const OnCloudRequestErrorCallback& error_callback) = 0;
-
   // Registers the device.
   // Returns a device ID on success.
   virtual std::string RegisterDevice(const std::string& ticket_id,
diff --git a/libweave/src/device_registration_info.cc b/libweave/src/device_registration_info.cc
index a011826..1e5566a 100644
--- a/libweave/src/device_registration_info.cc
+++ b/libweave/src/device_registration_info.cc
@@ -337,7 +337,7 @@
 
 void DeviceRegistrationInfo::RefreshAccessToken(
     const base::Closure& success_callback,
-    const CloudRequestErrorCallback& error_callback) {
+    const ErrorCallback& error_callback) {
   LOG(INFO) << "Refreshing access token.";
 
   ErrorPtr error;
@@ -363,8 +363,7 @@
   // one of the copies will be bad.
   auto shared_success_callback =
       std::make_shared<base::Closure>(success_callback);
-  auto shared_error_callback =
-      std::make_shared<CloudRequestErrorCallback>(error_callback);
+  auto shared_error_callback = std::make_shared<ErrorCallback>(error_callback);
 
   RequestSender sender{http::kPost, GetOAuthURL("token"), http_client_};
   sender.SetFormData({
@@ -386,7 +385,7 @@
 
 void DeviceRegistrationInfo::OnRefreshAccessTokenSuccess(
     const std::shared_ptr<base::Closure>& success_callback,
-    const std::shared_ptr<CloudRequestErrorCallback>& error_callback,
+    const std::shared_ptr<ErrorCallback>& error_callback,
     int id,
     const HttpClient::Response& response) {
   VLOG(1) << "Refresh access token request with ID " << id << " completed";
@@ -424,7 +423,7 @@
 
 void DeviceRegistrationInfo::OnRefreshAccessTokenError(
     const std::shared_ptr<base::Closure>& success_callback,
-    const std::shared_ptr<CloudRequestErrorCallback>& error_callback,
+    const std::shared_ptr<ErrorCallback>& error_callback,
     int id,
     const Error* error) {
   VLOG(1) << "Refresh access token request with ID " << id << " failed";
@@ -516,7 +515,7 @@
 
 void DeviceRegistrationInfo::GetDeviceInfo(
     const CloudRequestCallback& success_callback,
-    const CloudRequestErrorCallback& error_callback) {
+    const ErrorCallback& error_callback) {
   ErrorPtr error;
   if (!VerifyRegistrationCredentials(&error)) {
     if (!error_callback.is_null())
@@ -632,7 +631,7 @@
     const std::string& url,
     const base::DictionaryValue* body,
     const CloudRequestCallback& success_callback,
-    const CloudRequestErrorCallback& error_callback) {
+    const ErrorCallback& error_callback) {
   // We make CloudRequestData shared here because we want to make sure
   // there is only one instance of success_callback and error_calback since
   // those may have move-only types and making a copy of the callback with
@@ -881,7 +880,7 @@
 
 void DeviceRegistrationInfo::UpdateDeviceResource(
     const base::Closure& on_success,
-    const CloudRequestErrorCallback& on_failure) {
+    const ErrorCallback& on_failure) {
   queued_resource_update_callbacks_.emplace_back(on_success, on_failure);
   if (!in_progress_resource_update_callbacks_.empty()) {
     VLOG(1) << "Another request is already pending.";
@@ -1001,9 +1000,8 @@
   callback.Run(commands ? *commands : empty);
 }
 
-void DeviceRegistrationInfo::OnFetchCommandsError(
-    const CloudRequestErrorCallback& callback,
-    const Error* error) {
+void DeviceRegistrationInfo::OnFetchCommandsError(const ErrorCallback& callback,
+                                                  const Error* error) {
   OnFetchCommandsReturned();
   callback.Run(error);
 }
@@ -1017,7 +1015,7 @@
 
 void DeviceRegistrationInfo::FetchCommands(
     const base::Callback<void(const base::ListValue&)>& on_success,
-    const CloudRequestErrorCallback& on_failure) {
+    const ErrorCallback& on_failure) {
   fetch_commands_request_sent_ = true;
   fetch_commands_request_queued_ = false;
   DoCloudRequest(
diff --git a/libweave/src/device_registration_info.h b/libweave/src/device_registration_info.h
index 752a4e2..ac8e7a7 100644
--- a/libweave/src/device_registration_info.h
+++ b/libweave/src/device_registration_info.h
@@ -53,7 +53,6 @@
  public:
   using CloudRequestCallback =
       base::Callback<void(const base::DictionaryValue& response)>;
-  using CloudRequestErrorCallback = base::Callback<void(const Error* error)>;
 
   DeviceRegistrationInfo(const std::shared_ptr<CommandManager>& command_manager,
                          const std::shared_ptr<StateManager>& state_manager,
@@ -67,9 +66,6 @@
   // Cloud overrides.
   void AddOnRegistrationChangedCallback(
       const OnRegistrationChangedCallback& callback) override;
-  void GetDeviceInfo(
-      const OnCloudRequestCallback& success_callback,
-      const OnCloudRequestErrorCallback& error_callback) override;
   std::string RegisterDevice(const std::string& ticket_id,
                              ErrorPtr* error) override;
   void UpdateDeviceInfo(const std::string& name,
@@ -85,6 +81,9 @@
                            const std::string& service_url,
                            ErrorPtr* error) override;
 
+  void GetDeviceInfo(const CloudRequestCallback& success_callback,
+                     const ErrorCallback& error_callback);
+
   // Returns the GCD service request URL. If |subpath| is specified, it is
   // appended to the base URL which is normally
   //    https://www.googleapis.com/clouddevices/v1/".
@@ -148,17 +147,17 @@
 
   // Forcibly refreshes the access token.
   void RefreshAccessToken(const base::Closure& success_callback,
-                          const CloudRequestErrorCallback& error_callback);
+                          const ErrorCallback& error_callback);
 
   // Callbacks for RefreshAccessToken().
   void OnRefreshAccessTokenSuccess(
       const std::shared_ptr<base::Closure>& success_callback,
-      const std::shared_ptr<CloudRequestErrorCallback>& error_callback,
+      const std::shared_ptr<ErrorCallback>& error_callback,
       int id,
       const provider::HttpClient::Response& response);
   void OnRefreshAccessTokenError(
       const std::shared_ptr<base::Closure>& success_callback,
-      const std::shared_ptr<CloudRequestErrorCallback>& error_callback,
+      const std::shared_ptr<ErrorCallback>& error_callback,
       int id,
       const Error* error);
 
@@ -181,7 +180,7 @@
                       const std::string& url,
                       const base::DictionaryValue* body,
                       const CloudRequestCallback& success_callback,
-                      const CloudRequestErrorCallback& error_callback);
+                      const ErrorCallback& error_callback);
 
   // Helper for DoCloudRequest().
   struct CloudRequestData {
@@ -189,7 +188,7 @@
     std::string url;
     std::string body;
     CloudRequestCallback success_callback;
-    CloudRequestErrorCallback error_callback;
+    ErrorCallback error_callback;
   };
   void SendCloudRequest(const std::shared_ptr<const CloudRequestData>& data);
   void OnCloudRequestSuccess(
@@ -207,7 +206,7 @@
   void CheckAccessTokenError(const Error* error);
 
   void UpdateDeviceResource(const base::Closure& on_success,
-                            const CloudRequestErrorCallback& on_failure);
+                            const ErrorCallback& on_failure);
   void StartQueuedUpdateDeviceResource();
   // Success/failure callbacks for UpdateDeviceResource().
   void OnUpdateDeviceResourceSuccess(const base::DictionaryValue& device_info);
@@ -225,13 +224,12 @@
 
   void FetchCommands(
       const base::Callback<void(const base::ListValue&)>& on_success,
-      const CloudRequestErrorCallback& on_failure);
+      const ErrorCallback& on_failure);
   // Success/failure callbacks for FetchCommands().
   void OnFetchCommandsSuccess(
       const base::Callback<void(const base::ListValue&)>& callback,
       const base::DictionaryValue& json);
-  void OnFetchCommandsError(const CloudRequestErrorCallback& callback,
-                            const Error* error);
+  void OnFetchCommandsError(const ErrorCallback& callback, const Error* error);
   // Called when FetchCommands completes (with either success or error).
   // This method reschedules any pending/queued fetch requests.
   void OnFetchCommandsReturned();
@@ -316,7 +314,7 @@
   bool fetch_commands_request_queued_{false};
 
   using ResourceUpdateCallbackList =
-      std::vector<std::pair<base::Closure, CloudRequestErrorCallback>>;
+      std::vector<std::pair<base::Closure, ErrorCallback>>;
   // Success/error callbacks for device resource update request currently in
   // flight to the cloud server.
   ResourceUpdateCallbackList in_progress_resource_update_callbacks_;
diff --git a/libweave/src/weave_unittest.cc b/libweave/src/weave_unittest.cc
index bcd39e2..8250d8c 100644
--- a/libweave/src/weave_unittest.cc
+++ b/libweave/src/weave_unittest.cc
@@ -241,13 +241,6 @@
     cloud_ = device_->GetCloud();
     ASSERT_TRUE(cloud_);
 
-    cloud_->GetDeviceInfo(
-        base::Bind(
-            [](const base::DictionaryValue& response) { ADD_FAILURE(); }),
-        base::Bind([](const Error* error) {
-          EXPECT_TRUE(error->HasError("gcd", "device_not_registered"));
-        }));
-
     for (const auto& cb : http_server_changed_cb_)
       cb.Run(http_server_);