742168 - ParisBindingsEnabled should be renamed

Status: RESOLVED FIXED

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

Description

[Boris Zbarsky [:bzbarsky]]

Maybe ExperimentalBindingsEnabled(), since this pref will only control the moving edge of the conversion?

Comment 1

[Bobby Holley (:bholley)]

Shrug - I think Paris bindings is a fine way to refer to the "new dom bindings that aren't nodelists". I don't think it's any more confusing to have 'paris_bindings' control the moving edge of this stuff than it is to have 'new_bindings' control the moving edge of nodelist stuff.

I'm fine with changing it too - I just don't think it's really worth the effort.

Comment 2

[Boris Zbarsky [:bzbarsky]]

I think this would be really confusing to new contributors, who would then start searching about for what the hecko this "Paris" thing has to do with anything and come up empty.

Note that I think the nodelist situation is also silly, naming-wise.  ;)

Comment 3

[Boris Zbarsky [:bzbarsky]]

Attachment: Maybe like so?

Comment 4

[Peter Van der Beken [:peterv]]

I don't think the nodelist naming is silly personally, I think we should merge the two prefs instead after the next train.

Comment 5

[Boris Zbarsky [:bzbarsky]]

https://hg.mozilla.org/projects/birch/rev/abc682ac7eee

That said, do we want to get this into 14, since it changes the pref name?

Comment 6

[(no longer active)]

https://hg.mozilla.org/mozilla-central/rev/abc682ac7eee