Summary: | [EME][GStreamer] Fix logging in utilities | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Xabier Rodríguez Calvar <calvaris> | ||||
Component: | New Bugs | Assignee: | Xabier Rodríguez Calvar <calvaris> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | cgarcia, ews-watchlist, gustavo, menard, pnormand, vjaquez, webkit-bug-importer | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | WebKit Nightly Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Bug Depends on: | |||||||
Bug Blocks: | 218175 | ||||||
Attachments: |
|
Description
Xabier Rodríguez Calvar
2020-10-26 04:09:01 PDT
Created attachment 412303 [details]
Patch
Comment on attachment 412303 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=412303&action=review > Source/WebCore/platform/graphics/gstreamer/eme/GStreamerEMEUtilities.h:36 > +GST_DEBUG_CATEGORY_EXTERN(webkit_media_common_encryption_decrypt_debug_category); OK I am lost in all the bugs you opened :) Why not handle everything in a single patch? (In reply to Philippe Normand from comment #2) > Comment on attachment 412303 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=412303&action=review > > > Source/WebCore/platform/graphics/gstreamer/eme/GStreamerEMEUtilities.h:36 > > +GST_DEBUG_CATEGORY_EXTERN(webkit_media_common_encryption_decrypt_debug_category); > > OK I am lost in all the bugs you opened :) Why not handle everything in a > single patch? Don't get lost, they just have consecutive numbers almost all of them! Now speaking more serious, they tackled different things, at least most of them Committed r269031: <https://trac.webkit.org/changeset/269031> All reviewed patches have been landed. Closing bug and clearing flags on attachment 412303 [details]. |