Add unprefixed EME tests for LoadSession()

Add unprefixed EME tests for LoadSession()

BUG=338830
TEST=new EME tests pass

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

[jrummell, 2014-10-08 00:37:41]

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

[jrummell, 2014-10-08 00:37:42]

PTAL. This CL will land after the blink and Chromium changes go in.

[sandersd (OOO until July 31), 2014-10-08 18:20:02]

lgtm

[ddorwin, 2014-10-08 22:04:28]

LGTM % comment.

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

https://codereview.chromium.org/641443002/diff/1/chrome/browser/media/encrypt...
chrome/browser/media/encrypted_media_browsertest.cc:337: class
ECKUnprefixedEncryptedMediaTest : public EncryptedMediaTestBase {
TODO: Merge with ECKEncryptedMediaTest once unprefixed is enabled by default.

https://codereview.chromium.org/641443002/diff/1/media/test/data/eme_player_j...
File media/test/data/eme_player_js/player_utils.js (right):

https://codereview.chromium.org/641443002/diff/1/media/test/data/eme_player_j...
media/test/data/eme_player_js/player_utils.js:66: .catch(function(error) {
Utils.failTest(error, KEY_ERROR); });
In theory, these catches aren't necessary because of line 79.

[jrummell, 2014-10-16 18:53:59]

Thanks for the reviews.

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

https://codereview.chromium.org/641443002/diff/1/chrome/browser/media/encrypt...
chrome/browser/media/encrypted_media_browsertest.cc:337: class
ECKUnprefixedEncryptedMediaTest : public EncryptedMediaTestBase {
On 2014/10/08 22:04:28, ddorwin wrote:
> TODO: Merge with ECKEncryptedMediaTest once unprefixed is enabled by default.

Done.

https://codereview.chromium.org/641443002/diff/1/media/test/data/eme_player_j...
File media/test/data/eme_player_js/player_utils.js (right):

https://codereview.chromium.org/641443002/diff/1/media/test/data/eme_player_j...
media/test/data/eme_player_js/player_utils.js:66: .catch(function(error) {
Utils.failTest(error, KEY_ERROR); });
On 2014/10/08 22:04:28, ddorwin wrote:
> In theory, these catches aren't necessary because of line 79.

Failed promises don't throw, so line 79 won't run.

[jrummell, 2014-10-16 18:54:03]

The CQ bit was checked by jrummell@chromium.org

[commit-bot: I haz the power, 2014-10-16 18:54:36]

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

[commit-bot: I haz the power, 2014-10-16 19:45:14]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2014-10-16 19:48:12]

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