Fixing bug where paused resource loading needs final repaint

Adds logic in WebMediaPlayerImpl to trigger one last repaint before
going idle when the media is fully loaded.

BUG=49567
TEST=media_unittests
TBR=hclam

Review URL: http://codereview.chromium.org/3033027

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53659 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 8 insertions, 2 deletions

diff --git a/webkit/glue/webmediaplayer_impl.cc b/webkit/glue/webmediaplayer_impl.cc
index 51d67ab..f5b87cf 100644
--- a/webkit/glue/webmediaplayer_impl.cc
+++ b/webkit/glue/webmediaplayer_impl.cc
@@ -657,10 +657,16 @@ void WebMediaPlayerImpl::OnPipelineError() {
 void WebMediaPlayerImpl::OnNetworkEvent() {
   DCHECK(MessageLoop::current() == main_loop_);
   if (pipeline_->GetError() == media::PIPELINE_OK) {
-    if (pipeline_->IsNetworkActive())
+    if (pipeline_->IsNetworkActive()) {
       SetNetworkState(WebKit::WebMediaPlayer::Loading);
-    else
+    } else {
+      // If we are inactive because we just finished receiving all the data,
+      // do one final repaint to show final progress.
+      if (bytesLoaded() == totalBytes() &&
+          network_state_ != WebKit::WebMediaPlayer::Idle)
+        Repaint();
       SetNetworkState(WebKit::WebMediaPlayer::Idle);
+    }
   }
 }