Bug 226217

Summary: Loading WebM hangs if server sends only small amount of data
Product: WebKit Reporter: Kimmo Kinnunen <kkinnunen>
Component: MediaAssignee: Kimmo Kinnunen <kkinnunen>
Status: RESOLVED FIXED    
Severity: Normal CC: eric.carlson, ews-watchlist, glenn, jer.noble, mjs, philipj, sergio, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: Other   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=226128
https://bugs.webkit.org/show_bug.cgi?id=226078
https://bugs.webkit.org/show_bug.cgi?id=223139
Attachments:
Description Flags
Patch
none
Patch
ews-feeder: commit-queue-
Patch
ews-feeder: commit-queue-
Patch
none
Patch
eric.carlson: review+, ews-feeder: commit-queue-
Patch for landing
none
Patch for landing
ews-feeder: commit-queue-
Patch for landing
none
Patch for landing
ews-feeder: commit-queue-
Patch for landing ews-feeder: commit-queue-

Description Kimmo Kinnunen 2021-05-25 04:37:40 PDT 
Loading WebM hangs if server sends only small amount of data

If the data is less than the initial metadata, interacting with the media player will hang the browser.
There are various synchronous queries in the WebM format reader that cannot be answered synchronously.
Comment 1 Kimmo Kinnunen 2021-05-25 04:47:07 PDT 
Created attachment 429642 [details]
Patch
Comment 2 Radar WebKit Bug Importer 2021-06-01 04:38:16 PDT 
<rdar://problem/78709010>
Comment 3 Jer Noble 2021-06-24 21:13:09 PDT 
Created attachment 432232 [details]
Patch
Comment 4 Jer Noble 2021-06-24 21:18:08 PDT 
Created attachment 432233 [details]
Patch
Comment 5 Jer Noble 2021-06-24 21:26:05 PDT 
Created attachment 432234 [details]
Patch
Comment 6 Jer Noble 2021-06-25 08:41:53 PDT 
Created attachment 432275 [details]
Patch
Comment 7 Jer Noble 2021-06-25 13:43:48 PDT 
The AS debug failure is a test infrastructure problem. WebKitTestRunner uses an injected bundle to enable running the plugin format reader when it's not signed, but that code is only injected into the WebContent process, and not the GPU process.
Comment 8 Jer Noble 2021-06-25 15:28:58 PDT 
Created attachment 432298 [details]
Patch for landing
Comment 9 Jer Noble 2021-06-25 15:33:53 PDT 
Created attachment 432299 [details]
Patch for landing
Comment 10 Jer Noble 2021-06-28 09:40:08 PDT 
Created attachment 432400 [details]
Patch for landing
Comment 11 Jer Noble 2021-06-28 10:21:20 PDT 
Created attachment 432406 [details]
Patch for landing
Comment 12 Jer Noble 2021-06-28 14:46:59 PDT 
Created attachment 432424 [details]
Patch for landing
Comment 13 Jer Noble 2021-06-28 16:05:54 PDT 
<rdar://78622569>
Comment 14 EWS 2021-06-28 16:06:12 PDT 
NOBODY Eric Carlson found in /Volumes/Data/worker/Commit-Queue/build/LayoutTests/ChangeLog does not appear to be a valid reviewer according to contributors.json.
/Volumes/Data/worker/Commit-Queue/build/LayoutTests/ChangeLog neither lists a valid reviewer nor contains the string "Unreviewed" or "Rubber stamp" (case insensitive).
Comment 15 Maciej Stachowiak 2021-06-28 20:18:41 PDT 
LayoutTests/ChangeLog is the one that mistakenly says NOBODY Eric Carlson.
Comment 16 Jer Noble 2021-06-29 10:12:59 PDT 
Committed r279375 (239241@main): <https://commits.webkit.org/239241@main>