These should be replaced with shadow root access API which will be introduced by Bug
Created attachment 80166 [details] Patch
Hi Dimitri, could you could you take a look? I'd like to land this before your wave of changes arrives ;-o
Comment on attachment 80166 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=80166&action=review > Source/WebCore/ChangeLog:14 > + * dom/Node.cpp: > + (WebCore::Node::shadowAncestorNode): Added const > + (WebCore::Node::shadowTreeRootNode): Added const Recently Darin Adler and Eric Seidel have told that const Nodes in WebCore make no sense. I guess they object this change. How about removing const from mediaElement() instead?
I think Tamura-san's got this one! Also -- I am in the process of converting HTMLMediaElement. Can you take a different one, so that we don't step on each others toes? :)
Comment on attachment 80166 [details] Patch Kent-san, Dimitri, thank you for take a look. OK, so I'll cancel this at now and wait Dimitri's change. Thanks for your suggestion!
This should be a gimme now that media element conversion is done.
I'm back.
Created attachment 117550 [details] Patch
Hi Dimitri, could you take a look? This change should be much trivial compared to the size of ChangeLog ;-)
Comment on attachment 117550 [details] Patch Attachment 117550 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/10687863
Comment on attachment 117550 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=117550&action=review r- because of Chromium build failure. > Source/WebCore/html/shadow/MediaControls.cpp:42 > + Element* host = toShadowRoot(shadowNode->shadowTreeRootNode())->shadowHost(); You can use Node::shadowAncestorNode().
Created attachment 117560 [details] Patch
Comment on attachment 117560 [details] Patch Similar change was landed: http://trac.webkit.org/changeset/101810
http://trac.webkit.org/changeset/101810 Fixes this. *** This bug has been marked as a duplicate of bug 71410 ***