[PATCH v2 2/2] user32/tests: Do not stop checking the message list when skipping an optional or unsupported message.

[Zebediah Figura]

From: Zebediah Figura

This was either added by accident, or in an attempt to stop checking the message
list once we hit a winevent hook todo [and accidentally given too wide of a
scope]. However, the same commit also uses a global counter to ensure that only
one winevent todo is printed, so we don't need to break in the first hunk
anyway.

For trailing winevent todos, there is no global counter, so instead move the
goto inside the if block.

Fixes: a72bffe768c9fe462010d7e40e20226322fb82c7
---
 dlls/user32/tests/msg.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/dlls/user32/tests/msg.c b/dlls/user32/tests/msg.c
index 303d6dbf012..d1b43455464 100644
--- a/dlls/user32/tests/msg.c
+++ b/dlls/user32/tests/msg.c
@@ -2853,7 +2853,6 @@ static void ok_sequence_(const struct message *expected_list, const char *contex
                 }
             }
        expected++;
-            goto done;
         }
    else if (todo)
    {
@@ -2888,9 +2887,9 @@ static void ok_sequence_(const struct message *expected_list, const char *contex
                 ok_( file, line) (FALSE, "%s: %u: the msg sequence is not complete: expected 0x%04x - actual 0x%04x\n",
                                 context, count, expected->message, actual->message);
             }
+            goto done;
         }
    expected++;
-        goto done;
     }

if (todo)