CrOS: Add success/failure indicators for biod methods with no callback.

chromeos/dbus/biod/biod_client.h

Index: chromeos/dbus/biod/biod_client.h
diff --git a/chromeos/dbus/biod/biod_client.h b/chromeos/dbus/biod/biod_client.h
index c866e6269607655ade19f8301df42df4ee716cb5..094ce3301d8ac9aa194aa2c77d8b90942910f14b 100644
--- a/chromeos/dbus/biod/biod_client.h
+++ b/chromeos/dbus/biod/biod_client.h
@@ -95,8 +95,11 @@ class CHROMEOS_EXPORT BiodClient : public DBusClient {
   virtual void GetRecordsForUser(const std::string& user_id,
                                  const UserRecordsCallback& callback) = 0;
 
-  // Irreversibly destroys all records registered.
-  virtual void DestroyAllRecords() = 0;
+  // Irreversibly destroys all records registered. |callback| is called with

[Daniel Erat, 2017/04/08 00:12:34]

i'd change the second sentence to something simpler like "|callback| is called
asynchronously with the result."

applies to other methods too

[sammiequon, 2017/04/08 01:03:30]

Done.

+  // D-Bus failure if there is a D-Bus error along the way; called with D-Bus
+  // success otherwise.
+

[Daniel Erat, 2017/04/08 00:12:34]

delete blank line

[sammiequon, 2017/04/08 01:03:30]

Done.

+  virtual void DestroyAllRecords(const VoidDBusMethodCallback& callback) = 0;
 
   // Starts the biometric auth session. |callback| is called with the object
   // path of the auth session after the method succeeds.
@@ -106,20 +109,33 @@ class CHROMEOS_EXPORT BiodClient : public DBusClient {
   // type after the method succeeds.
   virtual void RequestType(const BiometricTypeCallback& callback) = 0;
 
-  // Cancels the enroll session at |enroll_session_path|.
-  virtual void CancelEnrollSession(
-      const dbus::ObjectPath& enroll_session_path) = 0;
+  // Cancels the enroll session at |enroll_session_path|. |callback| is called
+  // with D-Bus failure if there is a D-Bus error along the way; called with
+  // D-Bus success otherwise.
+

[Daniel Erat, 2017/04/08 00:12:34]

delete blank line

[sammiequon, 2017/04/08 01:03:30]

Oops, done. 

+  virtual void CancelEnrollSession(const dbus::ObjectPath& enroll_session_path,
+                                   const VoidDBusMethodCallback& callback) = 0;
+
+  // Ends the auth session at |auth_session_path|. |callback| is called with
+  // D-Bus failure if there is a D-Bus error along the way; called with D-Bus
+  // success otherwise.
+

[Daniel Erat, 2017/04/08 00:12:34]

delete blank line

[sammiequon, 2017/04/08 01:03:30]

Done.

+  virtual void EndAuthSession(const dbus::ObjectPath& auth_session_path,
+                              const VoidDBusMethodCallback& callback) = 0;
 
-  // Ends the auth session at |auth_session_path|.
-  virtual void EndAuthSession(const dbus::ObjectPath& auth_session_path) = 0;
+  // Changes the label of the record at |record_path| to |label|. |callback| is
+  // called with D-Bus failure if there is a D-Bus error along the way; called
+  // with D-Bus success otherwise.
 

[Daniel Erat, 2017/04/08 00:12:34]

delete blank line

[sammiequon, 2017/04/08 01:03:30]

Done.

-  // Changes the label of the record at |record_path| to |label|.
   virtual void SetRecordLabel(const dbus::ObjectPath& record_path,
-                              const std::string& label) = 0;
+                              const std::string& label,
+                              const VoidDBusMethodCallback& callback) = 0;
 
   // Removes the record at |record_path|. This record will no longer be able to
-  // used for authentication.
-  virtual void RemoveRecord(const dbus::ObjectPath& record_path) = 0;
+  // used for authentication. |callback| is called with D-Bus failure if there
+  // is a D-Bus error along the way; called with D-Bus success otherwise.
+  virtual void RemoveRecord(const dbus::ObjectPath& record_path,
+                            const VoidDBusMethodCallback& callback) = 0;
 
   // Requests the label of the record at |record_path|. |callback| is called
   // with the label of the record.