RESOLVED FIXED237078
Have SourceBufferParserWebM use SharedBuffer internally
https://bugs.webkit.org/show_bug.cgi?id=237078
Summary Have SourceBufferParserWebM use SharedBuffer internally
Jean-Yves Avenard [:jya]
Reported 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.
Attachments
Patch (30.99 KB, patch)
2022-02-23 05:14 PST, Jean-Yves Avenard [:jya]
no flags
Patch for EWS (40.03 KB, patch)
2022-02-23 05:22 PST, Jean-Yves Avenard [:jya]
no flags
Patch (31.22 KB, patch)
2022-02-24 05:36 PST, Jean-Yves Avenard [:jya]
no flags
Patch for EWS (40.70 KB, patch)
2022-02-24 05:37 PST, Jean-Yves Avenard [:jya]
no flags
Patch (31.26 KB, patch)
2022-02-24 14:27 PST, Jean-Yves Avenard [:jya]
ews-feeder: commit-queue-
Patch (40.75 KB, patch)
2022-02-24 14:29 PST, Jean-Yves Avenard [:jya]
no flags
Patch for review (31.24 KB, patch)
2022-02-24 18:53 PST, Jean-Yves Avenard [:jya]
no flags
Patch (40.71 KB, patch)
2022-02-24 18:54 PST, Jean-Yves Avenard [:jya]
no flags
Radar WebKit Bug Importer
Comment 1 2022-02-23 04:12:13 PST
Jean-Yves Avenard [:jya]
Comment 2 2022-02-23 05:14:53 PST
Jean-Yves Avenard [:jya]
Comment 3 2022-02-23 05:22:23 PST
Created attachment 452967 [details] Patch for EWS
Jean-Yves Avenard [:jya]
Comment 4 2022-02-24 05:36:33 PST
Jean-Yves Avenard [:jya]
Comment 5 2022-02-24 05:37:09 PST
Created attachment 453095 [details] Patch for EWS
Jean-Yves Avenard [:jya]
Comment 6 2022-02-24 14:27:33 PST
Jean-Yves Avenard [:jya]
Comment 7 2022-02-24 14:29:31 PST
Jean-Yves Avenard [:jya]
Comment 8 2022-02-24 18:53:49 PST
Created attachment 453163 [details] Patch for review
Jean-Yves Avenard [:jya]
Comment 9 2022-02-24 18:54:43 PST
EWS
Comment 10 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].
Note You need to log in before you can comment on or make changes to this bug.