1128479 - Update Crypto.webidl to recent interface changes

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[Tim Taubert [:ttaubert] (inactive)]

The interface was updated a few months ago:

https://dvcs.w3.org/hg/webcrypto-api/rev/810285715051

Comment 1

[Tim Taubert [:ttaubert] (inactive)]

Attachment: 0001-Bug-1128479-Update-Crypto.webidl-to-recent-interface.patch

Comment 2

[Olli Pettay [:smaug][[e-mail]]]

Comment on attachment 8557883 [details] [diff] [review]
0001-Bug-1128479-Update-Crypto.webidl-to-recent-interface.patch

>+interface GlobalCrypto {
>+  //[Throws] readonly attribute Crypto crypto;
>+  [Throws] readonly attribute nsIDOMCrypto crypto;
You want Crypto as the type.

> 
> // https://dvcs.w3.org/hg/webcrypto-api/raw-file/tip/spec/Overview.html
>-partial interface Window {
>-  //[Throws] readonly attribute Crypto crypto;
>-  [Throws] readonly attribute nsIDOMCrypto crypto;
>-};

... but I guess since it is nsIDOMCrypto here, fine.
But that really should be fixed. no nsIDOM* in .webidl, please.
Currently nsIDOMCrypto seems to be the only nsIDOM* in any .webidl file.

Comment 3

[Tim Taubert [:ttaubert] (inactive)]

Attachment: 0002-Bug-1128479-Don-t-use-nsIDOMCrypto-in-Crypto.webidl.patch

Comment 4

[Olli Pettay [:smaug][[e-mail]]]

Comment on attachment 8557915 [details] [diff] [review]
0002-Bug-1128479-Don-t-use-nsIDOMCrypto-in-Crypto.webidl.patch


>+Crypto*
> nsGlobalWindow::GetCrypto(ErrorResult& aError)
> {
>   FORWARD_TO_INNER_OR_THROW(GetCrypto, (aError), aError, nullptr);
> 
>   if (!mCrypto) {
>     mCrypto = new Crypto();
>-    mCrypto->Init(this);
>+    nsCOMPtr<nsIDOMCrypto> domCrypto(mCrypto);
>+    domCrypto->Init(this);

Why you need domCrypto?
Init doesn't do anything which might change mCrypto to point some other object, so
mCrypto->Init(this); should work.



Thanks.

Comment 5

[Tim Taubert [:ttaubert] (inactive)]

(In reply to Olli Pettay [:smaug] from comment #4)
> >     mCrypto = new Crypto();
> >-    mCrypto->Init(this);
> >+    nsCOMPtr<nsIDOMCrypto> domCrypto(mCrypto);
> >+    domCrypto->Init(this);
> 
> Why you need domCrypto?
> Init doesn't do anything which might change mCrypto to point some other
> object, so
> mCrypto->Init(this); should work.

Yeah, you're right. Got confused it seems. Thanks!

Comment 6

[Tim Taubert [:ttaubert] (inactive)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/ea48f65afe64
https://hg.mozilla.org/integration/mozilla-inbound/rev/d02b6575d969

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/ea48f65afe64
https://hg.mozilla.org/mozilla-central/rev/d02b6575d969