1437135 - Remove nsIDOMValidityState

Status: RESOLVED FIXED

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

Comment 1

[Adrian Wielgosik [:adrian17]]

Attachment: Bug 1437135 - Remove nsIDOMValidityState.

Review commit: https://reviewboard.mozilla.org/r/219136/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/219136/

Comment 2

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 8949827 [details]
Bug 1437135 - Remove nsIDOMValidityState.

https://reviewboard.mozilla.org/r/219136/#review224902

r=me

::: dom/html/nsIConstraintValidation.h:82
(Diff revision 1)
>  protected:
>  
>    // You can't instantiate an object from that class.
>    nsIConstraintValidation();
>  
> -  nsresult GetValidity(nsIDOMValidityState** aValidity);
> +  nsresult GetValidity(mozilla::dom::ValidityState** aValidity);

Is this function actually used?  It doesn't seem to be; you can probably just remove it instead of fixing it.

Comment 3

[Adrian Wielgosik [:adrian17]]

Comment on attachment 8949827 [details]
Bug 1437135 - Remove nsIDOMValidityState.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/219136/diff/1-2/

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/2f17747bf190
Remove nsIDOMValidityState. r=bz

Comment 5

[Bogdan Tara[:bogdan_tara | bogdant]]

https://hg.mozilla.org/mozilla-central/rev/2f17747bf190