Template strings functionality is hidden behind an ifdef flag. Remove the flag and enable the feature in all builds.

Comment on attachment 8465605 [details] [diff] [review] Patch to enable template strings in all builds v1 Review of attachment 8465605 [details] [diff] [review]: ----------------------------------------------------------------- ::: js/src/frontend/TokenStream.cpp @@ +1754,2 @@ > (*tp)->type = TOK_STRING; > + else { Style nit: Keep the curly braces. (According to the house style, if any clause in an if-else chain gets braces, they all do.)

Fixed nit.

https://tbpl.mozilla.org/?tree=Try&rev=6c5b90530208 https://tbpl.mozilla.org/?tree=Try&rev=669235005cc2

Release Note Request (optional, but appreciated) [Why is this notable]: new ES6 feature [Suggested wording]: Implemented & enabled JavaScript Template Strings [Links (documentation, blog post, etc)]: TBD

Added in the 34 release notes "JavaScript Template Strings Implemented (docs)" With docs pointing to https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/template_strings (which needs some love).

(In reply to Sylvestre Ledru [:sylvestre] from comment #9) > Added in the 34 release notes "JavaScript Template Strings Implemented > (docs)" > With docs pointing to > https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/ > template_strings > (which needs some love). Thanks, I updated the docs to be more complete. Also mentioned on https://developer.mozilla.org/en-US/Firefox/Releases/34#JavaScript

After the Array.contains issues, I'm wary... can we use this in chrome code and rely on it shipping in 34? I didn't see an announcement about this feature on m.d.platform, but maybe I missed it?

Discussed on IRC, http://logs.glob.uno/?c=mozilla%23developers&s=7+Oct+2014&e=7+Oct+2014#c1076183 onwards. TL,DR: use at own risk, in-tree usage will probably not kill anyone.