Closed Bug 1066322 Opened 5 years ago Closed 5 years ago
Enable ES6 symbols in all builds
No description provided.
Morphing this bug. Now it's about enabling symbols in all builds. But we'll leave the old code and the JS_HAS_SYMBOLS switch in place, in case problems turn up in Aurora or Beta and it needs to be disabled. Bug 1069416 is about killing the #ifdefs etc. (once we're shipping symbols).
Summary: un-#ifdef ES6 symbols → Enable ES6 symbols by default
Summary: Enable ES6 symbols by default → Enable ES6 symbols in all builds
Assignee: nobody → jorendorff
Status: NEW → ASSIGNED
Let's ship this. I know Jeff would rather let it bake another cycle or two, but the fuzzers have been quiet for a long time now. And leaving it disabled also has costs and risks. Having tests break when we go to Aurora (due to the change in behavior caused by #ifdef NIGHTLY_ONLY) is a burden, writing "if (typeof Symbol...)" in every test in the first place is a burden, and there's work like bug 1054756 that's blocked on shipping this.
Comment on attachment 8517269 [details] [diff] [review] Enable ES6 symbols in all builds Review of attachment 8517269 [details] [diff] [review]: ----------------------------------------------------------------- \o/ Nightly-only for 4 release cycles is certainly enough IMO.
Attachment #8517269 - Flags: review?(jdemooij) → review+
Btw, a blog post would be great. People love to know what we're doing, what's next etc. (Or maybe an "ES6 status" post so that we can also mention some smaller features and stuff we're working on right now.)
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
You need to log in before you can comment on or make changes to this bug.