[browser_tests] Remove unnecessary try-catch

The C++ side no longer uses DOMMessageQueue to call this function so
we can let the async function resolve/reject directly.

[email protected]

Bug: 382003727
Change-Id: I5385a9be1374c1321bdc524a4ba076cecaa048a3
Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6074128
Commit-Queue: Simon Zünd <[email protected]>
Auto-Submit: Simon Zünd <[email protected]>
Reviewed-by: Alex Rudenko <[email protected]>
Commit-Queue: Alex Rudenko <[email protected]>

Author: szuend

front_end/Tests.js

@@ -158,40 +158,33 @@
   };
 
   TestSuite.prototype.setupLegacyFilesForTest = async function() {
-    try {
-      // 'Tests.js' is executed on 'about:blank' so we can't use `import` directly without
-      // specifying the full devtools://devtools/bundled URL.
-      ([
-        Common,
-        HostModule,
-        Root,
-        SDK,
-        Sources,
-        Timeline,
-        UI,
-        Workspace,
-      ] =
-           await Promise.all([
-             self.runtime.loadLegacyModule('core/common/common.js'),
-             self.runtime.loadLegacyModule('core/host/host.js'),
-             self.runtime.loadLegacyModule('core/root/root.js'),
-             self.runtime.loadLegacyModule('core/sdk/sdk.js'),
-             self.runtime.loadLegacyModule('panels/sources/sources.js'),
-             self.runtime.loadLegacyModule('panels/timeline/timeline.js'),
-             self.runtime.loadLegacyModule('ui/legacy/legacy.js'),
-             self.runtime.loadLegacyModule('models/workspace/workspace.js'),
-           ]));
-
-      // We have to map 'Host.InspectorFrontendHost' as the C++ uses it directly.
-      self.Host = {};
-      self.Host.InspectorFrontendHost = HostModule.InspectorFrontendHost.InspectorFrontendHostInstance;
-      self.Host.InspectorFrontendHostAPI = HostModule.InspectorFrontendHostAPI;
-
-      this.reportOk_();
-    } catch (e) {
-      this.reportFailure_(e);
-      throw e;
-    }
+    // 'Tests.js' is executed on 'about:blank' so we can't use `import` directly without
+    // specifying the full devtools://devtools/bundled URL.
+    ([
+      Common,
+      HostModule,
+      Root,
+      SDK,
+      Sources,
+      Timeline,
+      UI,
+      Workspace,
+    ] =
+         await Promise.all([
+           self.runtime.loadLegacyModule('core/common/common.js'),
+           self.runtime.loadLegacyModule('core/host/host.js'),
+           self.runtime.loadLegacyModule('core/root/root.js'),
+           self.runtime.loadLegacyModule('core/sdk/sdk.js'),
+           self.runtime.loadLegacyModule('panels/sources/sources.js'),
+           self.runtime.loadLegacyModule('panels/timeline/timeline.js'),
+           self.runtime.loadLegacyModule('ui/legacy/legacy.js'),
+           self.runtime.loadLegacyModule('models/workspace/workspace.js'),
+         ]));
+
+    // We have to map 'Host.InspectorFrontendHost' as the C++ uses it directly.
+    self.Host = {};
+    self.Host.InspectorFrontendHost = HostModule.InspectorFrontendHost.InspectorFrontendHostInstance;
+    self.Host.InspectorFrontendHostAPI = HostModule.InspectorFrontendHostAPI;
   };
 
   /**