Bug 237078

Summary: Have SourceBufferParserWebM use SharedBuffer internally
Product: WebKit Reporter: Jean-Yves Avenard [:jya] <jean-yves.avenard>
Component: MediaAssignee: Jean-Yves Avenard [:jya] <jean-yves.avenard>
Status: RESOLVED FIXED    
Severity: Normal CC: eric.carlson, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: Other   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 237075    
Bug Blocks: 236754, 237472    
Attachments:
Description Flags
Patch
none
Patch for EWS
none
Patch
none
Patch for EWS
none
Patch
ews-feeder: commit-queue-
Patch
none
Patch for review
none
Patch none

Description Jean-Yves Avenard [:jya] 2022-02-23 03:50:54 PST 
Sub-task of bug 236754.

We can simplify the inner workings of SourceBufferParserWebM by using SharedBuffer instead of building CMBlockBuffer progressively. We can reduce the amount of memory used as CMBlockBuffer always allocate more memory than is actually needed.

It will also simplify the resolution of bug 236754.
Comment 1 Radar WebKit Bug Importer 2022-02-23 04:12:13 PST 
<rdar://problem/89348752>
Comment 2 Jean-Yves Avenard [:jya] 2022-02-23 05:14:53 PST 
Created attachment 452964 [details]
Patch
Comment 3 Jean-Yves Avenard [:jya] 2022-02-23 05:22:23 PST 
Created attachment 452967 [details]
Patch for EWS
Comment 4 Jean-Yves Avenard [:jya] 2022-02-24 05:36:33 PST 
Created attachment 453094 [details]
Patch
Comment 5 Jean-Yves Avenard [:jya] 2022-02-24 05:37:09 PST 
Created attachment 453095 [details]
Patch for EWS
Comment 6 Jean-Yves Avenard [:jya] 2022-02-24 14:27:33 PST 
Created attachment 453138 [details]
Patch
Comment 7 Jean-Yves Avenard [:jya] 2022-02-24 14:29:31 PST 
Created attachment 453139 [details]
Patch
Comment 8 Jean-Yves Avenard [:jya] 2022-02-24 18:53:49 PST 
Created attachment 453163 [details]
Patch for review
Comment 9 Jean-Yves Avenard [:jya] 2022-02-24 18:54:43 PST 
Created attachment 453164 [details]
Patch
Comment 10 EWS 2022-03-08 16:16:38 PST 
Committed r291020 (248194@main): <https://commits.webkit.org/248194@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 453163 [details].