Clear Key CDM should resolve the promise if session not found

media/test/data/eme_player_js/player_utils.js

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // The PlayerUtils provides utility functions to binding common media events
 // to specific player functions. It also provides functions to load media source
 // base on test configurations.
 var PlayerUtils = new function() {
 }
 
@@ skipping 49 matching lines @@
       });
     }
 
     try {
       if (player.testConfig.sessionToLoad) {
         Utils.timeLog('Loading session: ' + player.testConfig.sessionToLoad);
         var session =
             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: ' +
                       Utils.getHexString(new Uint8Array(message.initData)));
         var session = message.target.mediaKeys.createSession();
         addMediaKeySessionListeners(session);
         session.generateRequest(message.initDataType, message.initData)
             .catch(function(error) {
               // Ignore the error if a crash is expected. This ensures that
               // the decoder actually detects and reports the error.
@@ skipping 128 matching lines @@
       default:
         Utils.timeLog(keySystem + ' is not a known key system');
         if (usePrefixedEME)
           return PrefixedClearKeyPlayer;
         return ClearKeyPlayer;
     }
   }
   var Player = getPlayerType(testConfig.keySystem);
   return new Player(video, testConfig);
 };