Bug 225287

Summary: Copy PerformanceTests/MotionMark to WebSites/browserbench.org/MotionMark1.2
Product: WebKit Reporter: Myles C. Maxfield <mmaxfield>
Component: New BugsAssignee: Myles C. Maxfield <mmaxfield>
Status: RESOLVED FIXED    
Severity: Normal CC: ap, ggaren, mjs, rniwa, sabouhallawa, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch mjs: review+

Description Myles C. Maxfield 2021-05-02 18:02:03 PDT
Copy PerformanceTests/MotionMark to WebSites/browserbench.org/MotionMark1.2 Beta
Comment 1 Myles C. Maxfield 2021-05-02 18:04:47 PDT
Created attachment 427534 [details]
Patch
Comment 2 Maciej Stachowiak 2021-05-02 18:23:44 PDT
Comment on attachment 427534 [details]
Patch

Presuming this is a faithful result of `svn cp`, r=me

Optional comment: might be nicer to avoid using a space in the URL. Maybe run "Beta" up to the 1.2, or use a hyphen.
Comment 3 Ryosuke Niwa 2021-05-02 19:55:38 PDT
Comment on attachment 427534 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=427534&action=review

> Websites/browserbench.org/ChangeLog:13
> +        * MotionMark1.2 Beta/about.html: Added.

Please just use MotionMark1.2. There is no need to suffix with "Beta".
We've published preliminary versions of benchmarks with the new version number all the time in the past.
Comment 4 Myles C. Maxfield 2021-05-02 20:12:12 PDT
Committed r276889 (237235@main): <https://commits.webkit.org/237235@main>
Comment 5 Radar WebKit Bug Importer 2021-05-02 20:13:19 PDT
<rdar://problem/77442927>