Closed Bug 1594269 Opened 5 years ago Closed 4 years ago

[wpt-sync] Sync PR 20105 - Improve scroll to text WPT coverage.

Categories

(Testing :: web-platform-tests, task, P4)

task

Tracking

(firefox72 fixed)

RESOLVED FIXED
mozilla72
Tracking Status
firefox72 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

Nick Burris <nburris@chromium.org> wrote:

Improve scroll to text WPT coverage.

Adds many tests to the scroll to text WPT suite:

  • Setting window.location.fragmentDirective has no effect
  • All combinations of optional parameters in text directive
  • Matching TextMatchChars and PercentEncodedChars (in particular the
    syntactical characters '&', ‘,’ and ‘-’) including non-ASCII
  • Multiple matches in the page
  • Cross-whitespace/node matching (i.e. context terms and match terms
    can be separated by whitespace and node boundaries)

Also added a readable description to each test case.

Note all tests pass in Chrome except for non-ASCII character matching.

Change-Id: I2a692049ba81bef5412e7b73909fbcdc710eb0da

Reviewed-on: https://chromium-review.googlesource.com/1900648
WPT-Export-Revision: 6382d9a1452c120433d60c0eca002e2802ac483e

PR 20105 applied with additional changes from upstream: c1c27e227840599ad09a150f74851cbd9238c9b9
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → INVALID
Status: RESOLVED → REOPENED
Resolution: INVALID → ---
Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2586997ecf54
[wpt PR 20105] - Improve scroll to text WPT coverage., a=testonly
https://hg.mozilla.org/integration/autoland/rev/bf228d83752f
[wpt PR 20105] - Update wpt metadata, a=testonly
Test result changes from PR not available.
Status: REOPENED → RESOLVED
Closed: 5 years ago4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla72
You need to log in before you can comment on or make changes to this bug.