Allow status = 0 when XHR is completed in Audit.loadFileFromUrl().

third_party/WebKit/LayoutTests/webaudio/resources/audit.js

 // Copyright 2016 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.
 
 
 /**
  * @fileOverview  WebAudio layout test utility library. Built around W3C's
  *                testharness.js. Includes asynchronous test task manager,
  *                assertion utilities.
  * @dependency    testharness.js
@@ skipping 1173 matching lines @@
    * @example
    *   Audit.loadFileFromUrl('resources/my-sound.ogg').then((response) => {
    *       audioContext.decodeAudioData(response).then((audioBuffer) => {
    *           // Do something with AudioBuffer.
    *       });
    *   });
    */
   function loadFileFromUrl (fileUrl) {
     return new Promise((resolve, reject) => {
       let xhr = new XMLHttpRequest();
-      xhr.open('GET', fileUrl);
+      xhr.open('GET', fileUrl, true);
       xhr.responseType = 'arraybuffer';
 
       xhr.onload = () => {
-        if (xhr.status === 200) {
+        // |status = 0| should be acceptable here because the server used by
+        // run-webkit-test has cross-domain file access issue.

[Raymond Toy, 2017/03/21 21:16:33]

Do we actually know this (cross-domain)?

[hongchan, 2017/03/21 23:07:57]

Comment fixed.

+        if (xhr.status === 200 || xhr.status === 0) {
           resolve(xhr.response);
         } else {
           let errorMessage = 'loadFile: Request failed when loading ' +
-              fileUrl + '. (' + xhr.statusText + ')';
+              fileUrl + '. ' + xhr.statusText + '. (status = ' +
+              xhr.status + ')';
           if (reject) {
             reject(errorMessage);
           } else {
             new Error(errorMessage);
           }
         }
       };
 
       xhr.onerror = (event) => {
         let errorMessage =
@@ skipping 42 matching lines @@
 
     /**
      * Load file from a given URL and pass ArrayBuffer to the following promise.
      * See |loadFileFromUrl| method for the detail.
      */
     loadFileFromUrl: loadFileFromUrl
 
   };
 
 })();