[GStreamer][EME] Fix issue with allowed systems extraction
Created attachment 302386 [details] Patch
Comment on attachment 302386 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=302386&action=review > Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamerBase.cpp:271 > unsigned i; > - for (i = 0; !streamEncryptionAllowedSystems[i]; ++i) > + for (i = 0; streamEncryptionAllowedSystems[i]; ++i) for (unsigned i = 0
Comment on attachment 302386 [details] Patch (In reply to comment #2) > Comment on attachment 302386 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=302386&action=review > > > Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamerBase.cpp:271 > > unsigned i; > > - for (i = 0; !streamEncryptionAllowedSystems[i]; ++i) > > + for (i = 0; streamEncryptionAllowedSystems[i]; ++i) > > for (unsigned i = 0 I can't, i is used later.
Comment on attachment 302386 [details] Patch Clearing flags on attachment: 302386 Committed r212830: <http://trac.webkit.org/changeset/212830>
All reviewed patches have been landed. Closing bug.
(In reply to comment #3) > Comment on attachment 302386 [details] > Patch > > (In reply to comment #2) > > Comment on attachment 302386 [details] > > Patch > > > > View in context: > > https://bugs.webkit.org/attachment.cgi?id=302386&action=review > > > > > Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamerBase.cpp:271 > > > unsigned i; > > > - for (i = 0; !streamEncryptionAllowedSystems[i]; ++i) > > > + for (i = 0; streamEncryptionAllowedSystems[i]; ++i) > > > > for (unsigned i = 0 > > I can't, i is used later. It's used later in a similar for loop that should do the same.
(In reply to comment #6) > It's used later in a similar for loop that should do the same. Roger for the next time.