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

Use stable error messages in the link onload event tests by default. #10286

Merged
merged 1 commit into from Apr 3, 2018

Conversation

Projects
None yet
4 participants
@bzbarsky
Contributor

bzbarsky commented Apr 3, 2018

No description provided.

@wpt-pr-bot wpt-pr-bot added the html label Apr 3, 2018

@wpt-pr-bot wpt-pr-bot requested review from ayg, jdm, jgraham and zqzhang Apr 3, 2018

@w3c-bots

This comment has been minimized.

w3c-bots commented Apr 3, 2018

Build PASSED

Started: 2018-04-03 17:10:38
Finished: 2018-04-03 17:19:14

View more information about this build on:

@Hexcles

Hexcles approved these changes Apr 3, 2018

LGTM. Thanks!

@bzbarsky bzbarsky merged commit f527ada into web-platform-tests:master Apr 3, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@bzbarsky bzbarsky deleted the bzbarsky:stable-error-messages-for-link-load branch Apr 3, 2018

aarongable pushed a commit to chromium/chromium that referenced this pull request Apr 3, 2018

Remove failure expectations for two tests fixed upstream
The tests were fixed in
web-platform-tests/wpt#10286 and the output is
no longer flaky. Rebaselined.

TBR=foolip

Change-Id: I66e9ea8b8b169d9a3c30b4e80dd1a49c68f454a9
Reviewed-on: https://chromium-review.googlesource.com/994155
Reviewed-by: Robert Ma <robertma@chromium.org>
Reviewed-by: Philip Jägenstedt <foolip@chromium.org>
Commit-Queue: Robert Ma <robertma@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547861}
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment