Clear Key CDM should resolve the promise if session not found

media/test/data/eme_player_js/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.
 
 // Utils provide logging functions and other JS functions commonly used by the
 // app and media players.
 var Utils = new function() {
   this.titleChanged = false;
 };
 
@@ skipping 106 matching lines @@
       return;
     }
   }
   // The list does not have the option, let's add it and select it.
   var optionElement = new Option(option, option);
   optionElement.title = option;
   selectElement.options.add(optionElement);
   selectElement.value = option;
 };
 
 Utils.failTest = function(msg, newTitle) {

[ddorwin, 2015/07/09 00:29:36]

See below. We should comment that this should only be used for unexpected
failures.

However, I'm not sure the distinction is meaningful. A better solution might be
to rename this reportResult.

I think we should probably revisit this in a separate CL.

[jrummell, 2015/07/10 00:16:50]

Acknowledged.

   var failMessage = 'FAIL: ';
   var title = 'FAILED';
   // Handle exception messages;
   if (msg.message) {
     title = msg.name || 'Error';
     failMessage += title + ' ' + msg.message;
   } else if (msg instanceof Event) {
     // Handle failing events.
     failMessage = msg.target + '.' + msg.type;
     title = msg.type;
   } else {
     failMessage += msg;
   }
   // Force newTitle if passed.
   title = newTitle || title;
   // Log failure.
   Utils.documentLog(failMessage, false);
   console.log(failMessage, msg);
   Utils.setResultInTitle(title);
 };
 
+Utils.reportResult = function(msg, newTitle) {

[ddorwin, 2015/07/09 00:29:36]

Please add some comments. For example, reports a result. The result may or may
not be a failure depending on the test expectations.

[jrummell, 2015/07/10 00:16:50]

Removed function.

+  Utils.timeLog(msg);

[ddorwin, 2015/07/09 00:29:35]

Continuing from the first comment: If this really is a failure, we get a timeLog
instead of a documentLog. Should we in that case? What is the difference? (We're
missing comments for both of those methods. :( )

[jrummell, 2015/07/10 00:16:50]

Removed function.

+  Utils.setResultInTitle(newTitle);
+};
+
 Utils.getCurrentTimeString = function() {
   var date = new Date();
   var hours = ('0' + date.getHours()).slice(-2);
   var minutes = ('0' + date.getMinutes()).slice(-2);
   var secs = ('0' + date.getSeconds()).slice(-2);
   var milliSecs = ('00' + date.getMilliseconds()).slice(-3);
   return hours + ':' + minutes + ':' + secs + '.' + milliSecs;
 };
 
 Utils.getDefaultKey = function(forceInvalidResponse) {
@@ skipping 128 matching lines @@
   var time = Utils.getCurrentTimeString();
   // Log to document.
   Utils.documentLog(arguments[0], time);
   // Log to JS console.
   var logString = time + ' - ';
   for (var i = 0; i < arguments.length; i++) {
     logString += ' ' + arguments[i];
   }
   console.log(logString);
 };