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

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

Index: third_party/WebKit/LayoutTests/webaudio/resources/audit.js
diff --git a/third_party/WebKit/LayoutTests/webaudio/resources/audit.js b/third_party/WebKit/LayoutTests/webaudio/resources/audit.js
index d2946161efbb1f2cd2f707a8d6487db5145ddee8..256d303461f99f070d2bbb5f46a9cb236169527d 100644
--- a/third_party/WebKit/LayoutTests/webaudio/resources/audit.js
+++ b/third_party/WebKit/LayoutTests/webaudio/resources/audit.js
@@ -15,7 +15,7 @@
 
   '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'
@@ -55,6 +55,13 @@ window.Audit = (function () {
     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;
@@ -1137,7 +1144,7 @@ window.Audit = (function () {
 
       // 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