Add the Mojo implementation of authenticator.mojom's MakeCredential.

components/webauth/authenticator.mojom

 // Copyright 2016 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.
 
 [JavaPackage="org.chromium.webauth.mojom"]
 module webauth.mojom;
 
+enum AuthenticatorStatus {
+  SUCCESS,
+  CANCELLED,
+  UNKNOWN_ERROR,
+  NOT_ALLOWED_ERROR,
+  NOT_SUPPORTED_ERROR,
+  SECURITY_ERROR,
+};
+
 struct ScopedCredentialInfo {
   array<uint8> client_data;
   array<uint8> attestation;
 };
 
+/* The rpDisplayName member contains the friendly name of the Relying Party, such as "Acme Corporation", "Widgets Inc" or "Awesome Site".
+
+The displayName member contains the friendly name associated with the user account by the Relying Party, such as "John P. Smith".
+
+The id member contains an identifier for the account, specified by the Relying Party. This is not meant to be displayed to the user. It is used by the Relying Party to control the number of credentials - an authenticator will never contain more than one credential for a given Relying Party under the same id.
+
+The name member contains a detailed name for the account, such as "john.p.smith@example.com".*/
+
 struct RelyingPartyAccount {
+  // Friendly name of the Relying Party, e.g. "Acme Corporation"
   string rp_display_name;
+  // Friendly name associated with the user account, e.g. "John P. Smith"
   string display_name;
-  string id;
-  string name;
+  // Identifier for the account, corresponding to no more than one credential 
+  // per authenticator and Relying Party.
+  string id; 
+  // Detailed name for the account, e.g. john.p.smith@example.com
+  string? name;
   // TODO(kpaulhamus): Make this url.mojom.Url in followup CL
-  string image_url;
+  string? image_url;
 };
 
 struct ScopedCredentialParameters {
   ScopedCredentialType type;
   // TODO(kpaulhamus): add AlgorithmIdentifier algorithm;
 };
 
 struct ScopedCredentialOptions {
-  // TODO(kpaulhamus): Make this mojo.common.mojom.TimeDelta in followup CL
-  uint32 timeout_seconds;
-  string rp_id;
+  //TODO(kpaulhamus): Make this mojo.common.mojom.TimeDelta in followup CL
+  double adjusted_timeout;
+  string? rp_id;
   array<ScopedCredentialDescriptor> exclude_list;
   // TODO(kpaulhamus): add Extensions
 };
 
 enum ScopedCredentialType {
   SCOPEDCRED,
 };
 
 struct ScopedCredentialDescriptor {
   ScopedCredentialType type;
   array<uint8> id;
   array<Transport> transports;
 };
 
 enum Transport {
   USB,
   NFC,
   BLE,
 };
 
+// TODO add comments everywhere
+// |account_information|:  
 interface Authenticator {
-  makeCredential(RelyingPartyAccount account_information,
+  MakeCredential(RelyingPartyAccount account_information,
                  array<ScopedCredentialParameters> crypto_parameters,
                  array<uint8> attestation_challenge,
-                 ScopedCredentialOptions? options)
-      => (array<ScopedCredentialInfo> scoped_credentials);
+                 ScopedCredentialOptions options)
+      => (AuthenticatorStatus status, 
+          ScopedCredentialInfo? scoped_credential);
 };