chore: move shouldPauseBeforeStep into protocol.yml (#36964)

Author: dgozman

packages/playwright-core/src/server/debugger.ts

@@ -130,24 +130,13 @@ function shouldPauseOnCall(sdkObject: SdkObject, metadata: CallMetadata): boolea
 }
 
 function shouldPauseBeforeStep(metadata: CallMetadata): boolean {
-  // Don't stop on internal.
   if (metadata.internal)
     return false;
-  // Always stop on 'close'
-  if (metadata.method === 'close')
-    return true;
-  if (metadata.method === 'waitForSelector' || metadata.method === 'waitForEventInfo' || metadata.method === 'querySelector' || metadata.method === 'querySelectorAll')
-    return false;  // Never stop on those, primarily for the test harness.
-  const step = metadata.type + '.' + metadata.method;
-  // Stop before everything that generates snapshot. But don't stop before those marked as pausesBeforeInputActions
-  // since we stop in them on a separate instrumentation signal.
-  const metainfo = methodMetainfo.get(step);
-  if (metainfo?.internal)
-    return false;
-  return !!metainfo?.snapshot && !metainfo.pausesBeforeInput;
+  const metainfo = methodMetainfo.get(metadata.type + '.' + metadata.method);
+  return !!metainfo?.pausesBeforeAction;
 }
 
-export function shouldSlowMo(metadata: CallMetadata): boolean {
+function shouldSlowMo(metadata: CallMetadata): boolean {
   const metainfo = methodMetainfo.get(metadata.type + '.' + metadata.method);
   return !!metainfo?.slowMo;
 }