Bug 174324 - AX: VoiceOver silent or skipping over time values on media player.
Summary: AX: VoiceOver silent or skipping over time values on media player.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Accessibility (show other bugs)
Version: WebKit Nightly Build
Hardware: All All
: P2 Normal
Assignee: Aaron Chu
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2017-07-10 15:24 PDT by Aaron Chu
Modified: 2017-07-27 07:53 PDT (History)
5 users (show)

See Also:

Attachments
Patch (5.51 KB, patch)
2017-07-13 16:40 PDT, Aaron Chu 		no flags Details | Formatted Diff | Diff
Archive of layout-test-results from ews122 for ios-simulator-wk2 (939.13 KB, application/zip)
2017-07-13 18:07 PDT, Build Bot 		no flags Details
Patch (12.92 KB, patch)
2017-07-26 18:25 PDT, Aaron Chu 		no flags Details | Formatted Diff | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Aaron Chu 2017-07-10 15:24:51 PDT 
<rdar://problem/32021784>
Comment 1 Radar WebKit Bug Importer 2017-07-10 15:25:13 PDT 
<rdar://problem/33223721>
Comment 2 Aaron Chu 2017-07-13 16:40:25 PDT 
Created attachment 315382 [details]
Patch
Comment 3 Build Bot 2017-07-13 18:07:25 PDT 
Comment on attachment 315382 [details]
Patch

Attachment 315382 [details] did not pass ios-sim-ews (ios-simulator-wk2):
Output: http://webkit-queues.webkit.org/results/4115877

New failing tests:
media/modern-media-controls/time-label/time-label.html
Comment 4 Build Bot 2017-07-13 18:07:26 PDT 
Created attachment 315388 [details]
Archive of layout-test-results from ews122 for ios-simulator-wk2

The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews122  Port: ios-simulator-wk2  Platform: Mac OS X 10.12.5
Comment 5 WebKit Commit Bot 2017-07-14 00:07:53 PDT 
Comment on attachment 315382 [details]
Patch

Clearing flags on attachment: 315382

Committed r219500: <http://trac.webkit.org/changeset/219500>
Comment 6 WebKit Commit Bot 2017-07-14 00:07:54 PDT 
All reviewed patches have been landed.  Closing bug.
Comment 7 Matt Lewis 2017-07-14 09:46:13 PDT 
The test that was added in the patch is failing consistently on iOS simulator.

Build results:
https://build.webkit.org/results/Apple%20iOS%2010%20Simulator%20Debug%20WK2%20(Tests)/r219502%20(2720)/results.html

Diff:
--- /Volumes/Data/slave/ios-simulator-10-debug-tests-wk2/build/layout-test-results/media/modern-media-controls/time-label/time-label-expected.txt
+++ /Volumes/Data/slave/ios-simulator-10-debug-tests-wk2/build/layout-test-results/media/modern-media-controls/time-label/time-label-actual.txt
@@ -14,8 +14,8 @@
 
 PASS elaspedLabel is "Elapsed"
 PASS remainingLabel is "Remaining"
-PASS accessibilityController.accessibleElementById('remaining').role is "AXRole: AXStaticText"
-PASS accessibilityController.accessibleElementById('elasped').role is "AXRole: AXStaticText"
+FAIL accessibilityController.accessibleElementById('remaining').role should be AXRole: AXStaticText. Was .
+FAIL accessibilityController.accessibleElementById('elasped').role should be AXRole: AXStaticText. Was .
 
 elapsedTimeLabels[3].element.textContent = 0:01
 elapsedTimeLabels[3].width = 27
Comment 8 Matt Lewis 2017-07-14 10:31:19 PDT 
Reverted r219500 for reason:

The test is consistently failing on iOS simulator.

Committed r219508: <http://trac.webkit.org/changeset/219508>
Comment 9 Aaron Chu 2017-07-26 18:25:39 PDT 
Created attachment 316503 [details]
Patch
Comment 10 WebKit Commit Bot 2017-07-27 07:53:08 PDT 
Comment on attachment 316503 [details]
Patch

Clearing flags on attachment: 316503

Committed r219983: <http://trac.webkit.org/changeset/219983>
Comment 11 WebKit Commit Bot 2017-07-27 07:53:10 PDT 
All reviewed patches have been landed.  Closing bug.