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
Changes to improve the stability when running multiple tests with servodriver #6161
Conversation
Critic review: https://critic.hoppipolla.co.uk/r/5082 This is an external review system which you may optionally use for the code review of your pull request. In order to help critic track your changes, please do not make in-place history rewrites (e.g. via |
☔ The latest upstream changes (presumably #6175) made this pull request unmergeable. Please resolve the merge conflicts. |
c818209
to
e7f6e7b
Compare
Assigning to @glennw. |
@jgraham squash, rebase and then r=me |
e7f6e7b
to
81ac0b5
Compare
☔ The latest upstream changes (presumably #6246) made this pull request unmergeable. Please resolve the merge conflicts. |
* Wait for the correct pipeline to return a LoadComplete message before signalling a load is complete, and ensure that the root pipeline is the one corresponding to the active document of the top level browsing context, even if this has not yet painted. * Ensure that TakeScreenshot operates on the correct pipeline * Reset the screenshot ready flag whenever we decide that we are ready to take a screenshot.
81ac0b5
to
aa0f7a7
Compare
@bors-servo: r=glennw |
📌 Commit aa0f7a7 has been approved by |
This fixes various race conditions that affect test execution when using the servodriver product. It doesn't yet do enough to make servodriver a viable alternative to the normal servo test executor. <!-- Reviewable:start --> [<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/6161) <!-- Reviewable:end -->
This fixes various race conditions that affect test execution when using the servodriver product. It doesn't yet do enough to make servodriver a viable alternative to the normal servo test executor.