handle message sync for some significant events and method call results

lib/puppeteer/connection.rb

@@ -45,8 +45,11 @@ def initialize(url, transport, delay = 0)
     @transport = transport
     @transport.on_message do |data|
       message = JSON.parse(data)
-      sleep_before_handling_message(message)
-      async_handle_message(message)
+      if should_handle_synchronously?(message)
+        handle_message(message)
+      else
+        async_handle_message(message)
+      end
     end
     @transport.on_close do |reason, code|
       handle_close
@@ -61,14 +64,27 @@ def closed?
     @closed
   end
 
-  private def sleep_before_handling_message(message)
-    # Puppeteer doesn't handle any Network monitoring responses.
-    # So we don't have to sleep.
-    return if message['method']&.start_with?('Network.')
+  private def should_handle_synchronously?(message)
+    return true if message['id']
 
-    # For some reasons, sleeping a bit reduces trivial errors...
-    # 4ms is an interval of internal shared timer of WebKit.
-    sleep 0.004
+    case message['method']
+    when nil
+      false
+    when /^Network\./
+      # Puppeteer doesn't handle any Network monitoring responses.
+      # So we don't care their handling order.
+      false
+    when /^Target\./
+      true
+    when /^Runtime\.executionContext/
+      true
+    when /^Page\.frame/
+      true
+    when 'Page.lifecycleEvent'
+      true
+    else
+      false
+    end
   end
 
   def self.from_session(session)