MSE: Log a warning if muxed AV media segment has no A or has no V block

media/formats/mp4/mp4_stream_parser_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 <stddef.h>
 #include <stdint.h>
 
 #include <algorithm>
 #include <string>
 
@@ skipping 138 matching lines @@
     DVLOG(1) << "KeyNeededF: " << init_data.size();
     EXPECT_EQ(EmeInitDataType::CENC, type);
     EXPECT_FALSE(init_data.empty());
   }
 
   void NewSegmentF() {
     DVLOG(1) << "NewSegmentF";
     lower_bound_ = kNoDecodeTimestamp();
   }
 
-  void EndOfSegmentF() {
+  bool EndOfSegmentF() {
     DVLOG(1) << "EndOfSegmentF()";
     lower_bound_ =
         DecodeTimestamp::FromPresentationTime(base::TimeDelta::Max());
+    return true;  // BIG TODO test !

[chcunningham, 2016/01/07 22:23:57]

:)

[wolenetz, 2016/01/16 01:23:35]

No longer changed (PS5, at least).

   }
 
   void InitializeParserAndExpectLiveness(
       DemuxerStream::Liveness expected_liveness) {
     parser_->Init(
         base::Bind(&MP4StreamParserTest::InitF, base::Unretained(this),
                    expected_liveness),
         base::Bind(&MP4StreamParserTest::NewConfigF, base::Unretained(this)),
         base::Bind(&MP4StreamParserTest::NewBuffersF, base::Unretained(this)),
         true,
@@ skipping 163 matching lines @@
   scoped_refptr<DecoderBuffer> buffer =
       ReadTestDataFile("bear-640x360-non_square_pixel-with_pasp.mp4");
 
   EXPECT_MEDIA_LOG(VideoCodecLog("avc1.6401e"));
   EXPECT_TRUE(AppendDataInPieces(buffer->data(), buffer->data_size(), 512));
   EXPECT_EQ(gfx::Size(639, 360), video_decoder_config_.natural_size());
 }
 
 }  // namespace mp4
 }  // namespace media