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

Unified Diff: content/common/gpu/media/h264_parser.h

Issue 10710002: Add HE AAC support to ISO BMFF. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Remove unused entry from media.gyp Created 8 years, 5 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
« no previous file with comments | « content/common/gpu/media/h264_bit_reader_unittest.cc ('k') | content/common/gpu/media/h264_parser.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/common/gpu/media/h264_parser.h
diff --git a/content/common/gpu/media/h264_parser.h b/content/common/gpu/media/h264_parser.h
index ed76f164750db134ab5498d134a35512f3b725f7..e9aea189494c88b344833f32b9fac22062d60a21 100644
--- a/content/common/gpu/media/h264_parser.h
+++ b/content/common/gpu/media/h264_parser.h
@@ -13,6 +13,7 @@
#include "base/basictypes.h"
#include "content/common/content_export.h"
+#include "content/common/gpu/media/h264_bit_reader.h"
namespace content {
@@ -301,61 +302,6 @@ class CONTENT_EXPORT H264Parser {
Result ParseSEI(H264SEIMessage* sei_msg);
private:
- // A class to provide bit-granularity reading of H.264 streams.
- // This is not a generic bit reader class, as it takes into account
- // H.264 stream-specific constraints, such as skipping emulation-prevention
- // bytes and stop bits. See spec for more details.
- // TODO(posciak): need separate unittests for this class.
- class H264BitReader {
- public:
- H264BitReader();
- ~H264BitReader();
-
- // Initialize the reader to start reading at |data|, |size| being size
- // of |data| in bytes.
- // Return false on insufficient size of stream..
- // TODO(posciak,fischman): consider replacing Initialize() with
- // heap-allocating and creating bit readers on demand instead.
- bool Initialize(const uint8* data, off_t size);
-
- // Read |num_bits| next bits from stream and return in |*out|, first bit
- // from the stream starting at |num_bits| position in |*out|.
- // |num_bits| may be 1-32, inclusive.
- // Return false if the given number of bits cannot be read (not enough
- // bits in the stream), true otherwise.
- bool ReadBits(int num_bits, int *out);
-
- // Return the number of bits left in the stream.
- off_t NumBitsLeft();
-
- // See the definition of more_rbsp_data() in spec.
- bool HasMoreRBSPData();
-
- private:
- // Advance to the next byte, loading it into curr_byte_.
- // Return false on end of stream.
- bool UpdateCurrByte();
-
- // Pointer to the next unread (not in curr_byte_) byte in the stream.
- const uint8* data_;
-
- // Bytes left in the stream (without the curr_byte_).
- off_t bytes_left_;
-
- // Contents of the current byte; first unread bit starting at position
- // 8 - num_remaining_bits_in_curr_byte_ from MSB.
- int curr_byte_;
-
- // Number of bits remaining in curr_byte_
- int num_remaining_bits_in_curr_byte_;
-
- // Used in emulation prevention three byte detection (see spec).
- // Initially set to 0xffff to accept all initial two-byte sequences.
- int prev_two_bytes_;
-
- DISALLOW_COPY_AND_ASSIGN(H264BitReader);
- };
-
// Exp-Golomb code parsing as specified in chapter 9.1 of the spec.
// Read one unsigned exp-Golomb code from the stream and return in |*val|.
Result ReadUE(int* val);
« no previous file with comments | « content/common/gpu/media/h264_bit_reader_unittest.cc ('k') | content/common/gpu/media/h264_parser.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698