RESOLVED FIXED Bug 151379
Focus ordering should respect slot elements 
https://bugs.webkit.org/show_bug.cgi?id=151379
Summary Focus ordering should respect slot elements
Ryosuke Niwa
Reported 2015-11-17 21:51:23 PST
As discussed in 2015 TPAC, we should create a new tab ordering scope at each shadow root and slot.
Attachments
Fixes the bug (38.94 KB, patch)
2016-05-16 03:11 PDT, Ryosuke Niwa 		no flags
DetailsFormatted DiffDiff
Archive of layout-test-results from ews126 for ios-simulator-wk2 (651.78 KB, application/zip)
2016-05-16 04:15 PDT, Build Bot 		no flags
Details
Fixed GTK builds (39.97 KB, patch)
2016-05-16 12:11 PDT, Ryosuke Niwa 		no flags
DetailsFormatted DiffDiff
Fixed test expectation for iOS (39.78 KB, patch)
2016-05-16 12:11 PDT, Ryosuke Niwa 		koivisto: review+
DetailsFormatted DiffDiff
Show Obsolete (3) View All Add attachment proposed patch, testcase, etc.
Ryosuke Niwa
Comment 1 2016-05-16 03:11:35 PDT
Created attachment 279015 [details] Fixes the bug
Build Bot
Comment 2 2016-05-16 04:15:29 PDT
Comment on attachment 279015 [details] Fixes the bug Attachment 279015 [details] did not pass ios-sim-ews (ios-simulator-wk2): Output: http://webkit-queues.webkit.org/results/1329907 New failing tests: fast/shadow-dom/focus-across-details-element.html fast/shadow-dom/focus-navigation-across-slots.html
Build Bot
Comment 3 2016-05-16 04:15:34 PDT
Created attachment 279016 [details] Archive of layout-test-results from ews126 for ios-simulator-wk2 The attached test failures were seen while running run-webkit-tests on the ios-sim-ews. Bot: ews126 Port: ios-simulator-wk2 Platform: Mac OS X 10.11.4
Ryosuke Niwa
Comment 4 2016-05-16 12:11:11 PDT
Created attachment 279032 [details] Fixed GTK builds
Ryosuke Niwa
Comment 5 2016-05-16 12:11:50 PDT
Created attachment 279033 [details] Fixed test expectation for iOS
Antti Koivisto
Comment 6 2016-05-16 13:10:45 PDT
Comment on attachment 279033 [details] Fixed test expectation for iOS View in context: https://bugs.webkit.org/attachment.cgi?id=279033&action=review > Source/WebCore/page/FocusController.cpp:148 > + ContainerNode* parent = node.parentNode(); I would use auto* for various Node*s in this patch.
Antti Koivisto
Comment 7 2016-05-16 13:11:03 PDT
r=me
Ryosuke Niwa
Comment 8 2016-05-16 13:25:43 PDT
Committed r200964: <http://trac.webkit.org/changeset/200964>
Note You need to log in before you can comment on or make changes to this bug.