Bug 214128

Summary: [MSE][GStreamer] Don't cache duration in MediaSourceClientGStreamerMSE
Product: WebKit Reporter: Alicia Boya García <aboya>
Component: WebKitGTKAssignee: Alicia Boya García <aboya>
Status: RESOLVED FIXED    
Severity: Normal CC: bugs-noreply, calvaris, cgarcia, eric.carlson, ews-watchlist, glenn, gustavo, jer.noble, menard, philipj, pnormand, sergio, vjaquez
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Alicia Boya García
Reported 2020-07-09 04:45:17 PDT
MediaSource should be the single source of truth for the duration of the MediaSource, and querying it to MediaSource is efficient enough (trivial getter). There is no reason for MediaSourceClientGStreamerMSE to store a separate m_duration field. This patch introduces no behavior changes.
Attachments
Patch (3.72 KB, patch)
2020-07-09 04:46 PDT, Alicia Boya García 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Alicia Boya García
Comment 1 2020-07-09 04:46:22 PDT
Created attachment 403855 [details] Patch
EWS
Comment 2 2020-07-09 05:30:26 PDT
Committed r264166: <https://trac.webkit.org/changeset/264166> All reviewed patches have been landed. Closing bug and clearing flags on attachment 403855 [details].
Note You need to log in before you can comment on or make changes to this bug.