233791 2021-12-02 16:30:49 -0800 AX: Attempting to play local media files as part of a speech study causes Safari to hang. 2021-12-03 09:45:01 -0800 1 1 1 Unclassified WebKit Accessibility WebKit Nightly Build All All RESOLVED WONTFIX InRadar P2 Normal --- 1 cfleizach cfleizach andresg_22 cdumez eric.carlson ews-watchlist glenn jer.noble philipj sergio tyler_w webkit-bug-importer oldest_to_newest 1819927 0 cfleizach 2021-12-02 16:30:49 -0800 1. Enable VoiceOver. 2. Navigate using VoiceOver to the first audio file and press space bar when focused on the play button. Actual Results: Safari does not play the audio and instead becomes unresponsive. Expected Results: Playing the files should work as expected, even if they are local. <rdar://problem/85990360> 1819928 1 webkit-bug-importer 2021-12-02 16:31:05 -0800 <rdar://problem/85994691> 1819933 2 445790 cfleizach 2021-12-02 16:33:57 -0800 Created attachment 445790 patch 1819941 3 445790 tyler_w 2021-12-02 16:46:36 -0800 Comment on attachment 445790 patch View in context: https://bugs.webkit.org/attachment.cgi?id=445790&action=review > Source/WebKit/ChangeLog:9 > + If we send a sync message after user interaction, we need to inform that the process will susped so that VoiceOver doesn't get stuck. Typo. susped --> suspend 1819970 4 445790 eric.carlson 2021-12-02 17:29:56 -0800 Comment on attachment 445790 patch View in context: https://bugs.webkit.org/attachment.cgi?id=445790&action=review > Source/WebKit/ChangeLog:9 > + If we send a sync message after user interaction, we need to inform that the process will susped so that VoiceOver doesn't get stuck. s/susped/suspend/ 1820021 5 445806 cfleizach 2021-12-02 21:36:07 -0800 Created attachment 445806 patch 1820122 6 andresg_22 2021-12-03 07:23:58 -0800 (In reply to chris fleizach from comment #5) > Created attachment 445806 [details] > patch Couldn't we change sendSync(...) to SendSyncOption::InformPlatformProcessWillSuspend by default? So that it doesn't break again next time somebody adds another call. 1820125 7 cfleizach 2021-12-03 07:28:35 -0800 (In reply to Andres Gonzalez from comment #6) > (In reply to chris fleizach from comment #5) > > Created attachment 445806 [details] > > patch > > Couldn't we change sendSync(...) to > SendSyncOption::InformPlatformProcessWillSuspend by default? So that it > doesn't break again next time somebody adds another call. I gather there are a lot of sync calls. When we inform it will suspend it causes VO to not be able to interact with the app until it’s done. Given the lag in posting and processing these changes I think it would break normal navigation. Our problem is with synchronous calls that wait on safari to do expensive work. Most of the send sync calls are fairly straightforward. This is my impression on why we can’t just do this always 1820186 8 cfleizach 2021-12-03 09:45:01 -0800 Looked into this more and something is not lining up. this patch would allow VO to navigate Safari again, but that won't help the WebContnt process being blocked. 445790 2021-12-02 16:33:57 -0800 2021-12-02 21:36:07 -0800 patch file_233791.txt text/plain 1834 cfleizach ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nIGIvU291cmNlL1dlYktpdC9DaGFu Z2VMb2cNCmluZGV4IGM0MWQ4YzNmY2Q1My4uYzRjZjAyNTExMjE1IDEwMDY0NA0KLS0tIGEvU291 cmNlL1dlYktpdC9DaGFuZ2VMb2cNCisrKyBiL1NvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nDQpAQCAt MSwzICsxLDE2IEBADQorMjAyMS0xMi0wMiAgQ2hyaXMgRmxlaXphY2ggIDxjZmxlaXphY2hAYXBw bGUuY29tPg0KKw0KKyAgICAgICAgQVg6IEF0dGVtcHRpbmcgdG8gcGxheSBsb2NhbCBtZWRpYSBm aWxlcyBhcyBwYXJ0IG9mIGEgc3BlZWNoIHN0dWR5IGNhdXNlcyBTYWZhcmkgdG8gaGFuZy4NCisg ICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0yMzM3OTENCisg ICAgICAgIDxyZGFyOi8vcHJvYmxlbS84NTk5NDY5MT4NCisNCisgICAgICAgIFJldmlld2VkIGJ5 IE5PQk9EWSAoT09QUyEpLg0KKw0KKyAgICAgICAgSWYgd2Ugc2VuZCBhIHN5bmMgbWVzc2FnZSBh ZnRlciB1c2VyIGludGVyYWN0aW9uLCB3ZSBuZWVkIHRvIGluZm9ybSB0aGF0IHRoZSBwcm9jZXNz IHdpbGwgc3VzcGVkIHNvIHRoYXQgVm9pY2VPdmVyIGRvZXNuJ3QgZ2V0IHN0dWNrLg0KKw0KKyAg ICAgICAgKiBXZWJQcm9jZXNzL0dQVS9tZWRpYS9SZW1vdGVBdWRpb1Nlc3Npb24uY3BwOg0KKyAg ICAgICAgKFdlYktpdDo6UmVtb3RlQXVkaW9TZXNzaW9uOjp0cnlUb1NldEFjdGl2ZUludGVybmFs KToNCisNCiAyMDIxLTEyLTAyICBNZWdhbiBHYXJkbmVyICA8bWVnYW5fZ2FyZG5lckBhcHBsZS5j b20+DQogDQogICAgICAgICBSZS1hZGQgc3VwcG9ydCBvZiBpbWFnZSBjb250cm9sIG1lbnVzLg0K ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQvV2ViUHJvY2Vzcy9HUFUvbWVkaWEvUmVtb3RlQXVk aW9TZXNzaW9uLmNwcCBiL1NvdXJjZS9XZWJLaXQvV2ViUHJvY2Vzcy9HUFUvbWVkaWEvUmVtb3Rl QXVkaW9TZXNzaW9uLmNwcA0KaW5kZXggMDhlZmY5OGFhN2IzLi40Y2RhZWViMGUzOTUgMTAwNjQ0 DQotLS0gYS9Tb3VyY2UvV2ViS2l0L1dlYlByb2Nlc3MvR1BVL21lZGlhL1JlbW90ZUF1ZGlvU2Vz c2lvbi5jcHANCisrKyBiL1NvdXJjZS9XZWJLaXQvV2ViUHJvY2Vzcy9HUFUvbWVkaWEvUmVtb3Rl QXVkaW9TZXNzaW9uLmNwcA0KQEAgLTExNyw3ICsxMTcsNyBAQCB2b2lkIFJlbW90ZUF1ZGlvU2Vz c2lvbjo6c2V0UHJlZmVycmVkQnVmZmVyU2l6ZShzaXplX3Qgc2l6ZSkNCiBib29sIFJlbW90ZUF1 ZGlvU2Vzc2lvbjo6dHJ5VG9TZXRBY3RpdmVJbnRlcm5hbChib29sIGFjdGl2ZSkNCiB7DQogICAg IGJvb2wgc3VjY2VlZGVkOw0KLSAgICBlbnN1cmVDb25uZWN0aW9uKCkuc2VuZFN5bmMoTWVzc2Fn ZXM6OlJlbW90ZUF1ZGlvU2Vzc2lvblByb3h5OjpUcnlUb1NldEFjdGl2ZShhY3RpdmUpLCBNZXNz YWdlczo6UmVtb3RlQXVkaW9TZXNzaW9uUHJveHk6OlRyeVRvU2V0QWN0aXZlOjpSZXBseShzdWNj ZWVkZWQpLCB7IH0pOw0KKyAgICBlbnN1cmVDb25uZWN0aW9uKCkuc2VuZFN5bmMoTWVzc2FnZXM6 OlJlbW90ZUF1ZGlvU2Vzc2lvblByb3h5OjpUcnlUb1NldEFjdGl2ZShhY3RpdmUpLCBNZXNzYWdl czo6UmVtb3RlQXVkaW9TZXNzaW9uUHJveHk6OlRyeVRvU2V0QWN0aXZlOjpSZXBseShzdWNjZWVk ZWQpLCAwLCBJUEM6OlRpbWVvdXQ6OmluZmluaXR5KCksIElQQzo6U2VuZFN5bmNPcHRpb246Oklu Zm9ybVBsYXRmb3JtUHJvY2Vzc1dpbGxTdXNwZW5kKTsNCiAgICAgaWYgKHN1Y2NlZWRlZCkNCiAg ICAgICAgIGNvbmZpZ3VyYXRpb24oKS5pc0FjdGl2ZSA9IGFjdGl2ZTsNCiAgICAgcmV0dXJuIHN1 Y2NlZWRlZDsNCg== 445806 2021-12-02 21:36:07 -0800 2021-12-03 07:25:16 -0800 patch file_233791.txt text/plain 1835 cfleizach ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nIGIvU291cmNlL1dlYktpdC9DaGFu Z2VMb2cNCmluZGV4IGM0MWQ4YzNmY2Q1My4uYjNmNTc3MDU1ODZiIDEwMDY0NA0KLS0tIGEvU291 cmNlL1dlYktpdC9DaGFuZ2VMb2cNCisrKyBiL1NvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nDQpAQCAt MSwzICsxLDE2IEBADQorMjAyMS0xMi0wMiAgQ2hyaXMgRmxlaXphY2ggIDxjZmxlaXphY2hAYXBw bGUuY29tPg0KKw0KKyAgICAgICAgQVg6IEF0dGVtcHRpbmcgdG8gcGxheSBsb2NhbCBtZWRpYSBm aWxlcyBhcyBwYXJ0IG9mIGEgc3BlZWNoIHN0dWR5IGNhdXNlcyBTYWZhcmkgdG8gaGFuZy4NCisg ICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0yMzM3OTENCisg ICAgICAgIDxyZGFyOi8vcHJvYmxlbS84NTk5NDY5MT4NCisNCisgICAgICAgIFJldmlld2VkIGJ5 IE5PQk9EWSAoT09QUyEpLg0KKw0KKyAgICAgICAgSWYgd2Ugc2VuZCBhIHN5bmMgbWVzc2FnZSBh ZnRlciB1c2VyIGludGVyYWN0aW9uLCB3ZSBuZWVkIHRvIGluZm9ybSB0aGF0IHRoZSBwcm9jZXNz IHdpbGwgc3VzcGVuZCBzbyB0aGF0IFZvaWNlT3ZlciBkb2Vzbid0IGdldCBzdHVjay4NCisNCisg ICAgICAgICogV2ViUHJvY2Vzcy9HUFUvbWVkaWEvUmVtb3RlQXVkaW9TZXNzaW9uLmNwcDoNCisg ICAgICAgIChXZWJLaXQ6OlJlbW90ZUF1ZGlvU2Vzc2lvbjo6dHJ5VG9TZXRBY3RpdmVJbnRlcm5h bCk6DQorDQogMjAyMS0xMi0wMiAgTWVnYW4gR2FyZG5lciAgPG1lZ2FuX2dhcmRuZXJAYXBwbGUu Y29tPg0KIA0KICAgICAgICAgUmUtYWRkIHN1cHBvcnQgb2YgaW1hZ2UgY29udHJvbCBtZW51cy4N CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0L1dlYlByb2Nlc3MvR1BVL21lZGlhL1JlbW90ZUF1 ZGlvU2Vzc2lvbi5jcHAgYi9Tb3VyY2UvV2ViS2l0L1dlYlByb2Nlc3MvR1BVL21lZGlhL1JlbW90 ZUF1ZGlvU2Vzc2lvbi5jcHANCmluZGV4IDA4ZWZmOThhYTdiMy4uNGNkYWVlYjBlMzk1IDEwMDY0 NA0KLS0tIGEvU291cmNlL1dlYktpdC9XZWJQcm9jZXNzL0dQVS9tZWRpYS9SZW1vdGVBdWRpb1Nl c3Npb24uY3BwDQorKysgYi9Tb3VyY2UvV2ViS2l0L1dlYlByb2Nlc3MvR1BVL21lZGlhL1JlbW90 ZUF1ZGlvU2Vzc2lvbi5jcHANCkBAIC0xMTcsNyArMTE3LDcgQEAgdm9pZCBSZW1vdGVBdWRpb1Nl c3Npb246OnNldFByZWZlcnJlZEJ1ZmZlclNpemUoc2l6ZV90IHNpemUpDQogYm9vbCBSZW1vdGVB dWRpb1Nlc3Npb246OnRyeVRvU2V0QWN0aXZlSW50ZXJuYWwoYm9vbCBhY3RpdmUpDQogew0KICAg ICBib29sIHN1Y2NlZWRlZDsNCi0gICAgZW5zdXJlQ29ubmVjdGlvbigpLnNlbmRTeW5jKE1lc3Nh Z2VzOjpSZW1vdGVBdWRpb1Nlc3Npb25Qcm94eTo6VHJ5VG9TZXRBY3RpdmUoYWN0aXZlKSwgTWVz c2FnZXM6OlJlbW90ZUF1ZGlvU2Vzc2lvblByb3h5OjpUcnlUb1NldEFjdGl2ZTo6UmVwbHkoc3Vj Y2VlZGVkKSwgeyB9KTsNCisgICAgZW5zdXJlQ29ubmVjdGlvbigpLnNlbmRTeW5jKE1lc3NhZ2Vz OjpSZW1vdGVBdWRpb1Nlc3Npb25Qcm94eTo6VHJ5VG9TZXRBY3RpdmUoYWN0aXZlKSwgTWVzc2Fn ZXM6OlJlbW90ZUF1ZGlvU2Vzc2lvblByb3h5OjpUcnlUb1NldEFjdGl2ZTo6UmVwbHkoc3VjY2Vl ZGVkKSwgMCwgSVBDOjpUaW1lb3V0OjppbmZpbml0eSgpLCBJUEM6OlNlbmRTeW5jT3B0aW9uOjpJ bmZvcm1QbGF0Zm9ybVByb2Nlc3NXaWxsU3VzcGVuZCk7DQogICAgIGlmIChzdWNjZWVkZWQpDQog ICAgICAgICBjb25maWd1cmF0aW9uKCkuaXNBY3RpdmUgPSBhY3RpdmU7DQogICAgIHJldHVybiBz dWNjZWVkZWQ7DQo=