chore(trace): do not nest API actions based on time

packages/playwright/src/index.ts

@@ -261,7 +261,6 @@ const playwrightFixtures: Fixtures<TestFixtures, WorkerFixtures> = ({
           title: renderApiCall(apiName, params),
           apiName,
           params,
-          canNestByTime: true,
         });
         userData.userObject = step;
         out.stepId = step.stepId;

packages/playwright/src/worker/testInfo.ts

@@ -44,9 +44,6 @@ export interface TestStepInternal {
   infectParentStepsWithError?: boolean;
   box?: boolean;
   isStage?: boolean;
-  // TODO: this сould be decided based on the category, but pw:api
-  // is from a different abstraction layer.
-  canNestByTime?: boolean;
 }
 
 export type TestStage = {
@@ -255,11 +252,6 @@ export class TestInfoImpl implements TestInfo {
       parentStep = this._findLastStageStep();
     } else {
       parentStep = zones.zoneData<TestStepInternal>('stepZone');
-      if (!parentStep && data.canNestByTime) {
-        // API steps (but not test.step calls) can be nested by time, instead of by stack.
-        // However, do not nest chains of route.continue by checking the title.
-        parentStep = this._findLastNonFinishedStep(step => step.title !== data.title);
-      }
       if (!parentStep) {
         // If no parent step on stack, assume the current stage as parent.
         parentStep = this._findLastStageStep();

tests/playwright-test/playwright.trace.spec.ts

@@ -1216,9 +1216,9 @@ test('should not nest top level expect into unfinished api calls ', {
     '    browserContext.newPage',
     'page.route',
     'page.goto',
-    '  route.fetch',
+    'route.fetch',
-    '    page.unrouteAll',
     'expect.toBeVisible',
+    'page.unrouteAll',
     'After Hooks',
     '  fixture: page',
     '  fixture: context',