Skip to content

etc: Include script for running blink perf tests#41224

Merged
jschwe merged 7 commits intoservo:mainfrom
Narfinger:blink-perf-test-runner
Feb 3, 2026
Merged

etc: Include script for running blink perf tests#41224
jschwe merged 7 commits intoservo:mainfrom
Narfinger:blink-perf-test-runner

Conversation

@Narfinger
Copy link
Copy Markdown
Contributor

This is a simple webdriver script that can run a folder of the blink_perf_tests and output a bmf json file.

Signed-off-by: Narfinger [email protected]

Testing: Does not require testing as it is a separate tool.

@Narfinger Narfinger marked this pull request as ready for review December 15, 2025 14:59
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Dec 15, 2025
Copy link
Copy Markdown
Member

@jschwe jschwe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I haven't tried this out yet, but looks promising! Thanks for looking into this. How hard would it be to also add support to run on HarmonyOS via hdc (perhaps in a follow-up PR)?

@Narfinger Narfinger force-pushed the blink-perf-test-runner branch from 987c28d to 2ad85ee Compare December 16, 2025 09:38
@Narfinger
Copy link
Copy Markdown
Contributor Author

It seems that ./mach test-tidy conflicts with the #!env line at the start.

@Narfinger
Copy link
Copy Markdown
Contributor Author

Oh my mistake it should actually work now and pass tests. I also added simple error handling if the test directory does not exist.

@Narfinger Narfinger requested a review from jschwe January 23, 2026 12:07
@Narfinger Narfinger force-pushed the blink-perf-test-runner branch from 855b560 to e056a90 Compare January 29, 2026 14:47
Signed-off-by: Narfinger <[email protected]>
Signed-off-by: Narfinger <[email protected]>
Signed-off-by: Narfinger <[email protected]>
Signed-off-by: Narfinger <[email protected]>
Signed-off-by: Narfinger <[email protected]>
@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Feb 3, 2026
@jschwe jschwe added this pull request to the merge queue Feb 3, 2026
@servo-highfive servo-highfive added the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Feb 3, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 3, 2026
@servo-highfive servo-highfive added S-tests-failed The changes caused existing tests to fail. and removed S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. labels Feb 3, 2026
@jschwe jschwe added this pull request to the merge queue Feb 3, 2026
@servo-highfive servo-highfive added S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. and removed S-tests-failed The changes caused existing tests to fail. labels Feb 3, 2026
Merged via the queue into servo:main with commit 6a9876a Feb 3, 2026
35 checks passed
@servo-highfive servo-highfive removed the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Feb 3, 2026
@Narfinger Narfinger deleted the blink-perf-test-runner branch February 4, 2026 08:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants