Don't attempt playback if not possible

Don't attempt playback if not possible

Some key systems aren't supported on all platforms, so don't try the
multiple playback tests in those cases.

BUG=467505
TEST=EncryptedMediaTests pass

Committed: https://crrev.com/d9dc790dc2066fd61313d137b405b017dabe969c
Cr-Commit-Position: refs/heads/master@{#320792}

[jrummell, 2015-03-16 19:01:34]

jrummell@chromium.org changed reviewers:
+ ddorwin@chromium.org, sandersd@chromium.org

[jrummell, 2015-03-16 19:01:35]

PTAL.

[ddorwin, 2015-03-16 19:17:47]

https://codereview.chromium.org/1016483002/diff/1/chrome/browser/media/encryp...
File chrome/browser/media/encrypted_media_browsertest.cc (right):

https://codereview.chromium.org/1016483002/diff/1/chrome/browser/media/encryp...
chrome/browser/media/encrypted_media_browsertest.cc:419: // TestSimplePlayback()
checks for KEY_ERROR if playback is not possible.
Instead, I would follow the pattern at line 596.

[jrummell, 2015-03-16 20:13:06]

Updated.

https://codereview.chromium.org/1016483002/diff/1/chrome/browser/media/encryp...
File chrome/browser/media/encrypted_media_browsertest.cc (right):

https://codereview.chromium.org/1016483002/diff/1/chrome/browser/media/encryp...
chrome/browser/media/encrypted_media_browsertest.cc:419: // TestSimplePlayback()
checks for KEY_ERROR if playback is not possible.
On 2015/03/16 19:17:47, ddorwin wrote:
> Instead, I would follow the pattern at line 596.

Done.

[ddorwin, 2015-03-16 20:26:30]

lgtm

Thanks.

[jrummell, 2015-03-16 20:27:55]

The CQ bit was checked by jrummell@chromium.org

[commit-bot: I haz the power, 2015-03-16 20:28:40]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1016483002/20001

[commit-bot: I haz the power, 2015-03-16 20:59:36]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-03-16 21:00:13]

Patchset 2 (id:??) landed as
https://crrev.com/d9dc790dc2066fd61313d137b405b017dabe969c
Cr-Commit-Position: refs/heads/master@{#320792}