Bug 171722

Summary: [GStreamer] Do not report more errors after the first one
Product: WebKit Reporter: Carlos Garcia Campos <cgarcia>
Component: PlatformAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: bugs-noreply, calvaris
Priority: P2 Keywords: Gtk, LayoutTestFailure
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch calvaris: review+

Carlos Garcia Campos
Reported 2017-05-05 04:05:34 PDT
We can receive several error messages for the same error from different elements. That's confusing the media source selection algorithm implementation. I don't know if didn't happen with previous versions of GST, but since the upgrade to 1.10.4 several tests are failing because of this.
Attachments
Patch (2.32 KB, patch)
2017-05-05 04:07 PDT, Carlos Garcia Campos 		calvaris: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Carlos Garcia Campos
Comment 1 2017-05-05 04:07:32 PDT
Created attachment 309158 [details] Patch
Xabier Rodríguez Calvar
Comment 2 2017-05-05 04:57:04 PDT
Comment on attachment 309158 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=309158&action=review > Source/WebCore/ChangeLog:8 > + We can receive several error messages for the same error from different elements. That's confusing the media ... That's confusing FOR the media source ...
Carlos Garcia Campos
Comment 3 2017-05-05 05:00:21 PDT
(In reply to Xabier Rodríguez Calvar from comment #2) > Comment on attachment 309158 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=309158&action=review > > > Source/WebCore/ChangeLog:8 > > + We can receive several error messages for the same error from different elements. That's confusing the media > > ... That's confusing FOR the media source ... I meant the media source alg impl gets confused by this. Not sure it's correct English, though.
Carlos Garcia Campos
Comment 4 2017-05-05 05:05:49 PDT
Committed r216240: <http://trac.webkit.org/changeset/216240>
Note You need to log in before you can comment on or make changes to this bug.