Bug 186040

Summary: [GStreamer] Update "qtdemux: Clarify field name about stream-encryption-system" patch
Product: WebKit Reporter: Thibault Saunier <tsaunier>
Component: New BugsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: calvaris, commit-queue, pnormand, tsaunier, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: All   
OS: All   
Attachments:
Description Flags
[GStreamer] Update "qtdemux: Clarify field name about stream-encryption-system" patch none

Thibault Saunier
Reported 2018-05-28 14:07:47 PDT
Uploaded one was sensibly modified before merging.
Attachments
[GStreamer] Update "qtdemux: Clarify field name about stream-encryption-system" patch (8.18 KB, patch)
2018-05-28 14:11 PDT, Thibault Saunier 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Thibault Saunier
Comment 1 2018-05-28 14:11:54 PDT
Created attachment 341459 [details] [GStreamer] Update "qtdemux: Clarify field name about stream-encryption-system" patch
Xabier Rodríguez Calvar
Comment 2 2018-05-29 01:18:57 PDT
Comment on attachment 341459 [details] [GStreamer] Update "qtdemux: Clarify field name about stream-encryption-system" patch Lol, I check the "important" things like changelogs and such and I miss this typo.
Xabier Rodríguez Calvar
Comment 3 2018-05-29 01:24:25 PDT
Comment on attachment 341459 [details] [GStreamer] Update "qtdemux: Clarify field name about stream-encryption-system" patch I though something was missing but it was ok
WebKit Commit Bot
Comment 4 2018-05-29 01:45:23 PDT
Comment on attachment 341459 [details] [GStreamer] Update "qtdemux: Clarify field name about stream-encryption-system" patch Clearing flags on attachment: 341459 Committed r232254: <https://trac.webkit.org/changeset/232254>
WebKit Commit Bot
Comment 5 2018-05-29 01:45:25 PDT
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 6 2018-05-29 01:46:36 PDT
<rdar://problem/40609053>
Thibault Saunier
Comment 7 2018-05-29 14:24:49 PDT
(In reply to Xabier Rodríguez Calvar from comment #2) > Comment on attachment 341459 [details] > [GStreamer] Update "qtdemux: Clarify field name about > stream-encryption-system" patch > > Lol, I check the "important" things like changelogs and such and I miss this > typo. At least I didn't miss it before pushing upstream :D
Note You need to log in before you can comment on or make changes to this bug.