Bug 1241750 (enable-details-summary-nightly-aurora)

Enable <details> and <summary> on Nightly/Aurora

RESOLVED FIXED in Firefox 48

Status

()

defect
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: TYLin, Assigned: TYLin)

Tracking

(Blocks 1 bug, {dev-doc-needed})

unspecified
mozilla48
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox48 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

3 years ago
I think once bug 591737 and bug 1225412 are landed, all the requirements in the spec should be implemented. We can flip the pref to enable <details> and <summary> on Nightly and Aurora to gain more testing.
It seems to me that if a feature is complete, we could just make it enabled by default for all and let it ride the train, unless we find any serious issue whose fix cannot be done in time or is not upliftable.
That says, we can enable it here, but keep the pref there so that we can turn it off any time if necessary.
(Assignee)

Updated

3 years ago

Updated

3 years ago
Depends on: 1246185
(In reply to Xidorn Quan [:xidorn] (UTC+8) from comment #1)
> It seems to me that if a feature is complete, we could just make it enabled
> by default for all and let it ride the train, unless we find any serious
> issue whose fix cannot be done in time or is not upliftable.

There are other dependencies of bug 1226455.


(Also, adding the other dependency explicitly mentioned in comment 0.)
Alias: enable-details-summary-nightly-aurora
Depends on: 1225412
Comment on attachment 8730082 [details]
MozReview Request: Bug 1241750 - Enable <details> and <summary> on Nightly and Aurora. r=bz

https://reviewboard.mozilla.org/r/39657/#review36701

r=me.  Sorry for the terrible lag.  :(
Attachment #8730082 - Flags: review?(bzbarsky) → review+
(Assignee)

Comment 6

3 years ago
https://reviewboard.mozilla.org/r/39657/#review36701

Don't be sorry. Thank you for the review :)
(Assignee)

Comment 7

3 years ago
Comment on attachment 8730082 [details]
MozReview Request: Bug 1241750 - Enable <details> and <summary> on Nightly and Aurora. r=bz

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/39657/diff/1-2/
Attachment #8730082 - Attachment description: MozReview Request: Bug 1241750 - Enable <details> and <summary> on Nightly and Aurora. r?bz → MozReview Request: Bug 1241750 - Enable <details> and <summary> on Nightly and Aurora. r=bz
(Assignee)

Comment 8

3 years ago
https://reviewboard.mozilla.org/r/39657/#review36863

::: dom/tests/mochitest/general/test_interfaces.html:553
(Diff revision 2)
>  // IMPORTANT: Do not change this list without review from a DOM peer!
>      "HTMLDataElement",
>  // IMPORTANT: Do not change this list without review from a DOM peer!
>      "HTMLDataListElement",
>  // IMPORTANT: Do not change this list without review from a DOM peer!
> -    {name: "HTMLDetailsElement", disabled: true},
> +    {name: "HTMLDetailsElement", release: false},

Ouch. I forgot to change this in my previous patch ...
(Assignee)

Comment 9

3 years ago
Comment on attachment 8730082 [details]
MozReview Request: Bug 1241750 - Enable <details> and <summary> on Nightly and Aurora. r=bz

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/39657/diff/2-3/
(Assignee)

Comment 10

3 years ago
Since bug 1225412 is landed, some svg tests in web-platform-test are pass with toggle events after I rebased. I mark them all pass in patch set 3.

Boris, I touch two test files, which might need a re-review. mozreview don't support change the r+ to r?, so I just NI you here. See https://reviewboard.mozilla.org/r/39657/diff/3#index_header

Latest try (all green):
https://treeherder.mozilla.org/#/jobs?repo=try&revision=31914171d5f8
Flags: needinfo?(bzbarsky)
r=me, but please file a bug on mozreview for not being able to change "r+" to "r?".
Flags: needinfo?(bzbarsky)
(Assignee)

Comment 12

3 years ago
(In reply to Boris Zbarsky [:bz] from comment #11)
> r=me, but please file a bug on mozreview for not being able to change "r+"
> to "r?".

We'll have the fix in bug 1195661. I hope it land soon :)

Comment 14

3 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/72641acb6467
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
You need to log in before you can comment on or make changes to this bug.