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

Description Addy Osmani 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.
Comment 1 Addy Osmani 2017-04-28 17:06:59 PDT
Created attachment 308622 [details]
Patch
Comment 2 Addy Osmani 2017-04-28 17:26:02 PDT
Created attachment 308629 [details]
Patch
Comment 3 Ryosuke Niwa 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.
Comment 4 Ryosuke Niwa 2017-05-11 19:17:07 PDT
This patch also doesn't seem to apply on ToT for whatever reason.
Comment 5 Addy Osmani 2017-05-11 19:32:58 PDT
Created attachment 309848 [details]
Patch
Comment 6 Addy Osmani 2017-05-11 19:33:56 PDT
I've corrected the blank line and inline comments. Keeping fingers crossed on ToT this time :)
Comment 7 Ryosuke Niwa 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>
Comment 8 Ryosuke Niwa 2017-05-11 19:50:22 PDT
All reviewed patches have been landed.  Closing bug.