Let the effective media volume be 0 if the media element is muted

Source/web/WebMediaPlayerClientImpl.cpp

Index: Source/web/WebMediaPlayerClientImpl.cpp
diff --git a/Source/web/WebMediaPlayerClientImpl.cpp b/Source/web/WebMediaPlayerClientImpl.cpp
index e1e3724826298eedfdee8ea34cf7442f44aafbe1..835aa75f2d8a2f26587c4a5a62a6028ed5944cac 100644
--- a/Source/web/WebMediaPlayerClientImpl.cpp
+++ b/Source/web/WebMediaPlayerClientImpl.cpp
@@ -108,7 +108,7 @@ void WebMediaPlayerClientImpl::sizeChanged()
 
 double WebMediaPlayerClientImpl::volume() const

[acolwell GONE FROM CHROMIUM, 2014/08/07 15:52:59]

I'm assuming a followup CL to remove this function is coming after
https://codereview.chromium.org/443393002/ lands.

[philipj_slow, 2014/08/07 15:59:28]

Exactly, this CL is how I noticed this oddity. It would be nice to get rid of
the setVolume in load() as well.

 {
-    return mediaElement().playerVolume();
+    return mediaElement().effectiveMediaVolume();
 }
 
 void WebMediaPlayerClientImpl::playbackStateChanged()
@@ -212,7 +212,7 @@ void WebMediaPlayerClientImpl::load(WebMediaPlayer::LoadType loadType, const WTF
     m_audioSourceProvider.wrap(m_webMediaPlayer->audioSourceProvider());
 #endif
 
-    m_webMediaPlayer->setVolume(mediaElement().playerVolume());
+    m_webMediaPlayer->setVolume(mediaElement().effectiveMediaVolume());
 
     m_webMediaPlayer->setPoster(poster);