Add UMA stats for audio/video containers.

media/base/bit_reader_unittest.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/base/bit_reader.h"
 
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace media {
 
@@ skipping 27 matching lines @@
   uint8 value8;
   uint8 buffer[] = {0x12};
   BitReader reader1(buffer, sizeof(buffer));
 
   EXPECT_TRUE(reader1.ReadBits(4, &value8));
   EXPECT_FALSE(reader1.ReadBits(5, &value8));
   EXPECT_FALSE(reader1.ReadBits(1, &value8));
   EXPECT_TRUE(reader1.ReadBits(0, &value8));
 }
 
+TEST(BitReaderTest, SkipBitsTest) {
+  uint8 value8;
+  uint8 buffer[] = {0x0a};
+  BitReader reader1(buffer, sizeof(buffer));
+
+  EXPECT_TRUE(reader1.SkipBits(2));
+  EXPECT_TRUE(reader1.ReadBits(3, &value8));
+  EXPECT_EQ(value8, 1);
+  EXPECT_FALSE(reader1.SkipBits(8));
+  EXPECT_TRUE(reader1.SkipBits(0));

[acolwell GONE FROM CHROMIUM, 2013/05/18 01:22:09]

Isn't this supposed to return false since the previous line reads past the end?

[jrummell, 2013/05/22 18:27:39]

The original comment for ReadBits states "When return false, the stream will
enter a state where further ReadBits/SkipBits operations will always return
false unless |num_bits| is 0." Since it already referred to SkipBits (which
didn't exist), I figured that was the desired behavior.

+}
+
 }  // namespace media