Bug 214086

Summary: [MSE][GStreamer] Remove orphan code in SourceBufferPrivateGStreamer::append()
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, eocanha, ews-watchlist, gustavo, menard, pnormand, vjaquez
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Description Alicia Boya García 2020-07-08 09:04:11 PDT 
A refactor in r240784 missed this line, which was unreachable code
before and should have removed.

Instead, it has been run every time after sending an append to the
AppendPipeline, and it just happens it doesn't have visible
consequences.

This patch cleans that up removing that line. No visible behavior
changes are introduces.
Comment 1 Alicia Boya García 2020-07-08 09:05:19 PDT 
Created attachment 403786 [details]
Patch
Comment 2 Enrique Ocaña 2020-07-08 09:23:57 PDT 
I agree with this patch.

The changes on r240784 made AppendPipeline::pushNewBuffer() unable to return any error for early append failure. Therefore the call to sourceBufferPrivateAppendComplete() (which should happend only in case of error) wouldn't be needed anymore.
Comment 3 EWS 2020-07-09 01:29:39 PDT 
Committed r264161: <https://trac.webkit.org/changeset/264161>

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