Bug 171598 - REGRESSION(r216096): [GTK] Test accessibility/gtk/menu-list-unfocused-notifications.html is failing since r216096
Summary: REGRESSION(r216096): [GTK] Test accessibility/gtk/menu-list-unfocused-notific...
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Joanmarie Diggs (irc: joanie)
URL:
Keywords: Gtk, LayoutTestFailure
Depends on:
Blocks:
 
Reported: 2017-05-03 04:05 PDT by Carlos Garcia Campos
Modified: 2017-05-03 09:43 PDT (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Carlos Garcia Campos 2017-05-03 04:05:31 PDT
--- /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/accessibility/gtk/menu-list-unfocused-notifications-expected.txt
+++ /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/accessibility/gtk/menu-list-unfocused-notifications-actual.txt
@@ -9,7 +9,8 @@
 Focused element is: AXRole: AXButton
 
 Changing selected item in non-focused combobox.
-AXSelectedChildrenChanged: AXRole: AXComboBox
+AXChildrenAdded: AXRole: AXSection
+AXChildrenAdded: AXRole: AXComboBox
 Focused element is: AXRole: AXButton
 
 Changing selected item in non-focused combobox.
Comment 1 Carlos Garcia Campos 2017-05-03 04:06:39 PDT
The test added in r216096 is a different one, this one passed before.
Comment 2 Carlos Garcia Campos 2017-05-03 04:07:18 PDT
So, maybe it just needs a rebase, Joanie?
Comment 3 Joanmarie Diggs (irc: joanie) 2017-05-03 04:27:35 PDT
From just glancing at the output, we lost a selection-changed event.

That said, in looking into other issues related to ARIA 1.1, I realized that our accessibility selection-notification code needs to be burned to the ground so that a correct implementation can rise from its ashes. That's on my TODO list after I finish the ARIA 1.1 low-hanging fruit. (i.e. later this week)

So if you want to skip this one, go for it. Thanks for checking!!
Comment 4 Michael Catanzaro 2017-05-03 09:43:42 PDT
We should generally only skip individual tests if they are breaking other tests.