RESOLVED INVALID 77925
Introduces Element::youngestShadowRoot() and Element::oldestShadowRoot() 
https://bugs.webkit.org/show_bug.cgi?id=77925
Summary Introduces Element::youngestShadowRoot() and Element::oldestShadowRoot()
Shinya Kawanaka
Reported 2012-02-06 18:47:43 PST
Instead of shadowRoot(), we should implement youngestShadowRoot() and oldestShadowRoot(). youngestShadowRoot() will be used in renderer, and oldestShadowRoot() will be used in elements.
Attachments
Add attachment proposed patch, testcase, etc.
Dominic Cooney
Comment 1 2012-02-06 19:49:24 PST
I think shadow roots should be a doubly linked list. new ShadowRoot() pushes on the front of the list. Tree flattening iterator uses the back pointer to get from the last child of a SR to the next sibling of the <shadow> element that output it. Do we need oldestShadowRoot? Could callers just walk the list of roots themselves. I feel like this is builtInShadowRoot with a different name.
Shinya Kawanaka
Comment 2 2012-02-06 23:04:45 PST
(In reply to comment #1) > I think shadow roots should be a doubly linked list. new ShadowRoot() pushes on the front of the list. Tree flattening iterator uses the back pointer to get from the last child of a SR to the next sibling of the <shadow> element that output it. > > Do we need oldestShadowRoot? Could callers just walk the list of roots themselves. I feel like this is builtInShadowRoot with a different name. Yeah, it's almost the same as builtinShadowRoot(). I and morrita discussed it. We concluded that it would be better to have it instead of having a reference for now, because it's too error-prone. Of course we can remove oldestShadowRoot later...
Shinya Kawanaka
Comment 3 2012-02-08 21:38:55 PST
Since we will introduce ShadowRootList in Bug 78069, I make this INVALID.
Note You need to log in before you can comment on or make changes to this bug.