Return parsed sample frequency of ADTS header

media/formats/mp2t/es_parser_adts_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 <vector>
 
 #include "base/bind.h"
 #include "base/logging.h"
 #include "base/macros.h"
 #include "base/time/time.h"
 #include "media/base/stream_parser_buffer.h"
 #include "media/formats/mp2t/es_parser_adts.h"
 #include "media/formats/mp2t/es_parser_test_base.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace media {
 class AudioDecoderConfig;
 
 namespace mp2t {
+namespace {
+const char kAac44100PacketTimestamp[] = "(0) (23) (46) (69)";
+}
 
 class EsParserAdtsTest : public EsParserTestBase,
                          public testing::Test {
  public:
   EsParserAdtsTest();
 
  protected:
-  bool Process(const std::vector<Packet>& pes_packets, bool force_timing);
+  bool Process(const std::vector<Packet>& pes_packets, bool sbr_in_mimetype);
 
  private:
   DISALLOW_COPY_AND_ASSIGN(EsParserAdtsTest);
 };
 
 EsParserAdtsTest::EsParserAdtsTest() {
 }
 
-bool EsParserAdtsTest::Process(
-    const std::vector<Packet>& pes_packets,
-    bool force_timing) {
+bool EsParserAdtsTest::Process(const std::vector<Packet>& pes_packets,
+                               bool sbr_in_mimetype) {
   EsParserAdts es_parser(
       base::Bind(&EsParserAdtsTest::NewAudioConfig, base::Unretained(this)),
       base::Bind(&EsParserAdtsTest::EmitBuffer, base::Unretained(this)),
-      false);
-  return ProcessPesPackets(&es_parser, pes_packets, force_timing);
+      sbr_in_mimetype);
+  return ProcessPesPackets(&es_parser, pes_packets, false /* force_timing */);

[chcunningham, 2016/06/14 00:13:07]

Missed one ;)

[yucliu1, 2016/06/14 00:23:28]

Which one is missing?

[yucliu1, 2016/06/14 22:01:05]

Ping~ I'm not sure what I'm missing here.

[chcunningham, 2016/06/14 23:27:41]

sorry, I just mean that your comment says force_timing when it should be
sbr_in_mimetype

[yucliu1, 2016/06/14 23:42:25]

Ah, this one should be force_timing. It's used in es_parser_h264_unittests.

 }
 
 TEST_F(EsParserAdtsTest, NoInitialPts) {
   LoadStream("bear.adts");
   std::vector<Packet> pes_packets = GenerateFixedSizePesPacket(512);
   // Process should succeed even without timing info, we should just skip the
   // audio frames without timing info, but still should be able to parse and
   // play the stream after that.
-  EXPECT_TRUE(Process(pes_packets, false));
+  EXPECT_TRUE(Process(pes_packets, false /* sbr_in_mimetype */));
   EXPECT_EQ(1u, config_count_);
   EXPECT_EQ(0u, buffer_count_);
 }
 
 TEST_F(EsParserAdtsTest, SinglePts) {
   LoadStream("bear.adts");
 
   std::vector<Packet> pes_packets = GenerateFixedSizePesPacket(512);
   pes_packets.front().pts = base::TimeDelta::FromSeconds(10);
 
-  EXPECT_TRUE(Process(pes_packets, false));
+  EXPECT_TRUE(Process(pes_packets, false /* sbr_in_mimetype */));
   EXPECT_EQ(1u, config_count_);
   EXPECT_EQ(45u, buffer_count_);
 }
 
 TEST_F(EsParserAdtsTest, AacLcAdts) {
   LoadStream("sfx.adts");
   std::vector<Packet> pes_packets = GenerateFixedSizePesPacket(512);
   pes_packets.front().pts = base::TimeDelta::FromSeconds(1);
-  EXPECT_TRUE(Process(pes_packets, false));
+  EXPECT_TRUE(Process(pes_packets, false /* sbr_in_mimetype */));
   EXPECT_EQ(1u, config_count_);
   EXPECT_EQ(14u, buffer_count_);
 }
+
+TEST_F(EsParserAdtsTest, AacSampleRate) {
+  std::vector<Packet> pes_packets =
+      LoadPacketsFromFiles("aac-44100-packet-%d", 4);
+
+  pes_packets.front().pts = base::TimeDelta::FromSeconds(0);
+  EXPECT_TRUE(Process(pes_packets, true /* sbr_in_mimetype */));
+  EXPECT_EQ(4u, buffer_count_);
+  EXPECT_EQ(kAac44100PacketTimestamp, buffer_timestamps_);
+}
 }  // namespace mp2t
 }  // namespace media