Closed Bug 1875583 Opened 1 year ago Closed 1 year ago

[wpt-sync] Sync PR 44107 - Add getHTML() and modify getInnerHTML() accordingly [2/N]

Categories

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

task

Tracking

()

RESOLVED FIXED
124 Branch
Tracking Status
firefox124 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

(Blocks 2 open bugs, )

Details

(Whiteboard: [wptsync downstream])

Attachments

(2 files)

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

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

Mason Freed <masonf@chromium.org> wrote:

Add getHTML() and modify getInnerHTML() accordingly [2/N]

This CL adds a new getHTML() method, and modifies the existing
getInnerHTML() method accordingly. The new method follows the
discussion and conclusions here:
https://github.com/whatwg/html/issues/8867#issuecomment-1856696628

Essentially, that is:

  1. Provide a boolean option to getHTML() that says "please serialize
    opted-in shadow roots". The default for this option is false.
  2. Provide an option to getHTML() containing a sequence of shadowRoots
    that should be serialized, independent of whether they opted in via
    the flag above.

This work falls under these two chromestatus entries:
https://chromestatus.com/guide/editall/5081733588582400
https://chromestatus.com/guide/editall/5102952270528512
and these two blink-dev threads:
https://groups.google.com/a/chromium.org/g/blink-dev/c/PE4VwMjLVTo
https://groups.google.com/a/chromium.org/g/blink-dev/c/it0X7BOimKw

Bug: 1519972, 1517959
Change-Id: I5181a0702a12d550b4dab64c0c306ea2ccb25fa3
Reviewed-on: https://chromium-review.googlesource.com/5219591
WPT-Export-Revision: 4f5ae86d38587e24c2f21858468a1bca599f2839

Component: web-platform-tests → DOM: Core & HTML
Product: Testing → Core
Whiteboard: [wptsync downstream] → [wptsync downstream error]
Whiteboard: [wptsync downstream error] → [wptsync downstream]
Attached file Notable wpt changes
# CI Results Ran 9 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI Total 2 tests and 330 subtests ## Status Summary ### Firefox `OK` : 2 `FAIL`: 948 ### Chrome `OK` : 2 `PASS`: 330 `FAIL`: 618 ### Safari `OK` : 2 `FAIL`: 948 ## Links [Gecko CI (Treeherder)](https://treeherder.mozilla.org/#/jobs?repo=try&revision=d6938b7d43985510bc3c49ac7ca61d83458569c8) [GitHub PR Head](https://wpt.fyi/results/?sha=16c92cac9c5e8830dc57b2bb07202af233773a86&label=pr_head) [GitHub PR Base](https://wpt.fyi/results/?sha=16c92cac9c5e8830dc57b2bb07202af233773a86&label=pr_base) ## Details (See attachment for full changes)
Attached file Notable wpt changes
# CI Results Ran 9 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI Total 2 tests and 330 subtests ## Status Summary ### Firefox `OK` : 2 `FAIL`: 948 ### Chrome `OK` : 2 `PASS`: 330 `FAIL`: 618 ### Safari `OK` : 2 `FAIL`: 948 ## Links [Gecko CI (Treeherder)](https://treeherder.mozilla.org/#/jobs?repo=try&revision=d6938b7d43985510bc3c49ac7ca61d83458569c8) [GitHub PR Head](https://wpt.fyi/results/?sha=16c92cac9c5e8830dc57b2bb07202af233773a86&label=pr_head) [GitHub PR Base](https://wpt.fyi/results/?sha=16c92cac9c5e8830dc57b2bb07202af233773a86&label=pr_base) ## Details (See attachment for full changes)
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/bed358ab233d [wpt PR 44107] - Add getHTML() and modify getInnerHTML() accordingly [2/2], a=testonly https://hg.mozilla.org/integration/autoland/rev/ebf67c2a03fd [wpt PR 44107] - Update wpt metadata, a=testonly
Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 124 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: