Bug 171464

Summary: Speedometer: Add an Elm TodoMVC implementation
Product: WebKit Reporter: Addy Osmani <addyo>
Component: Tools / TestsAssignee: Addy Osmani <addyo>
Status: RESOLVED FIXED    
Severity: Normal CC: buildbot, cdumez, lforschler, rniwa
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 170513    
Attachments:
Description Flags
Patch
none
Patch
none
Patch none

Addy Osmani
Reported 2017-04-28 17:04:51 PDT
As part of Speedometer 2.0 (https://bugs.webkit.org/show_bug.cgi?id=170513), this adds an Elm (http://elm-lang.org/) implementation of TodoMVC to Speedometer. Elm is a functional programming language that compiles to JavaScript. It is popular in a subsection of the JavaScript ecosystem and demonstrates FP patterns we are likely to see more of the front-end community take on in the future. Including it in the benchmark as our current tracking of FP-like JS projects could be improved.
Attachments
Patch (920.58 KB, patch)
2017-04-28 17:06 PDT, Addy Osmani 		no flags
DetailsFormatted DiffDiff
Patch (372.55 KB, patch)
2017-04-28 17:26 PDT, Addy Osmani 		no flags
DetailsFormatted DiffDiff
Patch (369.00 KB, patch)
2017-05-11 19:32 PDT, Addy Osmani 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Addy Osmani
Comment 1 2017-04-28 17:06:59 PDT
Created attachment 308622 [details] Patch
Addy Osmani
Comment 2 2017-04-28 17:26:02 PDT
Created attachment 308629 [details] Patch
Ryosuke Niwa
Comment 3 2017-05-11 19:16:50 PDT
Comment on attachment 308629 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=308629&action=review > PerformanceTests/ChangeLog:7 > + Reviewed by NOBODY (OOPS!). > + Speedometer: Add Elm TodoMVC implementation Same comment about the blank line and inline comments.
Ryosuke Niwa
Comment 4 2017-05-11 19:17:07 PDT
This patch also doesn't seem to apply on ToT for whatever reason.
Addy Osmani
Comment 5 2017-05-11 19:32:58 PDT
Created attachment 309848 [details] Patch
Addy Osmani
Comment 6 2017-05-11 19:33:56 PDT
I've corrected the blank line and inline comments. Keeping fingers crossed on ToT this time :)
Ryosuke Niwa
Comment 7 2017-05-11 19:50:20 PDT
Comment on attachment 309848 [details] Patch Clearing flags on attachment: 309848 Committed r216717: <http://trac.webkit.org/changeset/216717>
Ryosuke Niwa
Comment 8 2017-05-11 19:50:22 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.