Bug 263447 - REGRESSION ( Sonoma?): [ Sonoma release ] TestWebKitAPI.WKInspectorExtension.EvaluateScriptOnPage is a consistent failure
Summary: REGRESSION ( Sonoma?): [ Sonoma release ] TestWebKitAPI.WKInspectorExtension....
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Razvan Caliman
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2023-10-20 09:23 PDT by Karl Rackler
Modified: 2023-12-12 05:58 PST (History)
5 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Karl Rackler 2023-10-20 09:23:16 PDT 
Description:
TestWebKitAPI.WKInspectorExtension.EvaluateScriptOnPage

Is a consistent failure on Sonoma release ToT and since bring-up.

This issue can be reproduced using the command: 
run-api-tests --release --no-build --iterations=1 TestWebKitAPI.WKInspectorExtension.EvaluateScriptOnPage

History:
https://results.webkit.org/?suite=api-tests&test=TestWebKitAPI.WKInspectorExtension.EvaluateScriptOnPage

Results:
  Failed

    TestWebKitAPI.WKInspectorExtension.EvaluateScriptOnPage
        RemoteLayerTreeDrawingAreaProxyMac::scheduleDisplayLink(): page has no displayID
        RemoteLayerTreeDrawingAreaProxyMac::scheduleDisplayLink(): page has no displayID
        
        /Volumes/Data/worker/Apple-Sonoma-Release-Build/build/Tools/TestWebKitAPI/Tests/WebKitCocoa/WKInspectorExtension.mm:455
        Value of: !(error)
          Actual: false
        Expected: true
        
        
        /Volumes/Data/worker/Apple-Sonoma-Release-Build/build/Tools/TestWebKitAPI/Tests/WebKitCocoa/WKInspectorExtension.mm:456
        Value of: result
          Actual: false
        Expected: true
        
        
        /Volumes/Data/worker/Apple-Sonoma-Release-Build/build/Tools/TestWebKitAPI/Tests/WebKitCocoa/WKInspectorExtension.mm:457
        Expected equality of these values:
          result
            Which is: 
          mainFrameSecretString
            Which is: 42-mainFrame
        
        
        /Volumes/Data/worker/Apple-Sonoma-Release-Build/build/Tools/TestWebKitAPI/Tests/WebKitCocoa/WKInspectorExtension.mm:478
        Value of: !(error)
          Actual: false
        Expected: true
        
        
        /Volumes/Data/worker/Apple-Sonoma-Release-Build/build/Tools/TestWebKitAPI/Tests/WebKitCocoa/WKInspectorExtension.mm:479
        Value of: result
          Actual: false
        Expected: true
        
        
        /Volumes/Data/worker/Apple-Sonoma-Release-Build/build/Tools/TestWebKitAPI/Tests/WebKitCocoa/WKInspectorExtension.mm:480
        Expected equality of these values:
          result
            Which is: 
          innerFrameSecretString
            Which is: 42-innerFrame
        
        
        /Volumes/Data/worker/Apple-Sonoma-Release-Build/build/Tools/TestWebKitAPI/Tests/WebKitCocoa/WKInspectorExtension.mm:491
        Value of: !(error)
          Actual: false
        Expected: true
        
        
        /Volumes/Data/worker/Apple-Sonoma-Release-Build/build/Tools/TestWebKitAPI/Tests/WebKitCocoa/WKInspectorExtension.mm:492
        Value of: result
          Actual: false
        Expected: true
        
        
        /Volumes/Data/worker/Apple-Sonoma-Release-Build/build/Tools/TestWebKitAPI/Tests/WebKitCocoa/WKInspectorExtension.mm:493
        Expected equality of these values:
          result
            Which is: 
          innerFrameSecretString
            Which is: 42-innerFrame
Comment 1 Radar WebKit Bug Importer 2023-10-20 09:23:39 PDT 
<rdar://problem/117265270>
Comment 2 Razvan Caliman 2023-11-08 08:35:52 PST 
It was a race condition within Web Inspector frontend. I have a fix coming up.
Comment 3 Razvan Caliman 2023-11-08 09:24:04 PST 
Pull request: https://github.com/WebKit/WebKit/pull/20173
Comment 4 EWS 2023-12-12 05:58:07 PST 
Committed 271925@main (3583b2ac96f1): <https://commits.webkit.org/271925@main>

Reviewed commits have been landed. Closing PR #20173 and removing active labels.