Commit 86181d1a authored by Kim Paulhamus's avatar Kim Paulhamus Committed by Commit Bot

Make the call to Fido2ApiHandlerInternal.getAssertion.

Removes the NOT_IMPLEMENTED placeholder.

Bug: 678885
Change-Id: I4572dc2e6aad8019b383b93173b7962cc97ad60d
Reviewed-on: https://chromium-review.googlesource.com/1109498
Commit-Queue: Kim Paulhamus <kpaulhamus@chromium.org>
Reviewed-by: default avatarTed Choc <tedchoc@chromium.org>
Reviewed-by: default avatarWei-Yin Chen (陳威尹) <wychen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#569734}
parent 87317b87
......@@ -60,7 +60,7 @@ public class AuthenticatorImpl implements Authenticator, HandlerResponseCallback
}
mIsOperationPending = true;
onError(AuthenticatorStatus.NOT_IMPLEMENTED);
Fido2ApiHandler.getInstance().getAssertion(options, mRenderFrameHost, this);
}
@Override
......@@ -88,7 +88,8 @@ public class AuthenticatorImpl implements Authenticator, HandlerResponseCallback
@Override
public void onError(Integer status) {
assert(mMakeCredentialCallback != null || mGetAssertionCallback != null);
assert((mMakeCredentialCallback != null && mGetAssertionCallback == null)
|| (mMakeCredentialCallback == null && mGetAssertionCallback != null));
if (mMakeCredentialCallback != null) {
mMakeCredentialCallback.call(status, null);
} else if (mGetAssertionCallback != null) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment