1271549 - Remove preference "dom.details_element.enabled" for <details> and <summary>

Status: RESOLVED FIXED

(Core :: Layout, defect, P3)

Description

[Ting-Yu Lin [:TYLin] (PDT, UTC-7)]

We can remove the preference "dom.details_element.enabled" once the feature reaches the release channel.

Comment 1

[Boris Zbarsky [:bzbarsky]]

Ting-Yu, do you have time to do this?

Comment 2

[Ting-Yu Lin [:TYLin] (PDT, UTC-7)]

Sure. I can do this. Thanks for reminding.

Comment 3

[Ting-Yu Lin [:TYLin] (PDT, UTC-7)]

Attachment: Bug 1271549 - Remove details and summary preference.

Bug 1259889 Part 2 [1] cannot be reverted cleanly, so I manually undo those
changes in this patch. That is, remove the ability for html.css to
invalidate dynamically since it was added specifically for details element.

Although reftest-stylo.list explicit mentions "DO NOT EDIT!", but I still
remove details pref from the file, since it doesn't harm to edit it anyway.

[1] https://hg.mozilla.org/mozilla-central/rev/30aaf3805b56

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

Comment 4

[Ting-Yu Lin [:TYLin] (PDT, UTC-7)]

Boris, do you have time to review this change?

Comment 5

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 8811164 [details]
Bug 1271549 - Remove details and summary preference.

https://reviewboard.mozilla.org/r/93372/#review93574

r=me

Comment 6

[Ting-Yu Lin [:TYLin] (PDT, UTC-7)]

Boris, thank you for the review.

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/654e7298c7fe
Remove details and summary preference. r=bz

Comment 8

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/654e7298c7fe

Comment 9

[Chris Mills [:cmills]]

I've added a note to the Fx53 release notes:

https://developer.mozilla.org/en-US/Firefox/Releases/53#HTML

I also removed the note about the elements being preffed off until Fx49 from the element ref pages:

https://developer.mozilla.org/en-US/docs/Web/HTML/Element/summary
https://developer.mozilla.org/en-US/docs/Web/HTML/Element/details

As it is no longer relevant.