Bug 225287 - Copy PerformanceTests/MotionMark to WebSites/browserbench.org/MotionMark1.2
Summary: Copy PerformanceTests/MotionMark to WebSites/browserbench.org/MotionMark1.2
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Myles C. Maxfield
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2021-05-02 18:02 PDT by Myles C. Maxfield
Modified: 2021-05-02 20:13 PDT (History)
6 users (show)

See Also:

Attachments
Patch (1.25 MB, patch)
2021-05-02 18:04 PDT, Myles C. Maxfield 		mjs: 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 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>