[Webkit-unassigned] [Bug 28310] Chromium: Show a "Mute Disabled" button on audio error.

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=28310


Andrew Scherkus <scherkus at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |scherkus at chromium.org




--- Comment #4 from Andrew Scherkus <scherkus at chromium.org>  2009-08-17 16:24:52 PDT ---
Kyle was my intern for the summer, I'll be taking over this patch.

To help explain the out-of-placeness, HTMLMediaElement is a base class to
HTMLVideoElement and HTMLAudioElement.  MediaPlayerPrivateInterface is a base
class to the different platform ports.

For HTMLMediaElement, we can probably implement the same method in both
HTMLAudioElement and HTMLVideoElement, since and HTMLMediaElement itself cannot
really exist and would never have a player associated with it.

For MediaPlayerPrivateInterface, Kyle chose to keep the existing behaviour
(audio is assumed to always be present) without being forced to implement the
actual behaviour for every port.  If it is preferred, we can have hasAudio() be
pure virtual and implement it as "notImplemented(); return true;" for the ports
we are unfamiliar with.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.