Flaky API Test: TestWebKitAPI.ProcessSwap.NavigateToDataURLThenBack

[Aakash Jain]

TestWebKitAPI.ProcessSwap.NavigateToDataURLThenBack seems flaky. In https://ews-build.webkit-uat.org/#/builders/20/builds/889, the test passed in run-api-tests step. However, in the immediately next retry step (re-run-api-tests), it failed.

[Aakash Jain]

Can we prioritize this?
I am working on EWS for API tests, and this flaky failure is creating problem.

[Chris Dumez]

TestWebKitAPI.ProcessSwap.SuspendedPageLimit
        Received data during response processing, queuing it.
        Received data during response processing, queuing it.
        Received data during response processing, queuing it.
        Received data during response processing, queuing it.
        Received data during response processing, queuing it.
        
        /Volumes/Data/worker/iOS-11-Simulator-Build-EWS/build/Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:2720
        Expected equality of these values:
          [processPool _webProcessCountIgnoringPrewarmed]
            Which is: 4
          (1U + maximumSuspendedPageCount)
            Which is: 3

[Chris Dumez]

It is actually:
 TestWebKitAPI.ProcessSwap.NavigateToDataURLThenBack
        Received data during response processing, queuing it.
        Received data during response processing, queuing it.
        
        /Volumes/Data/worker/iOS-11-Simulator-Build-EWS/build/Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:3484
        Expected equality of these values:
          3
          numberOfDecidePolicyCalls
            Which is: 4

[Chris Dumez]

(In reply to Chris Dumez from comment #3)
> It is actually:
>  TestWebKitAPI.ProcessSwap.NavigateToDataURLThenBack
>         Received data during response processing, queuing it.
>         Received data during response processing, queuing it.
>         
>        
> /Volumes/Data/worker/iOS-11-Simulator-Build-EWS/build/Tools/TestWebKitAPI/
> Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:3484
>         Expected equality of these values:
>           3
>           numberOfDecidePolicyCalls
>             Which is: 4

Not sure how we can end up with an extra policy decision here...

[Chris Dumez]

(In reply to Chris Dumez from comment #4)
> (In reply to Chris Dumez from comment #3)
> > It is actually:
> >  TestWebKitAPI.ProcessSwap.NavigateToDataURLThenBack
> >         Received data during response processing, queuing it.
> >         Received data during response processing, queuing it.
> >         
> >        
> > /Volumes/Data/worker/iOS-11-Simulator-Build-EWS/build/Tools/TestWebKitAPI/
> > Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:3484
> >         Expected equality of these values:
> >           3
> >           numberOfDecidePolicyCalls
> >             Which is: 4
> 
> Not sure how we can end up with an extra policy decision here...

Oh, I have an idea, I'll confirm.

[Chris Dumez]

Created attachment 363151 [details]
Patch

[WebKit Commit Bot]

Comment on attachment 363151 [details]
Patch

Clearing flags on attachment: 363151

Committed r242179: <https://trac.webkit.org/changeset/242179>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Radar WebKit Bug Importer]

<rdar://problem/48456644>