Skip to content

Commit

Permalink
Don't try RemoveTemporarySession test if license server not available.
Browse files Browse the repository at this point in the history
BUG=894422
TEST=test passes on Windows

Change-Id: I3907fecda3642f542db153e68ce12cb33c074402
Reviewed-on: https://chromium-review.googlesource.com/c/1278467
Commit-Queue: Xiaohan Wang <[email protected]>
Reviewed-by: Xiaohan Wang <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#599065}(cherry picked from commit 44229dd)
Reviewed-on: https://chromium-review.googlesource.com/c/1284389
Reviewed-by: John Rummell <[email protected]>
Cr-Commit-Position: refs/branch-heads/3578@{#60}
Cr-Branched-From: 4226ddf-refs/heads/master@{#599034}
  • Loading branch information
jrummell-chromium committed Oct 16, 2018
1 parent 91940c6 commit 2d3251e
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions chrome/browser/media/encrypted_media_browsertest.cc
Original file line number Diff line number Diff line change
Expand Up @@ -656,6 +656,12 @@ IN_PROC_BROWSER_TEST_P(EncryptedMediaTest, PolicyCheck) {
}

IN_PROC_BROWSER_TEST_P(EncryptedMediaTest, RemoveTemporarySession) {
if (!IsPlayBackPossible(CurrentKeySystem())) {
DVLOG(0) << "Skipping test - RemoveTemporarySession test requires license "
"server.";
return;
}

// Although this test doesn't play anything, there is no need to run it
// twice for the same key system.
if (CurrentSourceType() != SrcType::MSE) {
Expand Down

0 comments on commit 2d3251e

Please sign in to comment.