Add UMA for droppedframes count

media/base/pipeline.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "media/base/pipeline.h"
 
 #include <algorithm>
 
 #include "base/bind.h"
 #include "base/callback.h"
@@ skipping 508 matching lines @@
 
     return;
   }
 
   stop_cb_ = stop_cb;
 
   // We may already be stopping due to a runtime error.
   if (state_ == kStopping)
     return;
 
+  PipelineStatistics stats = GetStatistics();
+  if (renderer_ && renderer_->HasVideo() && stats.video_frames_decoded > 0) {

[DaleCurtis, 2014/08/28 18:25:24]

No {} required since it's a single line if.

[prabhur1, 2014/08/28 18:46:30]

Done.

+    UMA_HISTOGRAM_COUNTS("Media.DroppedFrameCount", stats.video_frames_dropped);
+  }
+
   SetState(kStopping);
   pending_callbacks_.reset();
   DoStop(base::Bind(&Pipeline::OnStopCompleted, weak_factory_.GetWeakPtr()));
 }
 
 void Pipeline::ErrorChangedTask(PipelineStatus error) {
   DCHECK(task_runner_->BelongsToCurrentThread());
   DCHECK_NE(PIPELINE_OK, error) << "PIPELINE_OK isn't an error!";
 
   media_log_->AddEvent(media_log_->CreatePipelineErrorEvent(error));
@@ skipping 130 matching lines @@
       base::Bind(&Pipeline::GetMediaDuration, base::Unretained(this)));
 }
 
 void Pipeline::BufferingStateChanged(BufferingState new_buffering_state) {
   DVLOG(1) << __FUNCTION__ << "(" << new_buffering_state << ") ";
   DCHECK(task_runner_->BelongsToCurrentThread());
   buffering_state_cb_.Run(new_buffering_state);
 }
 
 }  // namespace media