As part of the fix for bug 144653, aria-menubar-menuitems.html was rewritten to be more cross-platform-friendly. Road to hell seems to be paved with good intentions. My guess is that for some reason accessibleElementById is failing on the win port. I'll see about modifying the test to use focusedElement again.
<rdar://problem/20927122>
Created attachment 252998 [details] Patch
I'm *hoping* the attached will solve it. Note that the test will still fail because the expectations tree found here [1] looks very Mac-like and not Win-like. But if we get a tree rather than a TypeError, it will be progress. :) [1]https://build.webkit.org/results/Apple%20Win%207%20Release%20(Tests)/r184217%20(51780)/accessibility/aria-menubar-menuitems-pretty-diff.html
Chris: I'm playing "guess how I broke Win". This is just a change to the layout test, which works as expected for Gtk and Mac. Unless you have a better idea, r?
See also bug 140798 which makes me suspect my hunch is correct.
Comment on attachment 252998 [details] Patch Clearing flags on attachment: 252998 Committed r184227: <http://trac.webkit.org/changeset/184227>
All reviewed patches have been landed. Closing bug.
Reopening. I don't believe the changed test will automatically pass. But I need the test to run to get the tree needed for the win platform expectations. When I have that, I'll attach it here.
Created attachment 253010 [details] Patch
Comment on attachment 253010 [details] Patch This is gardening. As a result of the change to the test, we're now getting an accessibility tree for Windows. The tree is consistent with the other ports and the original test results wrt Windows role mapping.
Comment on attachment 253010 [details] Patch Clearing flags on attachment: 253010 Committed r184239: <http://trac.webkit.org/changeset/184239>