Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Run MotionMark benchmark #13373

Open
jgraham opened this issue Sep 22, 2016 · 7 comments
Open

Run MotionMark benchmark #13373

jgraham opened this issue Sep 22, 2016 · 7 comments
Labels

Comments

@jgraham
Copy link
Contributor

@jgraham jgraham commented Sep 22, 2016

Get MotionMark running in Servo. MotionMark is here: http://browserbench.org/MotionMark/

This metabug is specifically just to get it running. Performance issues will be tracked in #15163.

Part of #16717.

@jdm
Copy link
Member

@jdm jdm commented Sep 22, 2016

Filed #13376 for matchMedia. Turning this into a metabug.

@Manishearth
Copy link
Member

@Manishearth Manishearth commented Sep 26, 2016

MotionMark has JS errors in Firefox and Chrome. It also seems to have JS errors in Safari, which doesn't really make sense.

@Manishearth
Copy link
Member

@Manishearth Manishearth commented Sep 26, 2016

Oh, this is because the windows weren't active. Chrome/Safari work. TEsting firefox

@Manishearth
Copy link
Member

@Manishearth Manishearth commented Sep 26, 2016

Okay, works in firefox too. But firefox has some differences in the animations.

@metajack
Copy link
Contributor

@metajack metajack commented Sep 28, 2016

With matchMedia support (#13453) and the latest WR update (#13474 which hasn't yet landed) the test seems to run as expected, and all objects appear to be there at least until the error caused by #13451.

@metajack metajack assigned metajack and unassigned metajack Oct 20, 2016
@metajack metajack mentioned this issue May 3, 2017
1 of 4 tasks complete
@metajack metajack changed the title MotionMark benchmark doesn't run Run MotionMark benchmark May 3, 2017
@metajack metajack removed the I-perf-slow label May 3, 2017
@metajack
Copy link
Contributor

@metajack metajack commented May 3, 2017

Marking this easy. The task here is to run Servo on this benchmark and file bugs for any errors you find that prevent it from running.

@metajack metajack added the E-easy label May 3, 2017
@highfive
Copy link

@highfive highfive commented May 3, 2017

Hi! If you have any questions regarding this issue, feel free to make a comment here, or ask it in the #servo channel in IRC.

If you intend to work on this issue, then add @highfive: assign me to your comment, and I'll assign this to you. 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
5 participants
You can’t perform that action at this time.