Commit b1ed7a7d authored by Ted Choc's avatar Ted Choc Committed by Commit Bot

Update EncryptedMediaTest.FrameSizeChangeVideo to be disabled for LMR1 builds.

The bots have been updated to 5.1.1, which is LOLLIPOP_MR1.  This just
disables the test for anything pre-marshmallow.

Example failure:
https://ci.chromium.org/p/chromium/builders/ci/Lollipop%20Phone%20Tester/21211

../../content/browser/media/media_browsertest.cc:55: Failure
Expected equality of these values:
  expected_title
    Which is: "ENDED"
  final_title
    Which is: "error"
Stack trace:

[WARNING:discardable_shared_memory_manager.cc(410)] Some MojoDiscardableSharedMemoryManagerImpls are still alive. They will be leaked.
[  FAILED  ] MSE_ExternalClearKey/EncryptedMediaTest.FrameSizeChangeVideo/0, where GetParam() = (0x9d970a3b pointing to "org.chromium.externalclearkey", 4-byte object <01-00 00-00>) (5931 ms)
[----------] 1 test from MSE_ExternalClearKey/EncryptedMediaTest (5937 ms total)

TBR=liberato@chromium.org
BUG=778245

Change-Id: I6816cbabedd6c6a60b139e494b9dae35164f07b7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1540304Reviewed-by: default avatarTed Choc <tedchoc@chromium.org>
Reviewed-by: default avatarFrank Liberato <liberato@chromium.org>
Commit-Queue: Ted Choc <tedchoc@chromium.org>
Cr-Commit-Position: refs/heads/master@{#644529}
parent c2c9b435
......@@ -330,8 +330,8 @@ IN_PROC_BROWSER_TEST_P(EncryptedMediaTest,
IN_PROC_BROWSER_TEST_P(EncryptedMediaTest, FrameSizeChangeVideo) {
#if defined(OS_ANDROID)
// https://crbug.com/778245
if (base::android::BuildInfo::GetInstance()->sdk_int() <=
base::android::SDK_VERSION_LOLLIPOP) {
if (base::android::BuildInfo::GetInstance()->sdk_int() <
base::android::SDK_VERSION_MARSHMALLOW) {
DVLOG(0) << "Skipping test - FrameSizeChange is flaky on KitKat and "
"Lollipop devices.";
return;
......
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