Bug 226217 - Loading WebM hangs if server sends only small amount of data
Summary: Loading WebM hangs if server sends only small amount of data
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Media (show other bugs)
Version: Other
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Kimmo Kinnunen
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2021-05-25 04:37 PDT by Kimmo Kinnunen
Modified: 2021-06-29 10:12 PDT (History)
8 users (show)

See Also:

Attachments
Patch (3.65 KB, patch)
2021-05-25 04:47 PDT, Kimmo Kinnunen 		no flags Details | Formatted Diff | Diff
Patch (12.20 KB, patch)
2021-06-24 21:13 PDT, Jer Noble 		ews-feeder: commit-queue-
Details | Formatted Diff | Diff
Patch (12.21 KB, patch)
2021-06-24 21:18 PDT, Jer Noble 		ews-feeder: commit-queue-
Details | Formatted Diff | Diff
Patch (12.71 KB, patch)
2021-06-24 21:26 PDT, Jer Noble 		no flags Details | Formatted Diff | Diff
Patch (15.16 KB, patch)
2021-06-25 08:41 PDT, Jer Noble 		eric.carlson: review+
ews-feeder: commit-queue-
Details | Formatted Diff | Diff
Patch for landing (17.90 KB, patch)
2021-06-25 15:28 PDT, Jer Noble 		no flags Details | Formatted Diff | Diff
Patch for landing (17.74 KB, patch)
2021-06-25 15:33 PDT, Jer Noble 		ews-feeder: commit-queue-
Details | Formatted Diff | Diff
Patch for landing (17.42 KB, patch)
2021-06-28 09:40 PDT, Jer Noble 		no flags Details | Formatted Diff | Diff
Patch for landing (17.50 KB, patch)
2021-06-28 10:21 PDT, Jer Noble 		ews-feeder: commit-queue-
Details | Formatted Diff | Diff
Patch for landing (17.51 KB, patch)
2021-06-28 14:46 PDT, Jer Noble 		ews-feeder: commit-queue-
Details | Formatted Diff | Diff
Show Obsolete (8) 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 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>