Remove MediaControls::showVolumeSlider()

Source/core/html/shadow/MediaControlElements.cpp

Index: Source/core/html/shadow/MediaControlElements.cpp
diff --git a/Source/core/html/shadow/MediaControlElements.cpp b/Source/core/html/shadow/MediaControlElements.cpp
index 34785fdb50b90432eceb1476881307f2d8beeb8b..8b25059d775acea88da41f97ba93d6e53a803fbd 100644
--- a/Source/core/html/shadow/MediaControlElements.cpp
+++ b/Source/core/html/shadow/MediaControlElements.cpp
@@ -286,7 +286,6 @@ const AtomicString& MediaControlOverlayEnclosureElement::shadowPseudoId() const
 
 MediaControlPanelMuteButtonElement::MediaControlPanelMuteButtonElement(Document& document, MediaControls* controls)

[acolwell GONE FROM CHROMIUM, 2014/03/04 02:06:58]

Do we even need this class anymore? It appears that this is the only subclass of
MediaControlMuteButtonElement. Any reason we shouldn't just move the remaining
code into that class?

[philipj_slow, 2014/03/04 08:10:16]

Sorry, I noticed the same thing and should have pointed it out in the commit
message. MediaControlVolumeSliderElement can also be merged, so I've merged both
as a separate CL that depends on this and the other recent controls changes:
https://codereview.chromium.org/185363011/

     : MediaControlMuteButtonElement(document, MediaMuteButton)
-    , m_controls(controls)
 {
 }
 
@@ -300,14 +299,6 @@ PassRefPtr<MediaControlPanelMuteButtonElement> MediaControlPanelMuteButtonElemen
     return button.release();
 }
 
-void MediaControlPanelMuteButtonElement::defaultEventHandler(Event* event)
-{
-    if (event->type() == EventTypeNames::mouseover)

[acolwell GONE FROM CHROMIUM, 2014/03/04 23:23:17]

We don't need this now because the slider is always already visible when this
button is visible?

[philipj_slow, 2014/03/05 07:22:16]

showVolumeSlider() simply shows the slider if (m_mediaController->hasAudio()),
but MediaControls::reset() already does exactly that. I haven't dug deep in the
WebKit history, but this looks like it would show a vertical slider which would
then be hidden on some mouseout event, but that code doesn't exist anymore, the
slider is part of the main panel...

Note that because of MediaController this wasn't 100% dead code, you could have
a case where there's initially no audio but adding another media element to the
controller causes there to be. In that case hovering the mute button ought to
cause the volume slider to become visible. However, I doubt anyone had thought
of that, because the sane thing to do would be do call reset() when the
MediaController changes in some way. (I'm still trying to figure out everything
that is broken about MediaController in combination with MediaControls, this is
far from the only problem.)

-        m_controls->showVolumeSlider();
-
-    MediaControlMuteButtonElement::defaultEventHandler(event);
-}
-
 const AtomicString& MediaControlPanelMuteButtonElement::shadowPseudoId() const
 {
     DEFINE_STATIC_LOCAL(AtomicString, id, ("-webkit-media-controls-mute-button", AtomicString::ConstructFromLiteral));