Clean up histogram'd media enum max values.

media/ffmpeg/ffmpeg_common.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/ffmpeg/ffmpeg_common.h"
 
 #include "base/basictypes.h"
 #include "base/logging.h"
 #include "base/metrics/histogram.h"
 #include "base/strings/string_number_conversions.h"
@@ skipping 371 matching lines @@
     profile = VP8PROFILE_MAIN;
   else if (codec == kCodecVP9)
     profile = VP9PROFILE_MAIN;
   else
     profile = ProfileIDToVideoCodecProfile(stream->codec->profile);
 
   gfx::Size natural_size = GetNaturalSize(
       visible_rect.size(), aspect_ratio.num, aspect_ratio.den);
 
   if (record_stats) {
+    // Note the 'PRESUBMIT_IGNORE_UMA_MAX below, this silences the PRESUBMIT.py

[scherkus (not reviewing), 2014/02/25 03:44:49]

nit: remove leading ' ... or did you mean to terminate the quote?

if the former, then you should fix up the other comments in
media/audio/audio_output_resampler.cc

+    // check for uma enum max usage, since we're abusing
+    // UMA_HISTOGRAM_ENUMERATION to report a discrete value.
     UMA_HISTOGRAM_ENUMERATION("Media.VideoColorRange",
                               stream->codec->color_range,
-                              AVCOL_RANGE_NB);
+                              AVCOL_RANGE_NB);  // PRESUBMIT_IGNORE_UMA_MAX
   }
 
   VideoFrame::Format format = PixelFormatToVideoFormat(stream->codec->pix_fmt);
   if (codec == kCodecVP9) {
     // TODO(tomfinegan): libavcodec doesn't know about VP9.
     format = VideoFrame::YV12;
     coded_size = natural_size;
   }
 
   bool is_encrypted = false;
@@ skipping 131 matching lines @@
       return PIX_FMT_YUVJ420P;
     case VideoFrame::YV12A:
       return PIX_FMT_YUVA420P;
     default:
       DVLOG(1) << "Unsupported VideoFrame::Format: " << video_format;
   }
   return PIX_FMT_NONE;
 }
 
 }  // namespace media