Skip to content

webrtc wpt: attach MediaStream to element attached to the DOM#44742

Merged
chromium-wpt-export-bot merged 1 commit into
masterfrom
chromium-export-cl-5318131
Feb 23, 2024
Merged

webrtc wpt: attach MediaStream to element attached to the DOM#44742
chromium-wpt-export-bot merged 1 commit into
masterfrom
chromium-export-cl-5318131

Conversation

@chromium-wpt-export-bot

@chromium-wpt-export-bot chromium-wpt-export-bot commented Feb 23, 2024

Copy link
Copy Markdown
Collaborator

which is required to make some browsers render audio

BUG=40115293

Change-Id: I3206ed0b6436ee15cc25a5ce05fc806321693e0e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5318131
Commit-Queue: Philipp Hancke <[email protected]>
Reviewed-by: Harald Alvestrand <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1264569}

which is required to make some browsers render audio

BUG=40115293

Change-Id: I3206ed0b6436ee15cc25a5ce05fc806321693e0e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5318131
Commit-Queue: Philipp Hancke <[email protected]>
Reviewed-by: Harald Alvestrand <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1264569}

@wpt-pr-bot wpt-pr-bot left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The review process for this patch is being conducted in the Chromium project.

@chromium-wpt-export-bot chromium-wpt-export-bot merged commit 123c162 into master Feb 23, 2024
@chromium-wpt-export-bot chromium-wpt-export-bot deleted the chromium-export-cl-5318131 branch February 23, 2024 16:36
@fippo

fippo commented Feb 23, 2024

Copy link
Copy Markdown
Contributor

need to clean up metadata next week similar to web-platform-tests/wpt-metadata#5114

servo-wpt-sync pushed a commit that referenced this pull request May 5, 2026
The state of the document after `range.deleteContents` leads
to the delete command to traverse to the root element and
then crash. Instead, we should simply stop the loop and
continue the algorithm and delete backwards.

Fixes #44742

Signed-off-by: Tim van der Lippe <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants