Bug 144316 - [Win] Enable ES6 template literals
Summary: [Win] Enable ES6 template literals
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore JavaScript (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-04-28 00:08 PDT by peavo
Modified: 2015-04-28 03:49 PDT (History)
5 users (show)

See Also:

Attachments
Patch (12.41 KB, patch)
2015-04-28 00:10 PDT, peavo 		ossy: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description peavo 2015-04-28 00:08:21 PDT 
This fixes some JSC stress tests failures on Windows.
Comment 1 peavo 2015-04-28 00:10:14 PDT 
Created attachment 251829 [details]
Patch
Comment 2 Yusuke Suzuki 2015-04-28 01:00:13 PDT 
Oops, it's my fault.
When I add a compile time flag, I missed to add it for windows ports.
Comment 3 peavo 2015-04-28 01:40:43 PDT 
(In reply to comment #2)
> Oops, it's my fault.
> When I add a compile time flag, I missed to add it for windows ports.

No worries :)
Comment 4 Csaba Osztrogonác 2015-04-28 02:34:46 PDT 
Comment on attachment 251829 [details]
Patch

It was enabled everywhere, except Windows - accidentally. ( https://trac.webkit.org/changeset/183373 )

rs=me to enable on Windows too.
Comment 5 Yusuke Suzuki 2015-04-28 02:37:40 PDT 
(In reply to comment #4)
> Comment on attachment 251829 [details]
> Patch
> 
> It was enabled everywhere, except Windows - accidentally. (
> https://trac.webkit.org/changeset/183373 )
> 
> rs=me to enable on Windows too.

Thank you for your review!
BTW, it seems that ES6_CLASS_SYNTAX is also missing. Is it correct?
Comment 6 Csaba Osztrogonác 2015-04-28 03:11:58 PDT 
(In reply to comment #5)
> (In reply to comment #4)
> > Comment on attachment 251829 [details]
> > Patch
> > 
> > It was enabled everywhere, except Windows - accidentally. (
> > https://trac.webkit.org/changeset/183373 )
> > 
> > rs=me to enable on Windows too.
> 
> Thank you for your review!
> BTW, it seems that ES6_CLASS_SYNTAX is also missing. Is it correct?

It can be added in a different patch, but it is already 
enabled from FeaturesDefines.h if it is missing here.

https://trac.webkit.org/changeset/181618/trunk/Source/WTF/wtf/FeatureDefines.h
Comment 7 Yusuke Suzuki 2015-04-28 03:13:19 PDT 
(In reply to comment #6)
> It can be added in a different patch, but it is already 
> enabled from FeaturesDefines.h if it is missing here.
> 
> https://trac.webkit.org/changeset/181618/trunk/Source/WTF/wtf/FeatureDefines.
> h

Make sense!
Comment 8 peavo 2015-04-28 03:49:26 PDT 
Committed r183465: <http://trac.webkit.org/changeset/183465>