Backed out changeset 536420808253 (bug 1849056) for bustages on Authe…

…nticatorResponse.cpp
dothq · Aug 17, 2023 · 9716a0a · 9716a0a
1 parent 29d972f
commit 9716a0a
Show file tree

Hide file tree

Showing 9 changed files with 27 additions and 56 deletions.
diff --git a/dom/webauthn/AuthenticatorAssertionResponse.cpp b/dom/webauthn/AuthenticatorAssertionResponse.cpp
@@ -55,15 +55,11 @@ JSObject* AuthenticatorAssertionResponse::WrapObject(
 }
 
 void AuthenticatorAssertionResponse::GetAuthenticatorData(
-    JSContext* aCx, JS::MutableHandle<JSObject*> aValue, ErrorResult& aRv) {
+    JSContext* aCx, JS::MutableHandle<JSObject*> aRetVal) {
   if (!mAuthenticatorDataCachedObj) {
     mAuthenticatorDataCachedObj = mAuthenticatorData.ToArrayBuffer(aCx);
-    if (!mAuthenticatorDataCachedObj) {
-      aRv.NoteJSContextException(aCx);
-      return;
-    }
   }
-  aValue.set(mAuthenticatorDataCachedObj);
+  aRetVal.set(mAuthenticatorDataCachedObj);
 }
 
 nsresult AuthenticatorAssertionResponse::SetAuthenticatorData(
@@ -75,15 +71,11 @@ nsresult AuthenticatorAssertionResponse::SetAuthenticatorData(
 }
 
 void AuthenticatorAssertionResponse::GetSignature(
-    JSContext* aCx, JS::MutableHandle<JSObject*> aValue, ErrorResult& aRv) {
+    JSContext* aCx, JS::MutableHandle<JSObject*> aRetVal) {
   if (!mSignatureCachedObj) {
     mSignatureCachedObj = mSignature.ToArrayBuffer(aCx);
-    if (!mSignatureCachedObj) {
-      aRv.NoteJSContextException(aCx);
-      return;
-    }
   }
-  aValue.set(mSignatureCachedObj);
+  aRetVal.set(mSignatureCachedObj);
 }
 
 nsresult AuthenticatorAssertionResponse::SetSignature(CryptoBuffer& aBuffer) {
@@ -94,21 +86,17 @@ nsresult AuthenticatorAssertionResponse::SetSignature(CryptoBuffer& aBuffer) {
 }
 
 void AuthenticatorAssertionResponse::GetUserHandle(
-    JSContext* aCx, JS::MutableHandle<JSObject*> aValue, ErrorResult& aRv) {
+    JSContext* aCx, JS::MutableHandle<JSObject*> aRetVal) {
   // Per
   // https://w3c.github.io/webauthn/#ref-for-dom-authenticatorassertionresponse-userhandle%E2%91%A0
   // this should return null if the handle is unset.
   if (mUserHandle.IsEmpty()) {
-    aValue.set(nullptr);
+    aRetVal.set(nullptr);
   } else {
     if (!mUserHandleCachedObj) {
       mUserHandleCachedObj = mUserHandle.ToArrayBuffer(aCx);
-      if (!mUserHandleCachedObj) {
-        aRv.NoteJSContextException(aCx);
-        return;
-      }
     }
-    aValue.set(mUserHandleCachedObj);
+    aRetVal.set(mUserHandleCachedObj);
   }
 }
 

diff --git a/dom/webauthn/AuthenticatorAssertionResponse.h b/dom/webauthn/AuthenticatorAssertionResponse.h
@@ -32,18 +32,16 @@ class AuthenticatorAssertionResponse final : public AuthenticatorResponse {
   virtual JSObject* WrapObject(JSContext* aCx,
                                JS::Handle<JSObject*> aGivenProto) override;
 
-  void GetAuthenticatorData(JSContext* aCx, JS::MutableHandle<JSObject*> aValue,
-                            ErrorResult& aRv);
+  void GetAuthenticatorData(JSContext* aCx,
+                            JS::MutableHandle<JSObject*> aRetVal);
 
   nsresult SetAuthenticatorData(CryptoBuffer& aBuffer);
 
-  void GetSignature(JSContext* aCx, JS::MutableHandle<JSObject*> aValue,
-                    ErrorResult& aRv);
+  void GetSignature(JSContext* aCx, JS::MutableHandle<JSObject*> aRetVal);
 
   nsresult SetSignature(CryptoBuffer& aBuffer);
 
-  void GetUserHandle(JSContext* aCx, JS::MutableHandle<JSObject*> aValue,
-                     ErrorResult& aRv);
+  void GetUserHandle(JSContext* aCx, JS::MutableHandle<JSObject*> aRetVal);
 
   nsresult SetUserHandle(CryptoBuffer& aUserHandle);
 

diff --git a/dom/webauthn/AuthenticatorAttestationResponse.cpp b/dom/webauthn/AuthenticatorAttestationResponse.cpp
@@ -51,15 +51,11 @@ JSObject* AuthenticatorAttestationResponse::WrapObject(
 }
 
 void AuthenticatorAttestationResponse::GetAttestationObject(
-    JSContext* aCx, JS::MutableHandle<JSObject*> aValue, ErrorResult& aRv) {
+    JSContext* aCx, JS::MutableHandle<JSObject*> aRetVal) {
   if (!mAttestationObjectCachedObj) {
     mAttestationObjectCachedObj = mAttestationObject.ToArrayBuffer(aCx);
-    if (!mAttestationObjectCachedObj) {
-      aRv.NoteJSContextException(aCx);
-      return;
-    }
   }
-  aValue.set(mAttestationObjectCachedObj);
+  aRetVal.set(mAttestationObjectCachedObj);
 }
 
 nsresult AuthenticatorAttestationResponse::SetAttestationObject(

diff --git a/dom/webauthn/AuthenticatorAttestationResponse.h b/dom/webauthn/AuthenticatorAttestationResponse.h
@@ -32,8 +32,8 @@ class AuthenticatorAttestationResponse final : public AuthenticatorResponse {
   virtual JSObject* WrapObject(JSContext* aCx,
                                JS::Handle<JSObject*> aGivenProto) override;
 
-  void GetAttestationObject(JSContext* aCx, JS::MutableHandle<JSObject*> aValue,
-                            ErrorResult& aRv);
+  void GetAttestationObject(JSContext* aCx,
+                            JS::MutableHandle<JSObject*> aRetVal);
 
   nsresult SetAttestationObject(CryptoBuffer& aBuffer);
 

diff --git a/dom/webauthn/AuthenticatorResponse.cpp b/dom/webauthn/AuthenticatorResponse.cpp
@@ -34,15 +34,11 @@ AuthenticatorResponse::~AuthenticatorResponse() {
 nsISupports* AuthenticatorResponse::GetParentObject() const { return mParent; }
 
 void AuthenticatorResponse::GetClientDataJSON(
-    JSContext* aCx, JS::MutableHandle<JSObject*> aValue, ErrorResult& aRv) {
+    JSContext* aCx, JS::MutableHandle<JSObject*> aRetVal) {
   if (!mClientDataJSONCachedObj) {
     mClientDataJSONCachedObj = mClientDataJSON.ToArrayBuffer(aCx);
-    if (!mClientDataJSONCachedObj) {
-      aRv.NoteJSContextException(aCx);
-      return;
-    }
   }
-  aValue.set(mClientDataJSONCachedObj);
+  aRetVal.set(mClientDataJSONCachedObj);
 }
 
 nsresult AuthenticatorResponse::SetClientDataJSON(CryptoBuffer& aBuffer) {

diff --git a/dom/webauthn/AuthenticatorResponse.h b/dom/webauthn/AuthenticatorResponse.h
@@ -33,8 +33,7 @@ class AuthenticatorResponse : public nsISupports, public nsWrapperCache {
 
   void GetFormat(nsString& aRetVal) const;
 
-  void GetClientDataJSON(JSContext* aCx, JS::MutableHandle<JSObject*> aValue,
-                         ErrorResult& aRv);
+  void GetClientDataJSON(JSContext* aCx, JS::MutableHandle<JSObject*> aRetVal);
 
   nsresult SetClientDataJSON(CryptoBuffer& aBuffer);
 

diff --git a/dom/webauthn/PublicKeyCredential.cpp b/dom/webauthn/PublicKeyCredential.cpp
@@ -58,16 +58,11 @@ JSObject* PublicKeyCredential::WrapObject(JSContext* aCx,
 }
 
 void PublicKeyCredential::GetRawId(JSContext* aCx,
-                                   JS::MutableHandle<JSObject*> aValue,
-                                   ErrorResult& aRv) {
+                                   JS::MutableHandle<JSObject*> aRetVal) {
   if (!mRawIdCachedObj) {
     mRawIdCachedObj = mRawId.ToArrayBuffer(aCx);
-    if (!mRawIdCachedObj) {
-      aRv.NoteJSContextException(aCx);
-      return;
-    }
   }
-  aValue.set(mRawIdCachedObj);
+  aRetVal.set(mRawIdCachedObj);
 }
 
 already_AddRefed<AuthenticatorResponse> PublicKeyCredential::Response() const {

diff --git a/dom/webauthn/PublicKeyCredential.h b/dom/webauthn/PublicKeyCredential.h
@@ -32,8 +32,7 @@ class PublicKeyCredential final : public Credential {
   virtual JSObject* WrapObject(JSContext* aCx,
                                JS::Handle<JSObject*> aGivenProto) override;
 
-  void GetRawId(JSContext* aCx, JS::MutableHandle<JSObject*> aValue,
-                ErrorResult& aRv);
+  void GetRawId(JSContext* cx, JS::MutableHandle<JSObject*> aRetVal);
 
   already_AddRefed<AuthenticatorResponse> Response() const;
 

diff --git a/dom/webidl/WebAuthentication.webidl b/dom/webidl/WebAuthentication.webidl
@@ -12,7 +12,7 @@
 [SecureContext, Pref="security.webauth.webauthn",
  Exposed=Window]
 interface PublicKeyCredential : Credential {
-    [SameObject, Throws] readonly attribute ArrayBuffer      rawId;
+    [SameObject] readonly attribute ArrayBuffer              rawId;
     [SameObject] readonly attribute AuthenticatorResponse    response;
     AuthenticationExtensionsClientOutputs getClientExtensionResults();
 };
@@ -27,21 +27,21 @@ partial interface PublicKeyCredential {
 [SecureContext, Pref="security.webauth.webauthn",
  Exposed=Window]
 interface AuthenticatorResponse {
-    [SameObject, Throws] readonly attribute ArrayBuffer clientDataJSON;
+    [SameObject] readonly attribute ArrayBuffer clientDataJSON;
 };
 
 [SecureContext, Pref="security.webauth.webauthn",
  Exposed=Window]
 interface AuthenticatorAttestationResponse : AuthenticatorResponse {
-    [SameObject, Throws] readonly attribute ArrayBuffer attestationObject;
+    [SameObject] readonly attribute ArrayBuffer attestationObject;
 };
 
 [SecureContext, Pref="security.webauth.webauthn",
  Exposed=Window]
 interface AuthenticatorAssertionResponse : AuthenticatorResponse {
-    [SameObject, Throws] readonly attribute ArrayBuffer      authenticatorData;
-    [SameObject, Throws] readonly attribute ArrayBuffer      signature;
-    [SameObject, Throws] readonly attribute ArrayBuffer?     userHandle;
+    [SameObject] readonly attribute ArrayBuffer      authenticatorData;
+    [SameObject] readonly attribute ArrayBuffer      signature;
+    [SameObject] readonly attribute ArrayBuffer?     userHandle;
 };
 
 dictionary PublicKeyCredentialParameters {