Modify names of PipelineStatus histograms

content/browser/media/media_internals.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 "content/browser/media/media_internals.h"
 
 #include "base/metrics/histogram.h"
 #include "base/strings/string16.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/stringprintf.h"
@@ skipping 280 matching lines @@
       break;
   }
   return;
 }
 
 void MediaInternals::MediaInternalsUMAHandler::ReportUMAForPipelineStatus(
     const PipelineInfo& player_info) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
   if (player_info.has_video && player_info.has_audio) {
     if (player_info.video_codec_name == "vp8") {
-      UMA_HISTOGRAM_ENUMERATION("Media.PipelineStatus.AudioVideo.VP8",
+      UMA_HISTOGRAM_ENUMERATION("Media.PipelineStatus.AudioVideo.VP8.SW",
                                 player_info.last_pipeline_status,
                                 media::PIPELINE_STATUS_MAX + 1);
     } else if (player_info.video_codec_name == "vp9") {
-      UMA_HISTOGRAM_ENUMERATION("Media.PipelineStatus.AudioVideo.VP9",
+      UMA_HISTOGRAM_ENUMERATION("Media.PipelineStatus.AudioVideo.VP9.SW",
                                 player_info.last_pipeline_status,
                                 media::PIPELINE_STATUS_MAX + 1);
     } else if (player_info.video_codec_name == "h264") {
       if (player_info.video_decoder == media::GpuVideoDecoder::kDecoderName) {
-        UMA_HISTOGRAM_ENUMERATION("Media.PipelineStatus.AudioVideo.HW.H264",
+        UMA_HISTOGRAM_ENUMERATION("Media.PipelineStatus.AudioVideo.H264.HW",
                                   player_info.last_pipeline_status,
                                   media::PIPELINE_STATUS_MAX + 1);
       } else {
-        UMA_HISTOGRAM_ENUMERATION("Media.PipelineStatus.AudioVideo.SW.H264",
+        UMA_HISTOGRAM_ENUMERATION("Media.PipelineStatus.AudioVideo.H264.SW",
                                   player_info.last_pipeline_status,
                                   media::PIPELINE_STATUS_MAX + 1);
       }
     } else {
-      UMA_HISTOGRAM_ENUMERATION("Media.PipelineStatus.AudioVideo",
+      UMA_HISTOGRAM_ENUMERATION("Media.PipelineStatus.AudioVideo.Others",

[DaleCurtis, 2014/11/21 20:43:09]

Other?

                                 player_info.last_pipeline_status,
                                 media::PIPELINE_STATUS_MAX + 1);
     }
   } else if (player_info.has_audio) {
-    UMA_HISTOGRAM_ENUMERATION("Media.PipelineStatus.Audio",
+    UMA_HISTOGRAM_ENUMERATION("Media.PipelineStatus.AudioOnly",
                               player_info.last_pipeline_status,
                               media::PIPELINE_STATUS_MAX + 1);
   } else if (player_info.has_video) {
-    UMA_HISTOGRAM_ENUMERATION("Media.PipelineStatus.Video",
+    UMA_HISTOGRAM_ENUMERATION("Media.PipelineStatus.VideoOnly",
                               player_info.last_pipeline_status,
                               media::PIPELINE_STATUS_MAX + 1);
   } else {
     UMA_HISTOGRAM_ENUMERATION("Media.PipelineStatus.Unsupported",
                               player_info.last_pipeline_status,
                               media::PIPELINE_STATUS_MAX + 1);
   }
 }
 
 void MediaInternals::MediaInternalsUMAHandler::LogAndClearPlayersInRenderer(
@@ skipping 153 matching lines @@
     const std::string& function,
     const base::DictionaryValue* value) {
   SendUpdate(SerializeUpdate(function, value));
 
   base::AutoLock auto_lock(lock_);
   scoped_ptr<base::Value> out_value;
   CHECK(audio_streams_cached_data_.Remove(cache_key, &out_value));
 }
 
 }  // namespace content