Open Bug 2050115 Opened 1 day ago Updated 1 day ago

[wpt-sync] Sync PR 60851 - [Editing] Remove VisiblePosition from Inserttext

Categories

(Core :: DOM: Editor, task, P4)

task

Tracking

()

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

Shweta Bindal <shwetabindal@microsoft.com> wrote:

[Editing] Remove VisiblePosition from Inserttext

EditingUseDomPositionApi-gated VP removal in InsertTextCommand,
RebalanceWhitespaceOnTextSubstring, and PASEB. Flag-on path reads
EndingDomSelection() (added in crrev.com/c/7866547) instead of the
VP-canonicalized ending selection.

Refinements: targeted \<wbr> reanchor in InsertText only; drop the
ReanchorIntoSiblingText hop over \<br>; keep VP for PASEB structural
equality; bounce caret out of fully-collapsed source-formatting
whitespace before insert.

Includes baseline updates for editing-empty-divs, 5510537,
return-with-object-element, 4840662, insertparagraph_3001-4000, and
splits the flaky padding subtest of empty-elements-insertion.html.

Bug: 40547104
Change-Id: I736c63ccc7ccfefbc84f38ab16063a022535e5e4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7866236
Reviewed-by: Kent Tamura \<tkent@chromium.org>
Commit-Queue: Shweta Bindal \<shwetabindal@microsoft.com>
Cr-Commit-Position: refs/heads/main@{#1651514}

Component: web-platform-tests → DOM: Editor
Product: Testing → Core

CI Results

Ran 0 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI

Total 2 tests and 5 subtests

Status Summary

Firefox

OK : 2
PASS: 5
FAIL: 1

Chrome

OK : 2
PASS: 2
FAIL: 4

Safari

OK : 2
PASS: 2
FAIL: 4

Links

GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

You need to log in before you can comment on or make changes to this bug.