disable JS Set and Map before Firefox (12) Beta cycle

RESOLVED FIXED

Status

()

Core
JavaScript Engine
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: blizzard, Assigned: jorendorff)

Tracking

({dev-doc-complete})

12 Branch
dev-doc-complete
Points:
---

Firefox Tracking Flags

(firefox12+ fixed, firefox13-)

Details

(Whiteboard: [qa-])

(Reporter)

Description

6 years ago
We added some experimental JS features to trunk, and we want people to test them during the Aurora cycle, but we don't want to leak them onto the web before they are ready.  So we want to disable them during or before the beta process for Firefox 12.
Jason, can you grab this?

Comment 2

5 years ago
It's not clear to me if this is just a simple backout of 

https://hg.mozilla.org/mozilla-central/rev/6a5e20a0f741
https://hg.mozilla.org/mozilla-central/rev/51f43afa160e

on FF12 or something more complicated. Please confirm ASAP since we're performing the merge as I'm writing this.
(Assignee)

Comment 3

5 years ago
Yes, that should do it. If it doesn't, ping me and we can work through whatever issues come up. It should be straightforward; this stuff is a pure library feature that nothing is depending on yet.

Updated

5 years ago
Assignee: general → jorendorff
(Assignee)

Comment 4

5 years ago
I backed this out.
https://hg.mozilla.org/releases/mozilla-beta/rev/aa90687f6964
https://hg.mozilla.org/releases/mozilla-beta/rev/9f64b9445cd4
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED

Updated

5 years ago
status-firefox12: --- → fixed
tracking-firefox13: --- → +
Whiteboard: [qa-]

Comment 5

5 years ago
We're continuing to track for FF13 in case we need to disable JS map and set for that release as well.
Duplicate of this bug: 721916
mdn still states these are available in firefox 12, should be updated.
Keywords: dev-doc-needed
(In reply to Marco Bonardo [:mak] from comment #7)
> mdn still states these are available in firefox 12, should be updated.

Thanks for the note--I bumped the min version to 14 in both MDN docs.
Keywords: dev-doc-needed
David, Marco: sorry about that. It is very difficult for us to know when something is backed out in the Aurora or Beta phase. We usually follow the original bug, which stay RESOLVED / FIXED, but not the back-out bug (like this one).

So when a backout/disabling of a new feature happens, please add the dev-doc-needed to one of the two bugs (original or new one). That gives us a chance to see it. :-) 

(I set the keyword to dev-doc-complete here, so that we know it has been handled. Thanks David!)
Keywords: dev-doc-complete
I actually saw some articles about the FF12 release saying it includes Sets and Maps, and checking the release notes, we stated they are in the product :( (http://www.mozilla.org/en-US/firefox/12.0/releasenotes/)
(In reply to Marco Bonardo [:mak] from comment #10)
> I actually saw some articles about the FF12 release saying it includes Sets
> and Maps, and checking the release notes, we stated they are in the product
> :( (http://www.mozilla.org/en-US/firefox/12.0/releasenotes/)

I've fixed this now. We're going to do a better job of double checking our release notes ahead of our next release.

http://viewvc.svn.mozilla.org/vc?view=revision&revision=104777
Should we remove description about Set/Map from this page
https://developer.mozilla.org/en/Firefox_12_for_developers#JavaScript
and add to Firefox 14 page?
https://developer.mozilla.org/en/Firefox_14_for_developers
# if we will not disable Set/Map on 13beta, we should add to Firefox 13 page
It's removed from the Firefox 12 page and on the Firefox 14 page. If someone can tell me if they'll be disabled or not in 13, that would help.
That is, they're now listed on the Firefox 14 page.
At present, they're available in FF13 beta.  I had an IRC conversation with Jason shortly before the last FF train departure, and the consensus was to leave it in.  The reason:  the API is about 85% stable now.
(In reply to Alex Vincent [:WeirdAl] from comment #15)
> At present, they're available in FF13 beta.  I had an IRC conversation with
> Jason shortly before the last FF train departure, and the consensus was to
> leave it in.  The reason:  the API is about 85% stable now.

Given that, untracking for FF13.
tracking-firefox13: + → -
You need to log in before you can comment on or make changes to this bug.