cros: Remaining interfaces for DBUS biometrics client.

chromeos/dbus/biod_record_client.h

Index: chromeos/dbus/biod_record_client.h
diff --git a/chromeos/dbus/biod_record_client.h b/chromeos/dbus/biod_record_client.h
new file mode 100644
index 0000000000000000000000000000000000000000..e67db3716b85deb275088687a47a3c094730e4ff
--- /dev/null
+++ b/chromeos/dbus/biod_record_client.h
@@ -0,0 +1,55 @@
+// Copyright 2017 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CHROMEOS_DBUS_BIOD_RECORD_CLIENT_H_
+#define CHROMEOS_DBUS_BIOD_RECORD_CLIENT_H_
+
+#include <stdint.h>
+
+#include "base/callback.h"
+#include "base/macros.h"
+#include "chromeos/chromeos_export.h"
+#include "chromeos/dbus/dbus_client.h"
+#include "chromeos/dbus/dbus_client_implementation_type.h"
+#include "chromeos/dbus/dbus_method_call_status.h"
+
+namespace chromeos {
+
+// BiodRecordClient is used to communicate with a biod record via dbus
+// interface.
+class CHROMEOS_EXPORT BiodRecordClient : public DBusClient {
+ public:
+  ~BiodRecordClient() override;
+
+  // LabelCallback is for the RequestLabel method. It receives one argument
+  // which is the label of the record.
+  using LabelCallback = base::Callback<void(const std::string&)>;
+
+  // Changes the label of the record at |record_path| to |label|.
+  virtual void SetRecordLabel(const dbus::ObjectPath& record_path,

[rkc, 2017/03/25 00:41:38]

SetLabel here and Remove below should suffice.

[sammiequon, 2017/04/03 17:52:27]

Done.

+                              const std::string& label) = 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;
+
+  // Requests the label of the record at |record_path|. |callback| is called
+  // with the label of the record.
+  virtual void RequestRecordLabel(const dbus::ObjectPath& record_path,

[rkc, 2017/03/25 00:41:38]

This doesn't seem to exist in the D-Bus API at
https://www.chromium.org/chromium-os/packages/biod (or the gerrit CL).

[sammiequon, 2017/04/03 17:52:27]

It's the property at [1].
[1]
https://cs.chromium.org/chromium/src/third_party/cros_system_api/dbus/service...

+                                  const LabelCallback& callback) = 0;
+
+  // Creates the instance.
+  static BiodRecordClient* Create(DBusClientImplementationType type);
+
+ protected:
+  // Create() should be used instead.
+  BiodRecordClient();
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(BiodRecordClient);
+};
+
+}  // namespace chromeos
+
+#endif  // CHROMEOS_DBUS_BIOD_RECORD_CLIENT_H_