New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 764609 link

Starred by 1 user
Status: Assigned
Owner:
jianli@chromium.org
Cc:
█ mkwst@chromium.org
creis@chromium.org
hayato@chromium.org
editing-dev@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , Chrome , Mac , Fuchsia
Pri: 3
Type: Bug



Sign in to add a comment

Don't use Javascript for shadow dom fixups

Project Member Reported by dcheng@chromium.org, Sep 13 2017 
https://chromium.googlesource.com/chromium/src/+/a63a88f29ba7037e3b47bd7c10e88b76fd9e9958/third_party/WebKit/Source/core/exported/LocalFrameClientImpl.cpp#194 has a blob of script to do some fixups on shadow DOM. I think this should at least be using the C++ versions of those calls rather than going through JS.

Comment 1 by hayato@chromium.org, Sep 13 2017

Components: -Blink>DOM>ShadowDOM
That sounds right. You can use Element::attachShadow directly in C++.

Let me remove ShadowDOM label because we use this label for an actionable item for dom-team.
►

Sign in to add a comment