Use arrow functions in handleSleep calls. NFC (#20200)

I noticed this while reviewing #20182.

Author: sbc100

site/source/docs/porting/asyncify.rst

@@ -185,7 +185,7 @@ code will resume:
 .. code-block:: cpp
 
     EM_JS(int, do_fetch, (), {
-      return Asyncify.handleSleep(function (wakeUp) {
+      return Asyncify.handleSleep((wakeUp) => {
         out("waiting for a fetch");
         fetch("a.html").then(function (response) {
           out("got the fetch response");

src/library_async.js

@@ -191,7 +191,7 @@ addToLibrary({
     StackSize: {{{ ASYNCIFY_STACK_SIZE }}},
     currData: null,
     // The return value passed to wakeUp() in
-    // Asyncify.handleSleep(function(wakeUp){...}) is stored here,
+    // Asyncify.handleSleep((wakeUp) => {...}) is stored here,
     // so we can return it later from the C function that called
     // Asyncify.handleSleep() after rewinding finishes.
     handleSleepReturnValue: 0,

src/library_idbstore.js

@@ -81,8 +81,8 @@ var LibraryIDBStore = {
   emscripten_idb_load__async: true,
   emscripten_idb_load__deps: ['malloc'],
   emscripten_idb_load: (db, id, pbuffer, pnum, perror) => {
-    return Asyncify.handleSleep(function(wakeUp) {
-      IDBStore.getFile(UTF8ToString(db), UTF8ToString(id), function(error, byteArray) {
+    return Asyncify.handleSleep((wakeUp) => {
+      IDBStore.getFile(UTF8ToString(db), UTF8ToString(id), (error, byteArray) => {
         if (error) {
           {{{ makeSetValue('perror', 0, '1', 'i32') }}};
           wakeUp();
@@ -99,26 +99,26 @@ var LibraryIDBStore = {
   },
   emscripten_idb_store__async: true,
   emscripten_idb_store: (db, id, ptr, num, perror) => {
-    return Asyncify.handleSleep(function(wakeUp) {
-      IDBStore.setFile(UTF8ToString(db), UTF8ToString(id), new Uint8Array(HEAPU8.subarray(ptr, ptr+num)), function(error) {
+    return Asyncify.handleSleep((wakeUp) => {
+      IDBStore.setFile(UTF8ToString(db), UTF8ToString(id), new Uint8Array(HEAPU8.subarray(ptr, ptr+num)), (error) => {
         {{{ makeSetValue('perror', 0, '!!error', 'i32') }}};
         wakeUp();
       });
     });
   },
   emscripten_idb_delete__async: true,
   emscripten_idb_delete: (db, id, perror) => {
-    return Asyncify.handleSleep(function(wakeUp) {
-      IDBStore.deleteFile(UTF8ToString(db), UTF8ToString(id), function(error) {
+    return Asyncify.handleSleep((wakeUp) => {
+      IDBStore.deleteFile(UTF8ToString(db), UTF8ToString(id), (error) => {
         {{{ makeSetValue('perror', 0, '!!error', 'i32') }}};
         wakeUp();
       });
     });
   },
   emscripten_idb_exists__async: true,
   emscripten_idb_exists: (db, id, pexists, perror) => {
-    return Asyncify.handleSleep(function(wakeUp) {
-      IDBStore.existsFile(UTF8ToString(db), UTF8ToString(id), function(error, exists) {
+    return Asyncify.handleSleep((wakeUp) => {
+      IDBStore.existsFile(UTF8ToString(db), UTF8ToString(id), (error, exists) => {
         {{{ makeSetValue('pexists', 0, '!!exists', 'i32') }}};
         {{{ makeSetValue('perror',  0, '!!error', 'i32') }}};
         wakeUp();
@@ -128,9 +128,9 @@ var LibraryIDBStore = {
   // extra worker methods - proxied
   emscripten_idb_load_blob__async: true,
   emscripten_idb_load_blob: (db, id, pblob, perror) => {
-    return Asyncify.handleSleep(function(wakeUp) {
+    return Asyncify.handleSleep((wakeUp) => {
       assert(!IDBStore.pending);
-      IDBStore.pending = function(msg) {
+      IDBStore.pending = (msg) => {
         IDBStore.pending = null;
         var blob = msg.blob;
         if (!blob) {
@@ -154,9 +154,9 @@ var LibraryIDBStore = {
   },
   emscripten_idb_store_blob__async: true,
   emscripten_idb_store_blob: (db, id, ptr, num, perror) => {
-    return Asyncify.handleSleep(function(wakeUp) {
+    return Asyncify.handleSleep((wakeUp) => {
       assert(!IDBStore.pending);
-      IDBStore.pending = function(msg) {
+      IDBStore.pending = (msg) => {
         IDBStore.pending = null;
         {{{ makeSetValue('perror', 0, '!!msg.error', 'i32') }}};
         wakeUp();

test/browser/async_returnvalue.cpp

@@ -11,14 +11,14 @@
 // A "sync" tunnel that adds 1.
 #if USE_EM_JS
 EM_JS(int, sync_tunnel, (int value), {
-  return Asyncify.handleSleep(function(wakeUp) {
+  return Asyncify.handleSleep((wakeUp) => {
     setTimeout(function() {
       wakeUp(value + 1);
     }, 1);
   });
 })
 EM_JS(int, sync_tunnel_bool, (bool value), {
-  return Asyncify.handleSleep(function(wakeUp) {
+  return Asyncify.handleSleep((wakeUp) => {
     setTimeout(function() {
       wakeUp(!value);
     }, 1);

test/browser/async_returnvalue.js

@@ -1,13 +1,13 @@
 addToLibrary({
   sync_tunnel: function(value) {
-    return Asyncify.handleSleep(function(wakeUp) {
+    return Asyncify.handleSleep((wakeUp) => {
       setTimeout(function() {
         wakeUp(value + 1);
       }, 1);
     });
   },
   sync_tunnel_bool: function(value) {
-    return Asyncify.handleSleep(function(wakeUp) {
+    return Asyncify.handleSleep((wakeUp) => {
       setTimeout(function() {
         wakeUp(!value);
       }, 1);

test/core/test_asyncify_assertions.c

@@ -2,7 +2,7 @@
 #include <emscripten.h>
 
 EM_JS(void, suspend, (), {
-  return Asyncify.handleSleep(function(wakeUp) {
+  Asyncify.handleSleep((wakeUp) => {
     Module.resume = wakeUp;
     setTimeout(function() {
       Module._resume_from_inside_c();