Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(462)

Unified Diff: media/mp2t/ts_section_pat.cc

Issue 23566013: Mpeg2 TS stream parser for media source. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address comment about DCHECK vs RCHECK Created 7 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: media/mp2t/ts_section_pat.cc
diff --git a/media/mp2t/ts_section_pat.cc b/media/mp2t/ts_section_pat.cc
new file mode 100644
index 0000000000000000000000000000000000000000..64e02d8400abb01cd0d218d2e18586b49dc850e4
--- /dev/null
+++ b/media/mp2t/ts_section_pat.cc
@@ -0,0 +1,120 @@
+// Copyright (c) 2013 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/mp2t/ts_section_pat.h"
+
+#include <vector>
+
+#include "base/logging.h"
+#include "media/base/bit_reader.h"
+#include "media/mp2t/mp2t_common.h"
+
+namespace media {
+namespace mp2t {
+
+TsSectionPat::TsSectionPat(RegisterPmtCb register_pmt_cb)
+ : register_pmt_cb_(register_pmt_cb),
+ version_number_(-1) {
+}
+
+TsSectionPat::~TsSectionPat() {
+}
+
+bool TsSectionPat::ParsePsiSection(BitReader* bit_reader) {
+ // Read the fixed section length.
+ int table_id;
+ bool section_syntax_indicator;
+ bool dummy_zero;
+ int reserved;
+ int section_length;
+ int transport_stream_id;
+ int version_number;
+ bool current_next_indicator;
+ int section_number;
+ int last_section_number;
+ RCHECK(bit_reader->ReadBits(8, &table_id));
+ RCHECK(bit_reader->ReadBits(1, &section_syntax_indicator));
+ RCHECK(bit_reader->ReadBits(1, &dummy_zero));
+ RCHECK(bit_reader->ReadBits(2, &reserved));
+ RCHECK(bit_reader->ReadBits(12, &section_length));
+ RCHECK(bit_reader->ReadBits(16, &transport_stream_id));
+ RCHECK(bit_reader->ReadBits(2, &reserved));
+ RCHECK(bit_reader->ReadBits(5, &version_number));
+ RCHECK(bit_reader->ReadBits(1, &current_next_indicator));
+ RCHECK(bit_reader->ReadBits(8, &section_number));
+ RCHECK(bit_reader->ReadBits(8, &last_section_number));
+ section_length -= 5;
+
+ // Perform a few verifications:
+ // - Table ID should be 0 for a PAT.
+ // - section_syntax_indicator should be one.
+ // - section length should not exceed 1021
+ RCHECK(table_id == 0x0);
+ RCHECK(section_syntax_indicator);
+ RCHECK(!dummy_zero);
+ RCHECK(section_length <= 1021);
+
+ // Make sure there are enough bits for the variable section.
+ RCHECK(bit_reader->bits_available() >= 8 * section_length);
+
+ // Both the program table and the CRC have a size multiple of 4.
+ // Note for pmt_pid_count: minus 4 to account for the CRC.
+ RCHECK((section_length & 0x3) == 0);
+ int pmt_pid_count = (section_length - 4) / 4;
+
+ // Read the variable length section: program table & crc.
+ std::vector<int> program_number_array(pmt_pid_count);
+ std::vector<int> pmt_pid_array(pmt_pid_count);
+ for (int k = 0; k < pmt_pid_count; k++) {
+ int reserved;
+ RCHECK(bit_reader->ReadBits(16, &program_number_array[k]));
+ RCHECK(bit_reader->ReadBits(3, &reserved));
+ RCHECK(bit_reader->ReadBits(13, &pmt_pid_array[k]));
+ }
+ int crc32;
+ RCHECK(bit_reader->ReadBits(32, &crc32));
+
+ // Just ignore the PAT if not applicable yet.
+ if (!current_next_indicator) {
+ LOG(WARNING) << "Not supported: received a PAT not applicable yet";
damienv1 2013/09/13 01:23:01 DVLOG
damienv1 2013/09/13 16:07:09 Done.
+ return true;
+ }
+
+ // Ignore the program table if it hasn't changed.
+ if (version_number == version_number_)
+ return true;
+
+ // Both the MSE and the HLS spec specifies that TS streams should convey
+ // exactly one program.
+ if (pmt_pid_count > 1)
+ return false;
+
+ // Can now register the PMT.
+ int expected_version_number = version_number;
+ if (version_number_ >= 0)
+ expected_version_number = (version_number_ + 1) % 32;
+ LOG_IF(WARNING, version_number != expected_version_number)
+ << "Unexpected version number: "
+ << version_number << " vs " << version_number_;
+ for (int k = 0; k < pmt_pid_count; k++) {
+ if (program_number_array[k] != 0) {
+ // Program numbers different from 0 correspond to PMT.
+ register_pmt_cb_.Run(program_number_array[k], pmt_pid_array[k]);
+ // Even if there are multiple programs, only one can be supported now.
+ // HLS: "Transport Stream segments MUST contain a single MPEG-2 Program."
+ break;
+ }
+ }
+ version_number_ = version_number;
+
+ return true;
+}
+
+void TsSectionPat::ResetPsiSection() {
+ version_number_ = -1;
+}
+
+} // namespace mp2t
+} // namespace media
+

Powered by Google App Engine
This is Rietveld 408576698