Commit b242799c authored by liberato@chromium.org's avatar liberato@chromium.org Committed by Commit Bot

Allow any number of natual size changes in media pipeline tests.

Since https://chromium-review.googlesource.com/1026992 , the
pipeline now generates natural size changes correctly when the
encoded size changes.  Unfortunately, the pipeline integration tests
allowed at most one change.

This CL allows any number of size changes.

Change-Id: Id9fac03851623e45cc2cd62d2dc6e870ddc138d5
Reviewed-on: https://chromium-review.googlesource.com/1068021Reviewed-by: default avatarDan Sanders <sandersd@chromium.org>
Commit-Queue: Frank Liberato <liberato@chromium.org>
Cr-Commit-Position: refs/heads/master@{#560381}
parent c83af60d
......@@ -259,7 +259,7 @@ PipelineStatus PipelineIntegrationTestBase::StartInternal(
.WillRepeatedly(
Invoke(this, &PipelineIntegrationTestBase::CheckDuration));
}
EXPECT_CALL(*this, OnVideoNaturalSizeChange(_)).Times(AtMost(1));
EXPECT_CALL(*this, OnVideoNaturalSizeChange(_)).Times(AnyNumber());
EXPECT_CALL(*this, OnVideoOpacityChange(_)).WillRepeatedly(Return());
EXPECT_CALL(*this, OnAudioDecoderChange(_)).Times(AnyNumber());
EXPECT_CALL(*this, OnVideoDecoderChange(_)).Times(AnyNumber());
......@@ -599,7 +599,7 @@ PipelineStatus PipelineIntegrationTestBase::StartPipelineWithMediaSource(
EXPECT_CALL(*this, OnBufferingStateChange(BUFFERING_HAVE_NOTHING))
.Times(AnyNumber());
EXPECT_CALL(*this, OnDurationChange()).Times(AnyNumber());
EXPECT_CALL(*this, OnVideoNaturalSizeChange(_)).Times(AtMost(1));
EXPECT_CALL(*this, OnVideoNaturalSizeChange(_)).Times(AnyNumber());
EXPECT_CALL(*this, OnVideoOpacityChange(_)).Times(AtMost(1));
EXPECT_CALL(*this, OnAudioDecoderChange(_)).Times(AnyNumber());
EXPECT_CALL(*this, OnVideoDecoderChange(_)).Times(AnyNumber());
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment