Do not override Promise prorotype in layout test infrastructure

We can use the same fix as in https://crrev.com/c/1964310
and listen for "unhandledrejection" events instead.

[email protected]

Bug: none
Change-Id: I9e00f50f36d5cf9f2e54b2d1ce467ad6589afaff
Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/2919952
Auto-Submit: Tim van der Lippe <[email protected]>
Reviewed-by: Jack Franklin <[email protected]>
Commit-Queue: Jack Franklin <[email protected]>
Commit-Queue: Tim van der Lippe <[email protected]>

front_end/legacy_test_runner/sources_test_runner/DebuggerTestRunner.js

@@ -37,28 +37,10 @@ SourcesTestRunner.completeDebuggerTest = function() {
   });
 };
 
-(function() {
-const origThen = Promise.prototype.then;
-const origCatch = Promise.prototype.catch;
-
-Promise.prototype.then = function() {
-  const result = origThen.apply(this, arguments);
-  origThen.call(result, undefined, onUncaughtPromiseReject.bind(null, new Error().stack));
-  return result;
-};
-
-Promise.prototype.catch = function() {
-  const result = origCatch.apply(this, arguments);
-  origThen.call(result, undefined, onUncaughtPromiseReject.bind(null, new Error().stack));
-  return result;
-};
-
-function onUncaughtPromiseReject(stack, e) {
-  const message = typeof e === 'object' && e.stack || e;
-  TestRunner.addResult('FAIL: Uncaught exception in promise: ' + message + ' ' + stack);
+window.addEventListener('unhandledrejection', e => {
+  TestRunner.addResult('FAIL: Uncaught exception in promise: ' + e + ' ' + e.stack);
   SourcesTestRunner.completeDebuggerTest();
-}
-})();
+});
 
 SourcesTestRunner.runDebuggerTestSuite = function(testSuite) {
   const testSuiteTests = testSuite.slice();