Closed Bug 1758728 Opened 2 years ago Closed 2 years ago

[wpt-sync] Sync PR 33130 - Rebaseline a few Protocol Handler manual tests

Categories

(Core :: DOM: Core & HTML, task, P4)

task

Tracking

()

RESOLVED FIXED
100 Branch
Tracking Status
firefox100 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 33130 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/33130
Details from upstream follow.

Javier Fernández García-Boente <jfernandez@igalia.com> wrote:

Rebaseline a few Protocol Handler manual tests

The change landed in r887616 we added additional capabilities to the
url module to have a more consistent scheme fragment encoding, adding
the 'fragment percent-encode set' 1

This has caused that some URLs that have special characters like '\<'
or '"' were encoded using '%". This extra "%" will be encoded using
"%25", hence a scheme fragment with '\<' will become '%3C" which should
be '%253C' as result of the ProtocolHandler::TranslateUrl function.

The mentioned change performed a rebaseline of several web and unit
tests. However, there are a few manual tests that need to update the
expected results as well.

Bug: 1304190
Change-Id: I1389b6df1c7bc530ee45e27a12828520c53bbbbb

Reviewed-on: https://chromium-review.googlesource.com/3512616
WPT-Export-Revision: f8ae41b90f9edd17ba5e0dadb9a2da21ccbfdac7

Component: web-platform-tests → DOM: Core & HTML
Product: Testing → Core
The PR was not expected to affect any tests, but the try push wasn't a success. Check the try results for infrastructure issues
Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7cc261546750
[wpt PR 33130] - Rebaseline a few Protocol Handler manual tests, a=testonly
Test result changes from PR not available.
Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/07b747da2c1f
[wpt PR 33130] - Rebaseline a few Protocol Handler manual tests, a=testonly
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 100 Branch
You need to log in before you can comment on or make changes to this bug.