Updated AAC:Parse() to emit better error logs.

media/formats/mp4/aac_unittest.cc

 // Copyright 2014 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 <string>
 
 #include "media/base/mock_media_log.h"
 #include "media/formats/mp4/aac.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
@@ skipping 112 matching lines @@
   data.push_back(0x12);
   EXPECT_FALSE(Parse(data));
 }
 
 TEST_F(AACTest, IncorrectProfileTest) {
   InSequence s;
   uint8 buffer[] = {0x0, 0x08};
   std::vector<uint8> data;
 
   data.assign(buffer, buffer + sizeof(buffer));
-
-  EXPECT_MEDIA_LOG(AudioCodecLog("mp4a.40.0"));
   EXPECT_FALSE(Parse(data));
 
   data[0] = 0x08;
   EXPECT_MEDIA_LOG(AudioCodecLog("mp4a.40.1"));
   EXPECT_TRUE(Parse(data));
 
   data[0] = 0x28;
   // No media log for this profile 5, since not enough bits are in |data| to
   // first parse profile 5's extension frequency index.
   EXPECT_FALSE(Parse(data));
@@ skipping 12 matching lines @@
   EXPECT_MEDIA_LOG(AudioCodecLog("mp4a.40.1"));
   EXPECT_TRUE(Parse(data));
 }
 
 TEST_F(AACTest, IncorrectChannelTest) {
   uint8 buffer[] = {0x0e, 0x00};
   std::vector<uint8> data;
 
   data.assign(buffer, buffer + sizeof(buffer));
 
-  EXPECT_MEDIA_LOG(AudioCodecLog("mp4a.40.1")).Times(2);
+  EXPECT_MEDIA_LOG(AudioCodecLog("mp4a.40.1")).Times(1);

[wolenetz, 2015/08/28 23:29:35]

nit: .Times(1) is redundant. Drop this part of this line please.

[msu.koo, 2015/08/29 09:36:52]

Done.

 
   EXPECT_FALSE(Parse(data));
 
   data[1] = 0x08;
   EXPECT_TRUE(Parse(data));
 }
 
 }  // namespace mp4
 
 }  // namespace media