-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Bug 1579450 [wpt PR 18898] - Strip the fragment directive and update …
…scroll-to-text WPT, a=testonly Automatic update from web-platform-tests Strip the fragment directive and update scroll-to-text WPT Scroll to text defines a double-hash as the URL fragment directive[1]. The fragment directive should always be stripped from the URL to avoid breaking pages that use the fragment for state. Our implementation previously only stripped the fragment directive if we parsed targetText. Also improved the web platform test to test whether the target scrolls to the element or text fragment as expected. Tested updated WPT locally with run_web_tests.py --additional-driver-flag= '--enable-blink-features=TextFragmentIdentifiers' [1] whatwg/url#445 Bug: 994818 Change-Id: I48109683a5e5ba162f1db72b1c5b174f3b017251 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1772166 Commit-Queue: Nick Burris <nburrischromium.org> Reviewed-by: David Bokan <bokanchromium.org> Cr-Commit-Position: refs/heads/master{#694407} -- wpt-commits: 603a271948a7162bc6efc3c882856e618eabb30e wpt-pr: 18898 UltraBlame original commit: d6a259fed875176e1f7bc83e32e1eb6cc9ef1fb0
- Loading branch information
Showing
2 changed files
with
179 additions
and
48 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters