WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
171464
Speedometer: Add an Elm TodoMVC implementation
https://bugs.webkit.org/show_bug.cgi?id=171464
Summary
Speedometer: Add an Elm TodoMVC implementation
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
Details
Formatted Diff
Diff
Patch
(372.55 KB, patch)
2017-04-28 17:26 PDT
,
Addy Osmani
no flags
Details
Formatted Diff
Diff
Patch
(369.00 KB, patch)
2017-05-11 19:32 PDT
,
Addy Osmani
no flags
Details
Formatted Diff
Diff
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.
Top of Page
Format For Printing
XML
Clone This Bug