Repaint video controls when buffering during pause.

webkit/glue/webmediaplayer_impl.cc

Index: webkit/glue/webmediaplayer_impl.cc
diff --git a/webkit/glue/webmediaplayer_impl.cc b/webkit/glue/webmediaplayer_impl.cc
index 01ddcc06c9191fef3966592df87892058792b880..e6b17813e2fa1481e11c01029e95d504179333ba 100644
--- a/webkit/glue/webmediaplayer_impl.cc
+++ b/webkit/glue/webmediaplayer_impl.cc
@@ -395,6 +395,11 @@ void WebMediaPlayerImpl::setVolume(float volume) {
 void WebMediaPlayerImpl::setVisible(bool visible) {
   DCHECK_EQ(main_loop_, MessageLoop::current());
 
+  // Update controls as data buffers when paused. HTMLMediaElement issues a
+  // RenderVideo->updateFromElement() which ends in a call to here.

[scherkus (not reviewing), 2011/11/16 02:09:25]

I think the part I'm more interested in is when/why does it issue this call?

[DaleCurtis, 2011/11/16 02:31:49]

Done.

+  if (visible && paused_)
+    Repaint();
+
   // TODO(hclam): add appropriate method call when pipeline has it implemented.

[scherkus (not reviewing), 2011/11/16 02:09:25]

do you know what this TODO might be referring to?

perhaps it's time to remove it!

[DaleCurtis, 2011/11/16 02:31:49]

I sent him an email. I'll report back.

On 2011/11/16 02:09:25, scherkus wrote:

   return;
 }