Implemented HEVC video demuxing and parsing

media/media.gyp

Index: media/media.gyp
diff --git a/media/media.gyp b/media/media.gyp
index 312f4267a1d6c76ae1cf79f9f001e2cbc200b7b9..bf5f80ccad2515fb885445f00e69f439cdaca970 100644
--- a/media/media.gyp
+++ b/media/media.gyp
@@ -644,6 +644,8 @@
             'filters/ffmpeg_glue.h',
             'filters/ffmpeg_h264_to_annex_b_bitstream_converter.cc',
             'filters/ffmpeg_h264_to_annex_b_bitstream_converter.h',
+            'filters/ffmpeg_h265_to_annex_b_bitstream_converter.cc',

[ddorwin, 2015/06/12 00:52:27]

Should we block these on a separate hevc variable that is only set when
'chromecast==1'?

[servolk, 2015/06/12 21:57:31]

We had a separate lengthy discussion with rsleevi@ and brettw@ about this.
Earlier patchsets of this CL used to add a new global gyp flag in
build/common.gypi for toggling HEVC support (e.g. see patchset #17). But
rsleevi@ and brettw@ we against introducing any new global gyp flags and
suggested that instead we should use the '#if defined(ENABLE_HEVC_DEMUXING)'
approach everywhere. So if you look, for example at
fmpeg_h265_to_annex_b_bitstream_converter.cc, it has #if at the very top of the
file, to ensure no HEVC code is included into any builds that didn't enable it
explicitly. I think that's the best we can do without global gyp flag.

[ddorwin, 2015/06/19 21:40:14]

They didn't want *global* GYP flags. What I'm proposing is a GYP flag local to
this GYP file, which I believe is entirely acceptable. It's essentially a way of
defining a local variable to avoid duplicating logic.

We could #ifdef every Android-specific file, but we don't - we use GYP flags.

If you want to be extra careful, you could #error if ENABLE_HEVC_DEMUXING is not
defined in those files.

Speaking of that discussion, were they okay with ENABLE_HEVC_DEMUXING in
common.gypi? That has a similar cost to the entire project. See my comments
there.

+            'filters/ffmpeg_h265_to_annex_b_bitstream_converter.h',
             'filters/ffmpeg_video_decoder.cc',
             'filters/ffmpeg_video_decoder.h',
             'filters/in_memory_url_protocol.cc',
@@ -985,8 +987,12 @@
             'filters/ffmpeg_aac_bitstream_converter.h',
             'filters/ffmpeg_h264_to_annex_b_bitstream_converter.cc',
             'filters/ffmpeg_h264_to_annex_b_bitstream_converter.h',
+            'filters/ffmpeg_h265_to_annex_b_bitstream_converter.cc',
+            'filters/ffmpeg_h265_to_annex_b_bitstream_converter.h',
             'filters/h264_to_annex_b_bitstream_converter.cc',
             'filters/h264_to_annex_b_bitstream_converter.h',
+            'filters/h265_parser.cc',
+            'filters/h265_parser.h',
             'formats/mp2t/es_adapter_video.cc',
             'formats/mp2t/es_adapter_video.h',
             'formats/mp2t/es_parser.cc',
@@ -1025,6 +1031,8 @@
             'formats/mp4/cenc.h',
             'formats/mp4/es_descriptor.cc',
             'formats/mp4/es_descriptor.h',
+            'formats/mp4/hevc.cc',
+            'formats/mp4/hevc.h',
             'formats/mp4/mp4_stream_parser.cc',
             'formats/mp4/mp4_stream_parser.h',
             'formats/mp4/sample_to_group_iterator.cc',
@@ -1232,6 +1240,11 @@
             'USE_NEON'
           ],
         }],
+        ['proprietary_codecs==1', {
+          'sources': [
+            'filters/h265_parser_unittest.cc',
+          ],
+        }],
         ['media_use_ffmpeg==1', {
           'dependencies': [
             '../third_party/ffmpeg/ffmpeg.gyp:ffmpeg',