Bug 100887 – [chromium] platform/chromium/accessibility/other-aria-attribute-change-sends-notification.html failing

• Status: RESOLVED
• Resolution: FIXED
• Component: Accessibility

[Stephen White]

Started failing after r133069.

Looks like it just changed the order of TEXT results, so probably can be rebaselined:

E.g., 

--- /mnt/data/b/build/slave/WebKit_Linux/build/layout-test-results/platform/chromium/accessibility/other-aria-attribute-change-sends-notification-expected.txt 
+++ /mnt/data/b/build/slave/WebKit_Linux/build/layout-test-results/platform/chromium/accessibility/other-aria-attribute-change-sends-notification-actual.txt 
@@ -4,12 +4,12 @@
 
 
 
-PASS successfullyParsed is true
-
-TEST COMPLETE
 Got AriaAttributeChanged notification on aria-busy
 Got AriaAttributeChanged notification on aria-disabled
 Got AriaAttributeChanged notification on aria-readonly
 Got AriaAttributeChanged notification on aria-required
 All notifications received successfully.
+PASS successfullyParsed is true
 
+TEST COMPLETE
+

[Stephen White]

platform/chromium/accessibility/is-ignored-change-sends-notification.html also failing w/similar symptoms (just changes order of text results).

[Tony Chang]

The tests are manually calling waitUntilDone/notifyDone instead of setting window.jsTestIsAsync = true.  That should delay the call to finishJSTest which writes those log lines.

[Stephen White]

Added platform/chromium/accessibility/text-change-notification.html.

Now that we have more data, it looks like they're actually flaky since the cacheing change.  Marked as such.  Looks like this'll need an actual fix.

[Tony Chang]

Created attachment 172122 [details]
Patch

[WebKit Review Bot]

Comment on attachment 172122 [details]
Patch

Clearing flags on attachment: 172122

Committed r133347: <http://trac.webkit.org/changeset/133347>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.