Bug 244301 - Web Inspector: Screenshots timeline is unsupported in WebKitLegacy, we should disable it when inspect such targets
Summary: Web Inspector: Screenshots timeline is unsupported in WebKitLegacy, we should...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Patrick Angle
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2022-08-24 11:29 PDT by Karl Rackler
Modified: 2023-01-24 18:15 PST (History)
5 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Karl Rackler 2022-08-24 11:29:50 PDT
Description:
inspector/timeline/timeline-event-screenshots.html

This test is consistently failing on macOS wk1 since being added at 250535@main. 

REPRODUCTION STEPS
I can reproduce this 253720@main ToT.

Command: 
run-webkit-tests -1 --exit-after-n-failures 1 --exit-after-n-crashes-or-timeouts 1 --iterations 50 -f --no-retry inspector/timeline/timeline-event-screenshots.html

Result: 
Regressions: Unexpected text-only failures (1)
  inspector/timeline/timeline-event-screenshots.html [ Failure ]

History:
https://results.webkit.org/?suite=layout-tests&test=inspector%2Ftimeline%2Ftimeline-event-screenshots.html&flavor=wk1&limit=50000

Diff:
--- /Volumes/Data/worker/Apple-Monterey-Release-WK1-Tests/build/layout-test-results/inspector/timeline/timeline-event-screenshots-expected.txt
+++ /Volumes/Data/worker/Apple-Monterey-Release-WK1-Tests/build/layout-test-results/inspector/timeline/timeline-event-screenshots-actual.txt
@@ -1,4 +1,3 @@
-CONSOLE MESSAGE: Viewport
 Tests 'Screenshot' Timeline event records.
 
 
@@ -6,10 +5,5 @@
 -- Running test case: TimelineEvent.FireScreenshots.requestScreenshots
 Starting Capture...
 Evaluating...
-Stopping Capture...
-PASS: Should have at least 1 Screenshot record.
-PASS: Screenshot record should contain image data.
-Adding screenshot from console...
-PASS: Should have added another Screenshot record.
-PASS: Screenshot record from console should contain image data.
+!! TIMEOUT: took longer than 10000ms
Comment 1 Radar WebKit Bug Importer 2022-08-24 11:30:32 PDT
<rdar://problem/99094864>
Comment 2 Karl Rackler 2022-08-24 11:42:30 PDT
I have marked this test as a consistent failure while this issue is investigated.
Comment 3 EWS 2022-08-24 11:49:52 PDT
Test gardening commit 253735@main (105bd2d6f893): <https://commits.webkit.org/253735@main>

Reviewed commits have been landed. Closing PR #3619 and removing active labels.
Comment 4 Patrick Angle 2023-01-24 14:01:54 PST
Pull request: https://github.com/WebKit/WebKit/pull/9073
Comment 5 EWS 2023-01-24 18:15:56 PST
Committed 259326@main (2129556ec693): <https://commits.webkit.org/259326@main>

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