Bug 1873521: Code cleanup r=valentin,necko-reviewers,kershaw

Differential Revision: https://phabricator.services.mozilla.com/D197953
dothq · Jan 16, 2024 · f83e54c · f83e54c
1 parent 1549cd5
commit f83e54c
Showing 1 changed file with 20 additions and 22 deletions.
diff --git a/netwerk/dns/DNSRequestChild.cpp b/netwerk/dns/DNSRequestChild.cpp
@@ -395,32 +395,30 @@ DNSRequestSender::Cancel(nsresult reason) {
     return NS_ERROR_NOT_AVAILABLE;
   }
 
-  if (mIPCActor->CanSend()) {
-    // We can only do IPDL on the main thread
-    nsCOMPtr<nsIRunnable> runnable = NS_NewRunnableFunction(
-        "net::CancelDNSRequestEvent",
-        [actor(mIPCActor), host(mHost), trrServer(mTrrServer), port(mPort),
-         type(mType), originAttributes(mOriginAttributes), flags(mFlags),
-         reason]() {
-          if (!actor->CanSend()) {
-            return;
-          }
-
-          if (DNSRequestChild* child = actor->AsDNSRequestChild()) {
-            Unused << child->SendCancelDNSRequest(
-                host, trrServer, port, type, originAttributes, flags, reason);
-          } else if (DNSRequestParent* parent = actor->AsDNSRequestParent()) {
-            Unused << parent->SendCancelDNSRequest(
-                host, trrServer, port, type, originAttributes, flags, reason);
-          }
-        });
-    SchedulerGroup::Dispatch(runnable.forget());
-  }
+  // We can only do IPC on the MainThread
+  nsCOMPtr<nsIRunnable> runnable = NS_NewRunnableFunction(
+      "net::CancelDNSRequestEvent",
+      [actor(mIPCActor), host(mHost), trrServer(mTrrServer), port(mPort),
+       type(mType), originAttributes(mOriginAttributes), flags(mFlags),
+       reason]() {
+        if (!actor->CanSend()) {
+          return;
+        }
+
+        if (DNSRequestChild* child = actor->AsDNSRequestChild()) {
+          Unused << child->SendCancelDNSRequest(
+              host, trrServer, port, type, originAttributes, flags, reason);
+        } else if (DNSRequestParent* parent = actor->AsDNSRequestParent()) {
+          Unused << parent->SendCancelDNSRequest(
+              host, trrServer, port, type, originAttributes, flags, reason);
+        }
+      });
+  SchedulerGroup::Dispatch(runnable.forget());
   return NS_OK;
 }
 
 void DNSRequestSender::StartRequest() {
-  // we can only do IPDL on the main thread
+  // we can only do IPC on the MainThread
   if (!NS_IsMainThread()) {
     SchedulerGroup::Dispatch(
         NewRunnableMethod("net::DNSRequestSender::StartRequest", this,