Closed
Bug 988655
Opened 11 years ago
Closed 11 years ago
nsIContent::IsInAnonymousSubtree returns true for shadow DOM content
Categories
(Core :: DOM: Core & HTML, defect)
Tracking
()
RESOLVED
FIXED
mozilla31
Tracking | Status | |
---|---|---|
firefox31 | --- | fixed |
People
(Reporter: bholley, Assigned: bholley)
References
Details
(Whiteboard: [qa-])
Attachments
(1 file)
1.18 KB,
patch
|
mrbkap
:
review+
|
Details | Diff | Splinter Review |
The current implementation of this method is:
return IsInNativeAnonymousSubtree() || GetBindingParent() != nullptr;
Since shadow DOM nodes return a non-null GetBindingParent(), this returns true for those cases. Talking with wchen, it doesn't sound like it was intentional, and probably isn't something that callers are expecting. Though I am concerned that we're relying on various anonymous content features (like event retargetting) without realizing it.
I'm going to try fixing it - hopefully nothing breaks.
Assignee | ||
Comment 1•11 years ago
|
||
Attachment #8397497 -
Flags: review?(mrbkap)
Assignee | ||
Comment 2•11 years ago
|
||
Updated•11 years ago
|
Attachment #8397497 -
Flags: review?(mrbkap) → review+
Comment 3•11 years ago
|
||
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla31
(In reply to Bobby Holley (:bholley) from comment #0)
> I'm going to try fixing it - hopefully nothing breaks.
What could this break, potentially? Is there anything QA should be on the lookout for?
Flags: needinfo?(bobbyholley)
Assignee | ||
Comment 5•11 years ago
|
||
(In reply to Anthony Hughes, QA Mentor (:ashughes) from comment #4)
> (In reply to Bobby Holley (:bholley) from comment #0)
> > I'm going to try fixing it - hopefully nothing breaks.
>
> What could this break, potentially? Is there anything QA should be on the
> lookout for?
Possibly stuff related to Web Components. But the breakage I was expecting would have been visible on tinderbox, I think.
Flags: needinfo?(bobbyholley)
Comment 6•11 years ago
|
||
Another thing to keep in mind is that web components are pref'ed off and not exposed to the web by default. The only people using it are developers experimenting with the feature.
Thanks Bobby and William. I'm tagging this [qa-] for now. If you need testing around this please don't hesitate to needinfo me.
status-firefox31:
--- → fixed
Whiteboard: [qa-]
Updated•6 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•