WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
171722
[GStreamer] Do not report more errors after the first one
https://bugs.webkit.org/show_bug.cgi?id=171722
Summary
[GStreamer] Do not report more errors after the first one
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+
Details
Formatted Diff
Diff
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.
Top of Page
Format For Printing
XML
Clone This Bug