Remove MediaControls::bufferingProgressed()

Source/core/html/shadow/MediaControls.cpp

Index: Source/core/html/shadow/MediaControls.cpp
diff --git a/Source/core/html/shadow/MediaControls.cpp b/Source/core/html/shadow/MediaControls.cpp
index dcac50573138b68da29e4ff68da25d4eee997cb2..861ce74545745147c0ea4ed731a45c2fd37187b9 100644
--- a/Source/core/html/shadow/MediaControls.cpp
+++ b/Source/core/html/shadow/MediaControls.cpp
@@ -227,14 +227,6 @@ bool MediaControls::shouldHideControls()
     return !m_panel->hovered();
 }
 
-void MediaControls::bufferingProgressed()
-{
-    // We only need to update buffering progress when paused, during normal
-    // playback playbackProgressed() will take care of it.
-    if (m_mediaController->paused())
-        m_timeline->setPosition(m_mediaController->currentTime());

[acolwell GONE FROM CHROMIUM, 2014/03/06 21:33:51]

So we are relying on the playbackStopped() & playbackStarted() calls in
updatePlayState() to update this. It looks like all 'm_paused =' sites except
the one in prepareForLoad() end up calling updatePlayState(). 

ISTM that configureMediaControls() may need a mediaControls()->reset() call in
it so that the control state will get properly reset on prepareForLoad().

[philipj_slow, 2014/03/07 05:44:09]

Good catch! I was able to trigger this bug even without this patch by simply
removing src/source and calling load() after playback has begun. Sometimes, the
time display is not updated. Since this is a pre-existing bug, I'll fix it in a
separate CL.

-}
-
 void MediaControls::playbackStarted()
 {
     m_currentTimeDisplay->show();