cros: Remaining interfaces for DBUS biometrics client.

chromeos/dbus/biod/biod_auth_session_client.h

Index: chromeos/dbus/biod/biod_auth_session_client.h
diff --git a/chromeos/dbus/biod/biod_auth_session_client.h b/chromeos/dbus/biod/biod_auth_session_client.h
new file mode 100644
index 0000000000000000000000000000000000000000..1d5b14db055de483d9299d353fd4c88f3952d38d
--- /dev/null
+++ b/chromeos/dbus/biod/biod_auth_session_client.h
@@ -0,0 +1,42 @@
+// 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_BIOD_AUTH_SESSION_CLIENT_H_
+#define CHROMEOS_DBUS_BIOD_BIOD_AUTH_SESSION_CLIENT_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"
+
+namespace dbus {
+class ObjectPath;
+}
+
+namespace chromeos {
+
+// BiodAuthSessionClient is used to communicate with a biod auth sesion
+// dbus interface.
+class CHROMEOS_EXPORT BiodAuthSessionClient : public DBusClient {

[Daniel Erat, 2017/04/03 18:02:10]

i don't think it makes sense to add classes for these other clients if there's
only going to be one of each of them and each will only have one (or very few)
methods.

i'd recommend just adding new methods to BiodClient instead, e.g.:

  virtual void EndAuthSession(const dbus::ObjectPath auth_session_path) = 0;

+ public:
+  ~BiodAuthSessionClient() override;
+
+  // Ends the auth session at |auth_session_path|.
+  virtual void End(const dbus::ObjectPath& auth_session_path) = 0;
+
+  // Creates the instance.
+  static BiodAuthSessionClient* Create(DBusClientImplementationType type);
+
+ protected:
+  // Create() should be used instead.
+  BiodAuthSessionClient();
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(BiodAuthSessionClient);
+};
+
+}  // namespace chromeos
+
+#endif  // CHROMEOS_DBUS_BIOD_BIOD_AUTH_SESSION_CLIENT_H_