Bug 254443

Summary: AX: Only FRED VP visible on the Text To Speech Sound Check page
Product: WebKit Reporter: chris fleizach <cfleizach>
Component: AccessibilityAssignee: chris fleizach <cfleizach>
Status: RESOLVED FIXED    
Severity: Normal CC: andresg_22, tyler_w, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch
none
Patch
none
Patch
none
Patch none

chris fleizach
Reported 2023-03-24 12:52:39 PDT
Only Fred is available sometimes
Attachments
Patch (3.46 KB, patch)
2023-03-24 13:38 PDT, chris fleizach 		no flags
DetailsFormatted DiffDiff
Patch (4.36 KB, patch)
2023-03-24 15:19 PDT, chris fleizach 		no flags
DetailsFormatted DiffDiff
Patch (4.62 KB, patch)
2023-03-24 15:32 PDT, chris fleizach 		no flags
DetailsFormatted DiffDiff
Patch (5.72 KB, patch)
2023-03-24 16:52 PDT, chris fleizach 		no flags
DetailsFormatted DiffDiff
Show Obsolete (3) View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2023-03-24 12:52:49 PDT
<rdar://problem/107199586>
chris fleizach
Comment 2 2023-03-24 13:38:26 PDT
Created attachment 465575 [details] Patch
chris fleizach
Comment 3 2023-03-24 15:19:21 PDT
Created attachment 465576 [details] Patch
chris fleizach
Comment 4 2023-03-24 15:32:01 PDT
Created attachment 465577 [details] Patch
Tyler Wilcock
Comment 5 2023-03-24 15:58:28 PDT
Comment on attachment 465577 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=465577&action=review > COMMIT_MESSAGE:2 > +https://bugs.webkit.org/show_bug.cgi?id=254443 I think you need a Radar link below your Bugzilla URL. If you re-run `git-webkit setup` this should start appearing in your commit message template. > LayoutTests/fast/speechsynthesis/mac/samantha-voice-available.html:9 > + description("This tests that we can get Samantha."); Can probably move everything inside this script tag back one indentation level. > LayoutTests/fast/speechsynthesis/mac/samantha-voice-available.html:12 > + var speech = window.speechSynthesis; > + var list = speech.getVoices(); Seems like we only use this `var speech` in one place. Might make the test a bit more clean to directly use window.speechSynthesis to `getVoices()`. > LayoutTests/fast/speechsynthesis/mac/samantha-voice-available.html:22 > + This newline seems unnecessary.
chris fleizach
Comment 6 2023-03-24 16:52:54 PDT
Created attachment 465578 [details] Patch
EWS
Comment 7 2023-03-25 15:20:49 PDT
Committed 262117@main (f20573ec09c6): <https://commits.webkit.org/262117@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 465578 [details].
Note You need to log in before you can comment on or make changes to this bug.