[Webkit-unassigned] [Bug 146282] MediaSession: propagate MediaSessionMetadata to WebPageProxy

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Jun 24 14:58:10 PDT 2015


Darin Adler <darin at apple.com> changed:

           What    |Removed                     |Added
 Attachment #255491|review?                     |review+
              Flags|                            |

--- Comment #6 from Darin Adler <darin at apple.com> ---
Comment on attachment 255491
  --> https://bugs.webkit.org/attachment.cgi?id=255491

View in context: https://bugs.webkit.org/attachment.cgi?id=255491&action=review

> Source/WebCore/Modules/mediasession/MediaSession.cpp:104
> +    m_document.page()->chrome().client().mediaSessionMetadataDidChange(m_metadata);

What guarantees that page() is not null here? We should check it for null unless there is an ironclad guarantee it can’t be.

> Source/WebCore/page/ChromeClient.h:50
> +#include "MediaSessionMetadata.h"
> +#endif

We should not include this header. We should be able to just forward-declare MediaSessionMetadata to compile this header.

You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20150624/4d394fe9/attachment-0001.html>

More information about the webkit-unassigned mailing list