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

Allowed trailing slashes for WPT test test_file_protocol #17805

merged 1 commit into from Jul 12, 2019


Copy link

julianrkung commented Jul 12, 2019

Chrome automatically adds a trailing slash if the suffix of the file protocol is a directory. Chromedriver currently fails this test due to this trailing slash (the rest of the URL is identical). This changelist considers urls with and without a trailing slash correct.

Please see the following previously merged changelist for precedence:
Allow trailing slash in test_get_current_url_file_protocol

@andreastt andreastt merged commit 43a2ddb into web-platform-tests:master Jul 12, 2019
10 checks passed
10 checks passed
Azure Pipelines Build #20190712.48 succeeded
Azure Pipelines (./wpt test-jobs) ./wpt test-jobs succeeded
Azure Pipelines (affected tests without changes: Safari Technology Preview) affected tests without changes: Safari Technology Preview succeeded
Azure Pipelines (affected tests: Safari Technology Preview) affected tests: Safari Technology Preview succeeded
Azure Pipelines ( hook: safari-preview-affected-tests) hook: safari-preview-affected-tests succeeded
Azure Pipelines ( hook: safari-preview-affected-tests-without-changes) hook: safari-preview-affected-tests-without-changes succeeded
Taskcluster (pull_request) TaskGroup: success
Details - chrome[experimental] Chrome results
Details - firefox[experimental] Firefox results
Details - safari[experimental] Safari results
natechapin added a commit to natechapin/wpt that referenced this pull request Aug 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

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