Added UMA metrics for MediaSourcePlayer and MediaCodecPlayer

media/base/android/media_decoder_job.cc

 // Copyright 2013 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/android/media_decoder_job.h"
 
 #include "base/bind.h"
 #include "base/callback_helpers.h"
 #include "base/single_thread_task_runner.h"
 #include "base/thread_task_runner_handle.h"
 #include "base/trace_event/trace_event.h"
 #include "media/base/android/media_drm_bridge.h"
+#include "media/base/android/media_statistics.h"
 #include "media/base/bind_to_current_loop.h"
 #include "media/base/timestamp_constants.h"
 
 namespace media {
 
 // Timeout value for media codec operations. Because the first
 // DequeInputBuffer() can take about 150 milliseconds, use 250 milliseconds
 // here. See http://b/9357571.
 static const int kMediaCodecTimeoutInMilliseconds = 250;
 
 MediaDecoderJob::MediaDecoderJob(
     const scoped_refptr<base::SingleThreadTaskRunner>& decoder_task_runner,
     const base::Closure& request_data_cb,
-    const base::Closure& config_changed_cb)
+    const base::Closure& config_changed_cb,
+    FrameStatistics* frame_statistics)
     : need_to_reconfig_decoder_job_(false),
+      frame_statistics_(frame_statistics),
       ui_task_runner_(base::ThreadTaskRunnerHandle::Get()),
       decoder_task_runner_(decoder_task_runner),
       needs_flush_(false),
       input_eos_encountered_(false),
       output_eos_encountered_(false),
       skip_eos_enqueue_(true),
       prerolling_(true),
       request_data_cb_(request_data_cb),
       config_changed_cb_(config_changed_cb),
       current_demuxer_data_index_(0),
@@ skipping 440 matching lines @@
   // TODO(xhwang/qinmin): This logic is correct but strange. Clean it up.
   if (output_eos_encountered_)
     status = MEDIA_CODEC_OUTPUT_END_OF_STREAM;
   else if (has_format_change)
     status = MEDIA_CODEC_OUTPUT_FORMAT_CHANGED;
 
   bool render_output  = presentation_timestamp >= preroll_timestamp_ &&
       (status != MEDIA_CODEC_OUTPUT_END_OF_STREAM || size != 0u);
   base::TimeDelta time_to_render;
   DCHECK(!start_time_ticks.is_null());
-  if (render_output && ComputeTimeToRender()) {
-    time_to_render = presentation_timestamp - (base::TimeTicks::Now() -
-        start_time_ticks + start_presentation_timestamp);
+  if (render_output) {
+    DCHECK(frame_statistics_);

[xhwang, 2015/10/01 21:58:57]

ditto

[Tima Vaisburd, 2015/10/02 00:44:32]

Done.

+    frame_statistics_->AddFrame();
+
+    if (ComputeTimeToRender()) {
+      time_to_render =
+          presentation_timestamp - (base::TimeTicks::Now() - start_time_ticks +
+                                    start_presentation_timestamp);
+
+      // ComputeTimeToRender() returns true for video streams only, this is a
+      // video stream.
+      if (time_to_render < base::TimeDelta())
+        frame_statistics_->AddLateFrame((-1) * time_to_render);
+    }
   }
 
   if (time_to_render > base::TimeDelta()) {
     decoder_task_runner_->PostDelayedTask(
         FROM_HERE,
         base::Bind(&MediaDecoderJob::ReleaseOutputBuffer,
                    base::Unretained(this),
                    buffer_index,
                    offset,
                    size,
@@ skipping 187 matching lines @@
 
 void MediaDecoderJob::ReleaseMediaCodecBridge() {
   if (!media_codec_bridge_)
     return;
 
   media_codec_bridge_.reset();
   input_buf_index_ = -1;
 }
 
 }  // namespace media