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

servo: Upgrade `ipc-channel`, and make the timeout value in WPT match that of Firefox in debug mode. #6719

Closed
wants to merge 1 commit into from

Conversation

@pcwalton
Copy link
Contributor

pcwalton commented Jul 24, 2015

One WPT test is still too slow. No reasonable timeout value made it fast
enough. Therefore, I have marked the test as a timeout. Optimizing during WPT
running or landing rust-lang/cargo#1826 would fix this as well, I suspect.

r? @jdm

Review on Reviewable

… that of

Gecko in debug mode.

One WPT test is still too slow. No reasonable timeout value made it fast
enough. Therefore, I have marked the test as a timeout. Optimizing during WPT
running or landing rust-lang/cargo#1826 would fix this as well, I suspect.
@pcwalton
Copy link
Contributor Author

pcwalton commented Jul 24, 2015

This is now once again a possibility as release builds were rolled back due to failures. Another option will be selectively generating metadata that turn off the expectations for specific tests in release only and reenabling release builds.

e10s is blocked on this.

@bors-servo
Copy link
Contributor

bors-servo commented Jul 24, 2015

The latest upstream changes (presumably #6705) made this pull request unmergeable. Please resolve the merge conflicts.

@pcwalton
Copy link
Contributor Author

pcwalton commented Jul 25, 2015

Obsolete

@pcwalton pcwalton closed this Jul 25, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.