Add VideoRotation UMA Logging

media/filters/ffmpeg_demuxer.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/filters/ffmpeg_demuxer.h"
 
 #include <algorithm>
 #include <string>
 
 #include "base/base64.h"
 #include "base/bind.h"
 #include "base/callback.h"
 #include "base/callback_helpers.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/message_loop/message_loop_proxy.h"
+#include "base/metrics/histogram.h"
 #include "base/metrics/sparse_histogram.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/string_util.h"
 #include "base/strings/stringprintf.h"
 #include "base/sys_byteorder.h"
 #include "base/task_runner_util.h"
 #include "base/time/time.h"
 #include "media/base/audio_decoder_config.h"
 #include "media/base/bind_to_current_loop.h"
 #include "media/base/decoder_buffer.h"
@@ skipping 101 matching lines @@
         case 180:
           video_rotation_ = VIDEO_ROTATION_180;
           break;
         case 270:
           video_rotation_ = VIDEO_ROTATION_270;
           break;
         default:
           LOG(ERROR) << "Unsupported video rotation metadata: " << rotation;
           break;
       }
+      UMA_HISTOGRAM_ENUMERATION(
+          "Media.VideoRotation", video_rotation_, VIDEO_ROTATION_MAX + 1);

[scherkus (not reviewing), 2014/07/17 01:26:18]

I believe we'll want this logged inside of webmediaplayer_impl.cc inside of the
metadata callback

otherwise this path may end up logging rotation values for videos we don't end
up playing

 
       break;
     case AVMEDIA_TYPE_SUBTITLE:
       type_ = TEXT;
       break;
     default:
       NOTREACHED();
       break;
   }
 
@@ skipping 1050 matching lines @@
   }
   for (size_t i = 0; i < buffered.size(); ++i)
     host_->AddBufferedTimeRange(buffered.start(i), buffered.end(i));
 }
 
 void FFmpegDemuxer::OnDataSourceError() {
   host_->OnDemuxerError(PIPELINE_ERROR_READ);
 }
 
 }  // namespace media