Bug 1369950 - Remove aom_codec_stream_info_t sz field references. r=gerald draft
authorRalph Giles <giles@mozilla.com>
Tue, 09 May 2017 11:10:51 -0700
changeset 589256 e095d69017072a67fe9e02c582a4101d61c9045a
parent 589255 c64ee4786614fae2439c4ff030189a43e2d701dd
child 589257 386d20c25fb08c81d8e10116e18f1fff9a280d64
push id62303
push userbmo:giles@thaumas.net
push dateMon, 05 Jun 2017 23:33:20 +0000
reviewersgerald
bugs1369950
milestone55.0a1
Bug 1369950 - Remove aom_codec_stream_info_t sz field references. r=gerald Upstream has removed the requirement to set this when initializing the stream_info struct. MozReview-Commit-ID: 24OJ550Ral
dom/media/platforms/agnostic/AOMDecoder.cpp
--- a/dom/media/platforms/agnostic/AOMDecoder.cpp
+++ b/dom/media/platforms/agnostic/AOMDecoder.cpp
@@ -232,17 +232,16 @@ AOMDecoder::IsSupportedCodec(const nsASt
          aCodecType.Equals(version);
 }
 
 /* static */
 bool
 AOMDecoder::IsKeyframe(Span<const uint8_t> aBuffer) {
   aom_codec_stream_info_t info;
   PodZero(&info);
-  info.sz = sizeof(info);
 
   auto res = aom_codec_peek_stream_info(aom_codec_av1_dx(),
                                         aBuffer.Elements(),
                                         aBuffer.Length(),
                                         &info);
   if (res != AOM_CODEC_OK) {
     LOG_RESULT(res, "couldn't get keyframe flag with aom_codec_peek_stream_info");
     return false;
@@ -251,17 +250,16 @@ AOMDecoder::IsKeyframe(Span<const uint8_
   return bool(info.is_kf);
 }
 
 /* static */
 nsIntSize
 AOMDecoder::GetFrameSize(Span<const uint8_t> aBuffer) {
   aom_codec_stream_info_t info;
   PodZero(&info);
-  info.sz = sizeof(info);
 
   auto res = aom_codec_peek_stream_info(aom_codec_av1_dx(),
                                         aBuffer.Elements(),
                                         aBuffer.Length(),
                                         &info);
   if (res != AOM_CODEC_OK) {
     LOG_RESULT(res, "couldn't get frame size with aom_codec_peek_stream_info");
   }