Add UMA stats for audio/video containers.

media/base/bit_reader_unittest.cc

Index: media/base/bit_reader_unittest.cc
diff --git a/media/base/bit_reader_unittest.cc b/media/base/bit_reader_unittest.cc
index 48e8c5eac11fce366c5a2325780688f0c880aaff..0954d8e3454d9edf9b379ade136e08f4556f77b5 100644
--- a/media/base/bit_reader_unittest.cc
+++ b/media/base/bit_reader_unittest.cc
@@ -45,4 +45,16 @@ TEST(BitReaderTest, ReadBeyondEndTest) {
   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