Bug 137220 – Add a ChromeClient API that gets called whenever the Page's audio playing state has changed

• Component: Media
• Status: RESOLVED FIXED

[Ada Chan]

It'll most likely be:

virtual void isPlayingAudioDidChange(bool)

to match the

bool isPlayingAudio() const;

we are adding to WebCore::Page.  See https://bugs.webkit.org/show_bug.cgi?id=137218.

[Ada Chan]

Created attachment 239344 [details]
Patch

[Jer Noble]

Comment on attachment 239344 [details]
Patch

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

r=me, with nit.

> Source/WebCore/page/ChromeClient.h:434
> +    virtual void isPlayingAudioDidChange(bool /*newIsPlayingAudio*/) { }

Nit: You don't need to include the (commented out) parameter name here. It's obvious from the function name.

[Ada Chan]

(In reply to comment #2)
> (From update of attachment 239344 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=239344&action=review
> 
> r=me, with nit.
> 
> > Source/WebCore/page/ChromeClient.h:434
> > +    virtual void isPlayingAudioDidChange(bool /*newIsPlayingAudio*/) { }
> 
> Nit: You don't need to include the (commented out) parameter name here. It's obvious from the function name.

I'll remove it.  Thanks for the review!

[Ada Chan]

Committed:
https://trac.webkit.org/changeset/174365