[test] Improve test_sdl2_key. NFC

The TEST_EMSCRIPTEN_SDL_SETEVENTHANDLER macro is copied over from
`test_sdl_key.c` (the SDL1 version) but its not applicable with SDL2.

Author: sbc100

test/browser/test_sdl2_key.c

@@ -19,14 +19,14 @@ int EventHandler(void *userdata, SDL_Event *event) {
       break;
     case SDL_KEYDOWN:
       switch (event->key.keysym.sym) {
-        case SDLK_RIGHT: printf("right\n"); result *= 7; break;
-        case SDLK_LEFT: printf("left\n"); result *= 11; break;
-        case SDLK_DOWN: printf("down\n"); result *= 13; break;
-        case SDLK_UP: printf("up\n"); result *= 17; break;
-        case SDLK_a: printf("a\n"); result *= 19; break;
+        case SDLK_RIGHT: printf("KEYDOWN: right\n"); result *= 7; break;
+        case SDLK_LEFT: printf("KEYDOWN: left\n"); result *= 11; break;
+        case SDLK_DOWN: printf("KEYDOWN: down\n"); result *= 13; break;
+        case SDLK_UP: printf("KEYDOWN: up\n"); result *= 17; break;
+        case SDLK_a: printf("KEYDOWN: a\n"); result *= 19; break;
         default: {
           if (event->key.keysym.scancode == SDL_SCANCODE_B) {
-            printf("b scancode\n"); result *= 23; break;
+            printf("KEYDOWN: b scancode\n"); result *= 23; break;
           }
           printf("unknown key: sym %d scancode %d\n", event->key.keysym.sym, event->key.keysym.scancode);
           emscripten_force_exit(result); // comment this out to leave event handling active. Use the following to log DOM keys:
@@ -35,8 +35,9 @@ int EventHandler(void *userdata, SDL_Event *event) {
       }
       break;
     case SDL_TEXTINPUT:
+      printf("TEXTINPUT: %c\n", event->text.text[0]);
       if (event->text.text[0] == 'A') {
-        printf("a\n");result *= 5;
+        result *= 5;
       }
       break;
     default: /* Report an unhandled event */
@@ -45,13 +46,11 @@ int EventHandler(void *userdata, SDL_Event *event) {
   return 0;
 }
 
-void one() {
-#ifndef TEST_EMSCRIPTEN_SDL_SETEVENTHANDLER
+void pump_events() {
   SDL_Event event;
   while (SDL_PollEvent(&event)) {
     EventHandler(0, &event);
   }
-#endif
 }
 
 int main(int argc, char **argv) {
@@ -61,12 +60,6 @@ int main(int argc, char **argv) {
                             SDL_WINDOWPOS_CENTERED, SDL_WINDOWPOS_CENTERED,
                             640, 480, 0);
 
-#ifdef TEST_EMSCRIPTEN_SDL_SETEVENTHANDLER
-  emscripten_SDL_SetEventHandler(EventHandler, 0);
-#else
-  one();
-#endif
-
   SDL_StartTextInput();
 
   emscripten_run_script("keydown(38);keyup(38)"); // up
@@ -77,7 +70,7 @@ int main(int argc, char **argv) {
   emscripten_run_script("keydown(66);keyup(66);"); // b
   emscripten_run_script("keydown(100);keyup(100);"); // trigger the end
 
-  emscripten_exit_with_live_runtime();
+  emscripten_set_main_loop(pump_events, 3, 0);
   return 99;
 }
 

test/test_browser.py

@@ -3041,14 +3041,6 @@ def test_sdl2_image_formats(self):
   @no_wasm64('SDL2 + wasm64')
   def test_sdl2_key(self):
     create_file('pre.js', '''
-      Module.postRun = () => {
-        function doOne() {
-          Module._one();
-          setTimeout(doOne, 1000/60);
-        }
-        setTimeout(doOne, 1000/60);
-      }
-
       function keydown(c) {
         var event = new KeyboardEvent("keydown", { 'keyCode': c, 'charCode': c, 'view': window, 'bubbles': true, 'cancelable': true });
         var prevented = !document.dispatchEvent(event);
@@ -3066,7 +3058,7 @@ def test_sdl2_key(self):
       }
     ''')
 
-    self.btest_exit('test_sdl2_key.c', 37182145, args=['-sUSE_SDL=2', '--pre-js', 'pre.js', '-sEXPORTED_FUNCTIONS=_main,_one'])
+    self.btest_exit('test_sdl2_key.c', 37182145, args=['-sUSE_SDL=2', '--pre-js', 'pre.js'])
 
   @no_wasm64('SDL2 + wasm64')
   def test_sdl2_text(self):