Remove branch in WebMediaPlayerImpl that is never reached

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 76a99093b2c8c518153216cc69a39ab826d2bde8..f97ba132afe18258ef1262898279492396d65118 100644
--- a/webkit/glue/webmediaplayer_impl.cc
+++ b/webkit/glue/webmediaplayer_impl.cc
@@ -958,20 +958,10 @@ void WebMediaPlayerImpl::OnPipelineError(PipelineStatus error) {
 void WebMediaPlayerImpl::OnNetworkEvent(PipelineStatus status) {

[scherkus (not reviewing), 2011/08/16 18:07:00]

hmm.. I wonder if this should trigger a repaint

could be too spammy though

we have a bug where the progress bar isn't re-painting even though it's updating
(you have to wiggle the mouse over it to get it to repaint)

   DCHECK(MessageLoop::current() == main_loop_);
   if (status == media::PIPELINE_OK) {
-    if (pipeline_->IsNetworkActive()) {
+    if (pipeline_->IsNetworkActive())
       SetNetworkState(WebKit::WebMediaPlayer::Loading);
-    } 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::Loaded);
-      }
-
+    else
       SetNetworkState(WebKit::WebMediaPlayer::Idle);
-    }
   }
 }