diff options
author | miu <miu@chromium.org> | 2014-08-28 18:31:12 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2014-08-29 01:32:28 +0000 |
commit | 4b0ebd322e95bd691a510c19b16fb258be86ef51 (patch) | |
tree | e351171ab7077edd196d0f69e07761afc7de3442 /mojo/PRESUBMIT.py | |
parent | d4dd2df0e956f0ba64e988d48acc3d81c02aad35 (diff) | |
download | chromium_src-4b0ebd322e95bd691a510c19b16fb258be86ef51.zip chromium_src-4b0ebd322e95bd691a510c19b16fb258be86ef51.tar.gz chromium_src-4b0ebd322e95bd691a510c19b16fb258be86ef51.tar.bz2 |
Deflakify CastStreamingApiTest.EndToEnd, enabling it for Release builds.
An analysis was performed for sources of flakiness in this test. A
number of implementation problems have been resolved in recent changes,
with this change focusing on addressing problems with the test code
itself:
1. Do not require the color to match the tone at the same time. A/V
sync belongs in a perf test.
2. Do not require a color or tone to be held for a full half-second.
Instead, any momentary observation that matches expectations is
sufficient.
3. Only examine the content part of each video frame (i.e., exclude the
surrounding black borders caused by letterboxing).
4. Generate the source content using a canvas and
requestAnimationFrame(). The source content has a little "ball"
animating within it to ensure changing video frames are being sent
through end-to-end.
Also added AES keys to test end-to-end with encryption enabled.
This change re-enables the test for Release builds only. Once the test
has proven to be stable again, it will be re-enabled for Debug builds as
well. Baby steps...
BUG=396413
Review URL: https://codereview.chromium.org/515473004
Cr-Commit-Position: refs/heads/master@{#292539}
Diffstat (limited to 'mojo/PRESUBMIT.py')
0 files changed, 0 insertions, 0 deletions