Clean up histogram'd media enum max values.

media/base/pipeline_status.h

 // 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.
 
 #ifndef MEDIA_BASE_PIPELINE_STATUS_H_
 #define MEDIA_BASE_PIPELINE_STATUS_H_
 
 #include "base/callback.h"
 
 #include <string>
@@ skipping 14 matching lines @@
   PIPELINE_ERROR_COULD_NOT_RENDER = 8,
   PIPELINE_ERROR_READ = 9,
   PIPELINE_ERROR_OPERATION_PENDING = 10,
   PIPELINE_ERROR_INVALID_STATE = 11,
   // Demuxer related errors.
   DEMUXER_ERROR_COULD_NOT_OPEN = 12,
   DEMUXER_ERROR_COULD_NOT_PARSE = 13,
   DEMUXER_ERROR_NO_SUPPORTED_STREAMS = 14,
   // Decoder related errors.
   DECODER_ERROR_NOT_SUPPORTED = 15,
-  PIPELINE_STATUS_MAX,  // Must be greater than all other values logged.
+  PIPELINE_STATUS_MAX = DECODER_ERROR_NOT_SUPPORTED,  // Must be equal to

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

nit: instead of squeezing a comment here I'd stick it on the line before

(the rest of this enum has comments preceeding values, so consistency is nice)

+                                                      // largest value ever
+                                                      // logged.
 };
 
 typedef base::Callback<void(PipelineStatus)> PipelineStatusCB;
 
 // TODO(scherkus): this should be moved alongside host interface definitions.
 struct PipelineStatistics {
   PipelineStatistics()
       : audio_bytes_decoded(0),
         video_bytes_decoded(0),
         video_frames_decoded(0),
         video_frames_dropped(0) {
   }
 
   uint32 audio_bytes_decoded;  // Should be uint64?
   uint32 video_bytes_decoded;  // Should be uint64?
   uint32 video_frames_decoded;
   uint32 video_frames_dropped;
 };
 
 // Used for updating pipeline statistics.
 typedef base::Callback<void(const PipelineStatistics&)> StatisticsCB;
 
 }  // namespace media
 
 #endif  // MEDIA_BASE_PIPELINE_STATUS_H_