Bug 1332585 - Check that encoder completed after EOS in existing test. r?jesup draft
authorAndreas Pehrson <pehrsons@gmail.com>
Fri, 20 Jan 2017 12:25:48 +0100
changeset 464236 3b4820bc1fc83dba9f621cb0ba0bafe09c2f303f
parent 464235 b4b284c0bce5f2eb6844eef78f1f7d445057b84b
child 464237 e369214fb173e8325c352075dd10466a9f90b03f
push id42309
push userbmo:pehrson@telenordigital.com
push dateFri, 20 Jan 2017 18:13:48 +0000
reviewersjesup
bugs1332585
milestone53.0a1
Bug 1332585 - Check that encoder completed after EOS in existing test. r?jesup MozReview-Commit-ID: FFLyGhggDyM
dom/media/gtest/TestVideoTrackEncoder.cpp
--- a/dom/media/gtest/TestVideoTrackEncoder.cpp
+++ b/dom/media/gtest/TestVideoTrackEncoder.cpp
@@ -347,9 +347,11 @@ TEST(VP8VideoTrackEncoder, EncodeComplet
   VideoSegment segment;
   encoder.NotifyQueuedTrackChanges(nullptr, 0, 0, TrackEventCommand::TRACK_EVENT_ENDED, segment);
 
   // Pull Encoded Data back from encoder. Since we have sent
   // EOS to encoder, encoder.GetEncodedTrack should return
   // NS_OK immidiately.
   EncodedFrameContainer container;
   EXPECT_TRUE(NS_SUCCEEDED(encoder.GetEncodedTrack(container)));
+
+  EXPECT_TRUE(encoder.IsEncodingComplete());
 }