Clear Key CDM should resolve the promise if session not found

media/test/data/eme_player_js/player_utils.js

Index: media/test/data/eme_player_js/player_utils.js
diff --git a/media/test/data/eme_player_js/player_utils.js b/media/test/data/eme_player_js/player_utils.js
index 1fd75435af20add8acede57c1db957c17d30f54b..a532e1af7a37224d2d1f967fa1b2b7e5272a9d33 100644
--- a/media/test/data/eme_player_js/player_utils.js
+++ b/media/test/data/eme_player_js/player_utils.js
@@ -67,7 +67,12 @@ PlayerUtils.registerEMEEventListeners = function(player) {
             message.target.mediaKeys.createSession('persistent-license');
         addMediaKeySessionListeners(session);
         session.load(player.testConfig.sessionToLoad)
-            .catch(function(error) { Utils.failTest(error, EME_LOAD_FAILED); });
+            .then(
+                function(result) {
+                  if (!result)
+                    Utils.failTest('Session not found.', EME_SESSION_NOT_FOUND);

[ddorwin, 2015/07/08 20:32:13]

failTest() is actually used to report expected "errors" in some cases. :(

[jrummell, 2015/07/08 20:58:17]

I guess all that is required is a title change to end the test. I could simply
call Utils.setResultInTitle() here, but I don't think that makes things any
clearer.

[ddorwin, 2015/07/08 21:50:13]

Hmm, that might be more consistent with the fact that this is not an "error" in
the spec.

More generally, "fail" depends on what the test that calls this expected.
There's nothing to do in this CL - it was just an observation.
Utils.ReportResult or something might be a better name. These "utils" functions
probably shouldn't ever "fail" unless it's something that would be considered a
DCHECK.

[jrummell, 2015/07/08 23:15:08]

Done.

+                },
+                function(error) { Utils.failTest(error, EME_LOAD_FAILED); });
       } else {
         Utils.timeLog('Creating new media key session for initDataType: ' +
                       message.initDataType + ', initData: ' +