Remove dispatch optimization (#2732)

Deno.core.dispatch() used to push the "control" buf onto the shared
array buffer before calling into V8, with the idea that it was one less
argument to parse. Turns out there is no more overhead passing the
control ArrayBuffer directly over. Furthermore this optimization was
making the refactors outlined in #2730 more complex. Therefore it is
being removed.

1 files changed, 1 insertions, 5 deletions

diff --git a/core/shared_queue.js b/core/shared_queue.js
index d7ab382d6..1b338b052 100644
--- a/core/shared_queue.js
+++ b/core/shared_queue.js
@@ -165,11 +165,7 @@ SharedQueue Binary Layout
 
   function dispatch(control, zeroCopy = null) {
     maybeInit();
-    // First try to push control to shared.
-    const success = push(control);
-    // If successful, don't use first argument of core.send.
-    const arg0 = success ? null : control;
-    return Deno.core.send(arg0, zeroCopy);
+    return Deno.core.send(control, zeroCopy);
   }
 
   const denoCore = {