792980 - Make it possible to skip adding an Bindings.conf entry for the classes which are currently empty

Status: RESOLVED FIXED

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

Description

[(no longer active)]

See bug 792646 comment 2.

Comment 1

[Boris Zbarsky [:bzbarsky]]

Attachment: Don't force consumers to list descriptors for interfaces that are using all teh defaults. new setup is:

* Consequential interfaces with no explicit descriptor are still skipped.
* If no information is provided about an interface, an empty descriptor is
  assumed.
* If a list is provided and the only entry is a worker descriptor, an empty
  main-thread descriptor is assumed.

Comment 2

[(no longer active)]

This should be documented in Bindings.conf as well.

Comment 3

[Boris Zbarsky [:bzbarsky]]

Once this lands, I should also update the docs on MDN.

Comment 5

[Peter Van der Beken [:peterv]]

Comment on attachment 663441 [details] [diff] [review]
Don't force consumers to list descriptors for interfaces that are using all teh defaults.   new setup is:

Review of attachment 663441 [details] [diff] [review]:
-----------------------------------------------------------------

Please add some documentation to Bindings.conf, especially about the worker stuff.

::: dom/bindings/Configuration.py
@@ +36,5 @@
> +                # because chances are we don't need to do anything interesting
> +                # with them.
> +                if iface.isConsequential():
> +                    continue
> +                else:

no need for the else

Comment 6

[Boris Zbarsky [:bzbarsky]]

Made that change, and https://hg.mozilla.org/integration/mozilla-inbound/rev/86cc9c289960

Comment 7

[Boris Zbarsky [:bzbarsky]]

Updated MDN.

Comment 8

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/86cc9c289960