New issue
Advanced search Search tips

Issue 882869 link

Starred by 1 user

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug


Participants' hotlists:
Modern-Media-Controls


Sign in to add a comment

Nested DOM removal in media controls

Project Member Reported by futhark@chromium.org, Sep 11

Issue description

MediaControlsImpl::RemovedFrom() which is called during a DOM removal mutation calls Hide(), which eventually calls MediaControlLoadingPanelElement::CleanupShadowDOM(), which does another DOM removal. This causes problems with Node::isConnected() not being up-to-date as expected for the inner removal. Ideally, I don't think we should have nested removals like this.

I needed to do a workaround for this for https://chromium-review.googlesource.com/c/chromium/src/+/1149300

 
Cc: steimel@chromium.org

Sign in to add a comment