Bug 225287

Summary:

Copy PerformanceTests/MotionMark to WebSites/browserbench.org/MotionMark1.2

Product:

WebKit

Reporter:

Myles C. Maxfield <mmaxfield>

Component:

New Bugs

Assignee:

Myles C. Maxfield <mmaxfield>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

ap, ggaren, mjs, rniwa, sabouhallawa, webkit-bug-importer

Priority:

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>