Summary: | MediaSession artwork URL is showing in logs. | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Jean-Yves Avenard [:jya] <jean-yves.avenard> | ||||||
Component: | Media | Assignee: | Jean-Yves Avenard [:jya] <jean-yves.avenard> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | eric.carlson, ews-watchlist, glenn, jer.noble, philipj, sergio, webkit-bug-importer, youennf | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | Other | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Jean-Yves Avenard [:jya]
2021-08-19 18:45:49 PDT
Created attachment 435939 [details]
Patch
Comment on attachment 435939 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=435939&action=review > Source/WebCore/platform/audio/cocoa/MediaSessionManagerCocoa.mm:-393 > - I would tend to keep this logging but remove some information like title and src then. Comment on attachment 435939 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=435939&action=review > Source/WebCore/platform/audio/cocoa/MediaSessionManagerCocoa.mm:392 > +#ifdef DEBUG `#ifndef LOG_DISABLED` would be better I think. >> Source/WebCore/platform/audio/cocoa/MediaSessionManagerCocoa.mm:-393 >> - > > I would tend to keep this logging but remove some information like title and src then. I agree, just drop the url when LOG_DISABLED Created attachment 436264 [details]
Patch
apply comment
Committed r281533 (240904@main): <https://commits.webkit.org/240904@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 436264 [details]. |