Throw when testharness::done() is called by layout test code

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
  */
 
 
 (function () {
 
   'use strict';
 
-  // Selected properties from testharness.js
+  // Selected methods from testharness.js.
   let testharnessProperties = [
     'test', 'async_test', 'promise_test', 'promise_rejects',
     'generate_tests', 'setup', 'done', 'assert_true', 'assert_false'
   ];
 
   // Check if testharness.js is properly loaded. Throw otherwise.
   for (let name in testharnessProperties) {
     if (!self.hasOwnProperty(testharnessProperties[name]))
       throw new Error('Cannot proceed. testharness.js is not loaded.');
   }
@@ skipping 19 matching lines @@
   function _logFailed (message, detail) {
     test(function () {
       assert_true(false, detail);
     }, message);
   }
 
   function _throwException (message) {
     throw new Error(message);
   }
 
+  // TODO(hongchan): remove this hack after confirming all the tests are
+  // finished correctly. (crbug.com/708817)
+  const _testharnessDone = window.done;
+  window.done = () => {
+    _throwException('Do NOT call done() method from the test code.');
+  };
+
   // Generate a descriptive string from a target value in various types.
   function _generateDescription (target, options) {
     let targetString;
 
     switch (typeof target) {
       case 'object':
         // Handle Arrays.
         if (target instanceof Array || target instanceof Float32Array ||
             target instanceof Float64Array || target instanceof Uint8Array) {
           let arrayElements = target.length < options.numberOfArrayElements
@@ skipping 1062 matching lines @@
       if (numberOfFailures > 0) {
         _logFailed(prefix + numberOfFailures + ' out of '
           + this._taskSequence.length + ' tasks were failed.');
       } else {
         _logPassed(prefix + this._taskSequence.length
           + ' tasks ran successfully.');
       }
 
       // From testharness.js, report back to the test infrastructure that
       // the task runner completed all the tasks.
-      done();
+      _testharnessDone();
     }
 
     // |taskLabel| can be either a string or a dictionary. See Task constructor
     // for the detail.
     define (taskLabel, taskFunction) {
       let task = new Task(this, taskLabel, taskFunction);
       if (this._tasks.hasOwnProperty(task.label)) {
         _throwException('Audit.define:: Duplicate task definition.');
         return;
       }
@@ skipping 118 matching lines @@
 
     /**
      * Load file from a given URL and pass ArrayBuffer to the following promise.
      * See |loadFileFromUrl| method for the detail.
      */
     loadFileFromUrl: loadFileFromUrl
 
   };
 
 })();