From 1c16445989b79bbc2aba008762e8a2927104ea0f Mon Sep 17 00:00:00 2001
From: Kevin Chyn <kchyn@google.com>
Date: Wed, 13 Feb 2019 10:16:52 -0800
Subject: [PATCH] resetLockout should return Status instead of bool

The actual lockout reset is asynchronous and returns its result in the
onLockoutChanged() callback

Bug: 121196511
Bug: 121198195
Bug: 123262389

Test: Builds
Change-Id: Ia5fd34d77ae1f8dba1a943e0e1fc51d458753090
---
 biometrics/face/1.0/IBiometricsFace.hal | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/biometrics/face/1.0/IBiometricsFace.hal b/biometrics/face/1.0/IBiometricsFace.hal
index 813f04000b..0499c5d0fd 100644
--- a/biometrics/face/1.0/IBiometricsFace.hal
+++ b/biometrics/face/1.0/IBiometricsFace.hal
@@ -242,8 +242,10 @@ interface IBiometricsFace {
      * Reset lockout for the current user.
      *
      * @param hat A valid Hardware Authentication Token, generated when the
-     *     user authenticates with Pin/Pattern/Pass.
-     * @return true if lockout was reset, false otherwise.
+     *     user authenticates with Pin/Pattern/Pass. When the Hardware
+     *     Authentication Token is verified, lockout must be reset and
+     *     onLockoutChanged must be called with duration 0.
+     * @return status The status of this method call.
      */
-    resetLockout(vec<uint8_t> hat) generates (bool success);
+    resetLockout(vec<uint8_t> hat) generates (Status status);
 };