We disables it in Bug 104644. We need to make FocusController::advanceFocus(FocusDirectionLeft/Right,...) workable for shadow nodes.
Created attachment 179218 [details] WIP
Comment on attachment 179218 [details] WIP Attachment 179218 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/15312195 New failing tests: fast/forms/date-multiple-fields/date-multiple-fields-keyboard-events.html
Created attachment 179405 [details] Patch
Comment on attachment 179405 [details] Patch Attachment 179405 [details] did not pass mac-ews (mac): Output: http://queues.webkit.org/results/15309647 New failing tests: fast/spatial-navigation/snav-media-elements.html
Created attachment 179684 [details] Patch 2 Fix media elements failure
Comment on attachment 179684 [details] Patch 2 View in context: https://bugs.webkit.org/attachment.cgi?id=179684&action=review > Source/WebCore/dom/NodeTraversal.cpp:132 > +namespace ElementTraversalWithShadow { Can we use ComposedShadowTreeWalker to traverse nodes in shadow trees? It seems a kind of duplications for me.
Comment on attachment 179684 [details] Patch 2 View in context: https://bugs.webkit.org/attachment.cgi?id=179684&action=review >> Source/WebCore/dom/NodeTraversal.cpp:132 >> +namespace ElementTraversalWithShadow { > > Can we use ComposedShadowTreeWalker to traverse nodes in shadow trees? > It seems a kind of duplications for me. I don't know because I don't understand ComposedShadowTreeWalker well. - What is "composed" shadow tree? - Will it be wrapped with ENABLE_SHADOW_DOM? - Does it work for non-shadow nodes? - Is the traversal order same as NodeTraversal/ElementTraversal? - Does it have a function like NodeTraversal::nextSkippingChildren? - What is UpperBoundary?
Comment on attachment 179684 [details] Patch 2 View in context: https://bugs.webkit.org/attachment.cgi?id=179684&action=review >>> Source/WebCore/dom/NodeTraversal.cpp:132 >>> +namespace ElementTraversalWithShadow { >> >> Can we use ComposedShadowTreeWalker to traverse nodes in shadow trees? >> It seems a kind of duplications for me. > > I don't know because I don't understand ComposedShadowTreeWalker well. > > - What is "composed" shadow tree? > - Will it be wrapped with ENABLE_SHADOW_DOM? > - Does it work for non-shadow nodes? > - Is the traversal order same as NodeTraversal/ElementTraversal? > - Does it have a function like NodeTraversal::nextSkippingChildren? > - What is UpperBoundary? I added NodeRenderingTraversal.h (http://trac.webkit.org/browser/trunk/Source/WebCore/dom/NodeRenderingTraversal.h) That is basically SHADOW_DOM flag agnostic version of shadow tree traversal routines. It would be great if we can make these to fit there. I'm thinking about hiding ComposedShadowTreeWalker behind the flag. But it's long way to go.
(In reply to comment #7) > - Does it have a function like NodeTraversal::nextSkippingChildren? I found CSTW didn't have this feature. Also, CSTW doesn't have the stayWithin feature.
Created attachment 183397 [details] Patch 3 Use CSTW
We already shipped this behavior for LTR locales a few months ago, and we haven't seen any requests to support RTL locales. IMO we don't need to do this change yet.