Descriptionveatest: fix keyframe verification test.
The test was not resilient against arbitrary pipeline depths and encode
delays. It was possible for the encode request to keep overwriting
keframe_requested_at_ faster than the deadline check by which we would've
failed if no keyframe was provided. Thus it was possible for the test to
always pass, even without getting any keyframes.
Simplify this by just checking that each keyframe arrives before
every keyframe_period_ + kMaxKeyframeDelay.
Also, some cleanup for fake encoder validator creation.
BUG=None
TEST=veatest --gtest_filter=ForceKeyframes*
Committed: https://crrev.com/4fc28962e2da8b3f278b952c55fefe10487db952
Cr-Commit-Position: refs/heads/master@{#317367}
Patch Set 1 #
Messages
Total messages: 7 (2 generated)
|