Bug 1687292 - [devtools] Remove unecessary await for TargetList.destroy usages. r=jdescottes

Differential Revision: https://phabricator.services.mozilla.com/D102191

devtools/shared/resources/tests/browser_resources_client_caching.js

@@ -45,7 +45,7 @@ add_task(async function() {
   assertContents(cachedResources1, messages);
   assertResources(cachedResources2, cachedResources1);
 
-  await targetList.destroy();
+  targetList.destroy();
   await client.close();
 });
 
@@ -93,7 +93,7 @@ add_task(async function() {
   assertContents(availableResources, allMessages);
   assertResources(cachedResources, availableResources);
 
-  await targetList.destroy();
+  targetList.destroy();
   await client.close();
 });
 
@@ -134,7 +134,7 @@ add_task(async function() {
 
   is(cachedResources.length, 0, "The cache in ResourceWatcher is cleared");
 
-  await targetList.destroy();
+  targetList.destroy();
   await client.close();
 });
 
@@ -192,7 +192,7 @@ add_task(async function() {
 
   assertResources(cachedResources, availableResources);
 
-  await targetList.destroy();
+  targetList.destroy();
   await client.close();
 });
 
@@ -261,7 +261,7 @@ async function testIgnoreExistingResources(isFirstListenerIgnoreExisting) {
   assertContents(cachedResourcesWithFlag, additionalMessages);
   assertContents(cachedResourcesWithoutFlag, allMessages);
 
-  await targetList.destroy();
+  targetList.destroy();
   await client.close();
 }
 
@@ -308,7 +308,7 @@ add_task(async function() {
   resourceWatcher.unwatchResources([ResourceWatcher.TYPES.CONSOLE_MESSAGE], {
     onAvailable,
   });
-  await targetList.destroy();
+  targetList.destroy();
   await client.close();
 });
 

devtools/shared/resources/tests/browser_resources_console_messages.js

@@ -167,7 +167,7 @@ async function testTabConsoleMessagesResourcesWithIgnoreExistingResources(
     checkConsoleAPICall(message, expected);
   }
 
-  await targetList.destroy();
+  targetList.destroy();
   await client.close();
 
   await SpecialPowers.spawn(tab.linkedBrowser, [], async () => {

devtools/shared/resources/tests/browser_resources_css_changes.js

@@ -89,7 +89,7 @@ add_task(async function() {
   is(availableResources[2], existingResources[2], "3rd resource is correct");
   is(availableResources[3], existingResources[3], "4th resource is correct");
 
-  await targetList.destroy();
+  targetList.destroy();
   await client.close();
 });
 

devtools/shared/resources/tests/browser_resources_document_events.js

@@ -59,7 +59,7 @@ async function testDocumentEventResources() {
   );
   ok(true, "Document events are fired after reloading");
 
-  await targetList.destroy();
+  targetList.destroy();
   await client.close();
 }
 
@@ -86,7 +86,7 @@ async function testDocumentEventResourcesWithIgnoreExistingResources() {
   await waitUntil(() => documentEvents.length === 3);
   assertEvents(...documentEvents);
 
-  await targetList.destroy();
+  targetList.destroy();
   await client.close();
 }
 

devtools/shared/resources/tests/browser_resources_error_messages.js

@@ -138,7 +138,7 @@ async function testErrorMessagesResourcesWithIgnoreExistingResources() {
   }
 
   Services.console.reset();
-  await targetList.destroy();
+  targetList.destroy();
   await client.close();
 }
 

devtools/shared/resources/tests/browser_resources_getAllResources.js

@@ -75,7 +75,7 @@ add_task(async function() {
     []
   );
 
-  await targetList.destroy();
+  targetList.destroy();
   await client.close();
 });
 

devtools/shared/resources/tests/browser_resources_network_event_stacktraces.js

@@ -98,7 +98,7 @@ add_task(async function() {
     }
   );
 
-  await targetList.destroy();
+  targetList.destroy();
   await client.close();
   BrowserTestUtils.removeTab(tab);
 });

devtools/shared/resources/tests/browser_resources_network_events.js

@@ -240,7 +240,7 @@ async function testNetworkEventResources(options) {
       onUpdated: onResourceUpdated,
     }
   );
-  await targetList.destroy();
+  targetList.destroy();
   await client.close();
   BrowserTestUtils.removeTab(tab);
 }

devtools/shared/resources/tests/browser_resources_platform_messages.js

@@ -144,6 +144,6 @@ async function testPlatformMessagesResourcesWithIgnoreExistingResources() {
   }
 
   Services.console.reset();
-  await targetList.destroy();
+  targetList.destroy();
   await client.close();
 }

devtools/shared/resources/tests/browser_resources_stylesheets.js

@@ -169,7 +169,7 @@ async function testResourceAvailableFeature() {
     ADDITIONAL_FROM_ACTOR_RESOURCE
   );
 
-  await targetList.destroy();
+  targetList.destroy();
   await client.close();
 }
 
@@ -297,7 +297,7 @@ async function testResourceUpdateFeature() {
   );
   assertMediaRules(styleSheetResult.mediaRules, expectedMediaRules);
 
-  await targetList.destroy();
+  targetList.destroy();
   await client.close();
 }
 
@@ -407,7 +407,7 @@ async function testNestedResourceUpdateFeature() {
 
   tab.ownerGlobal.resizeTo(originalWindowWidth, originalWindowHeight);
 
-  await targetList.destroy();
+  targetList.destroy();
   await client.close();
 }
 

devtools/shared/resources/tests/browser_resources_target_switching.js

@@ -96,6 +96,6 @@ add_task(async function() {
     onAvailable: onSourceAvailable,
   });
 
-  await targetList.destroy();
+  targetList.destroy();
   await client.close();
 });

devtools/shared/resources/tests/browser_resources_websocket.js

@@ -115,7 +115,7 @@ add_task(async function() {
     );
   }
 
-  await targetList.destroy();
+  targetList.destroy();
   await client.close();
 });
 

devtools/shared/resources/tests/browser_target_list_processes.js

@@ -29,7 +29,7 @@ add_task(async function() {
 
   await testProcesses(targetList, mainProcess);
 
-  await targetList.destroy();
+  targetList.destroy();
   // Wait for all the targets to be fully attached so we don't have pending requests.
   await Promise.all(
     targetList