Add support for ControlList API's controlList attribute

[raglan-road]

In version 58, Chrome shipped support for the HTMLMediaElement ControlList API and, by extension, the controlList attribute on elements like video and audio. Here's how the attribute looks in practice.

React <= 15.5.4 does not support this attribute and strips it out with a Warning: Unknown prop controlsList on <video> tag. Remove this prop from the element. warning in the console.

Could you add support for this new attribute? This isn't a pressing issue as it's possible to set the attribute via javascript (i.e. by setting it in a ref or in a component lifecycle event), but it would be nice to have it recognized as the valid attribute it is.

[mikecx]

Just ran into this today as well but inside a stateless/pure component so the ref/lifecycle events really wont work.

[arsduo]

Ran into the same issue today. Support for this (and/or a standardized way to specify unknown props as HTML attributes as described in #7311 etc.) would be awesome.

[nhunzaker]

@arsduo I'll reference this issue in the description of my PR so that we can be sure to circle back to this issue when it merges.

[arsduo]

Thank you very much!

…

On May 22, 2017 18:10, "Nathan Hunzaker" ***@***.***> wrote: @arsduo <https://github.com/arsduo> I'll reference this issue in the description of my PR so that we can be sure to circle back to this issue when it merges. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#9594 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAC8xd7hb4h-IcLGBmiTPeZYYxiDRCAZks5r8hXFgaJpZM4NPeBB> .

[Nopzen]

Ran into this issue aswell, would be amazing to have this!

[rickyH]

Seems like it will need adding to these files:
https://github.com/facebook/react/blob/b1768b5a48d1f82e4ef4150e0036c5f846d3758a/src/renderers/dom/shared/HTMLDOMPropertyConfig.js

https://github.com/facebook/react/blob/550b0220ca146fbe6fa0695c2fb9d97944d6abfe/docs/docs/reference-dom-elements.md

[nhunzaker]

Ah I might have misdiagnosed this. @rickyH is right, it looks like we just need to add these to a whitelist. I'll throw together a PR and get something up for testing.

[nhunzaker]

Done in #9940. I've also put up a jsfiddle if anyone doesn't mind taking a gander to make sure I haven't missed something:

https://jsfiddle.net/84v837e9/88/

[nhunzaker]

Merged the addition. I've also sent out a pull request to ship it in the next minor release:
#9941

[uncleramsay]

Do we know when this will be released?