Retry license request on timeouts and errors

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 196 matching lines @@
 };
 
 Utils.resetTitleChange = function() {
   this.titleChanged = false;
   document.title = '';
 };
 
 Utils.sendRequest = function(requestType, responseType, message, serverURL,
                              onSuccessCallbackFn, forceInvalidResponse) {
   var requestAttemptCount = 0;
   var MAXIMUM_REQUEST_ATTEMPTS = 3;

[ddorwin, 2014/10/01 21:00:29]

If we're seeing this sometimes requiring 3 locally, should we up this to 4 so
that we never (hopefully) see this on VMs?

[jrummell, 2014/10/01 21:28:34]

Removed this so we try until the framework decides it's taking too long.

   var REQUEST_RETRY_DELAY_MS = 3000;
+  var REQUEST_TIMEOUT_MS = 1000;
 
   function sendRequestAttempt() {
     requestAttemptCount++;
-    if (requestAttemptCount == MAXIMUM_REQUEST_ATTEMPTS) {
+    if (requestAttemptCount > MAXIMUM_REQUEST_ATTEMPTS) {
       Utils.failTest('FAILED: Exceeded maximum license request attempts.');
       return;
     }
     var xmlhttp = new XMLHttpRequest();
     xmlhttp.responseType = responseType;
     xmlhttp.open(requestType, serverURL, true);
     xmlhttp.onerror = function(e) {
       Utils.timeLog('Request status: ' + this.statusText);
-      Utils.failTest('FAILED: License request XHR failed with network error.');
+      Utils.timeLog('FAILED: License request XHR failed with network error.');
+      Utils.timeLog('Retrying request if possible in ' +
+                    REQUEST_RETRY_DELAY_MS + 'ms');
+      setTimeout(sendRequestAttempt, REQUEST_RETRY_DELAY_MS);

[ddorwin, 2014/10/01 21:00:28]

Why do we wait? Is there an advantage to fewer retries spaced further apart (vs.
closer together but more attempts)?

[jrummell, 2014/10/01 21:28:34]

In this case the license server isn't up, so we might as well wait before trying
again.

     };
     xmlhttp.onload = function(e) {
       if (this.status == 200) {
         if (onSuccessCallbackFn)
           onSuccessCallbackFn(this.response);
       } else {
         Utils.timeLog('Bad response status: ' + this.status);
         Utils.timeLog('Bad response: ' + this.response);
         Utils.timeLog('Retrying request if possible in ' +
                       REQUEST_RETRY_DELAY_MS + 'ms');
         setTimeout(sendRequestAttempt, REQUEST_RETRY_DELAY_MS);
       }
     };
+    xmlhttp.timeout = REQUEST_TIMEOUT_MS;
+    xmlhttp.ontimeout = function(e) {
+      Utils.timeLog('Request timeout');
+      Utils.timeLog('Retrying request if possible in ' +
+                    REQUEST_RETRY_DELAY_MS + 'ms');
+      setTimeout(sendRequestAttempt, REQUEST_RETRY_DELAY_MS);
+    }
     Utils.timeLog('Attempt (' + requestAttemptCount +
                   '): sending request to server: ' + serverURL);
     xmlhttp.send(message);
   }
 
   if (forceInvalidResponse) {
     Utils.timeLog('Not sending request - forcing an invalid response.');
     return onSuccessCallbackFn([0xAA]);
   }
   sendRequestAttempt();
@@ skipping 15 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);
 };