Summary: | Web Inspector: Timelines: can't reliably stop/start a recording | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Devin Rousso <hi> | ||||||||||
Component: | Web Inspector | Assignee: | Devin Rousso <hi> | ||||||||||
Status: | RESOLVED FIXED | ||||||||||||
Severity: | Normal | CC: | commit-queue, ews-watchlist, hi, inspector-bugzilla-changes, joepeck, keith_miller, mark.lam, msaboff, saam, timothy, tsavell, webkit-bug-importer | ||||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||||
Version: | WebKit Nightly Build | ||||||||||||
Hardware: | All | ||||||||||||
OS: | All | ||||||||||||
See Also: | https://bugs.webkit.org/show_bug.cgi?id=152904 | ||||||||||||
Bug Depends on: | |||||||||||||
Bug Blocks: | 203935 | ||||||||||||
Attachments: |
|
Description
Devin Rousso
2019-04-10 12:56:02 PDT
Created attachment 367156 [details]
Patch
Patch to see how bots like the new state system.
This patch modifies the inspector protocol. Please ensure that any frontend changes appropriately use feature checks for new protocol features. Comment on attachment 367156 [details] Patch Clearing flags on attachment: 367156 Committed r244158: <https://trac.webkit.org/changeset/244158> All reviewed patches have been landed. Closing bug. It looks like the changes in https://trac.webkit.org/changeset/244158/webkit has caused 8 test failures: inspector/timeline/line-column.html inspector/timeline/timeline-event-CancelAnimationFrame.html inspector/timeline/timeline-event-FireAnimationFrame.html inspector/timeline/timeline-event-RequestAnimationFrame.html inspector/timeline/timeline-event-TimerFire.html inspector/timeline/timeline-event-TimerInstall.html inspector/timeline/timeline-event-TimerRemove.html inspector/timeline/timeline-recording.html These tests are effecting all of Mac. Can this be resolved quickly? Diff of inspector/timeline/line-column.html: https://build.webkit.org/results/Apple%20High%20Sierra%20Release%20WK2%20(Tests)/r244167%20(10623)/inspector/timeline/line-column-diff.txt History of inspector/timeline/line-column.html: https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=inspector%2Ftimeline%2Fline-column.html Reverted r244158 for reason: Casued 8 inspector/timeline/ test failures. Committed r244191: <https://trac.webkit.org/changeset/244191> Created attachment 367230 [details]
Patch
Comment on attachment 367230 [details] Patch Rejecting attachment 367230 [details] from commit-queue. Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-02', 'validate-changelog', '--check-oops', '--non-interactive', 367230, '--port=mac']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit ChangeLog entry in LayoutTests/ChangeLog contains OOPS!. Full output: https://webkit-queues.webkit.org/results/11844142 Created attachment 367235 [details]
Patch
Comment on attachment 367235 [details] Patch Rejecting attachment 367235 [details] from commit-queue. Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-01', 'validate-changelog', '--check-oops', '--non-interactive', 367235, '--port=mac']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit ChangeLog entry in Source/JavaScriptCore/ChangeLog contains OOPS!. Full output: https://webkit-queues.webkit.org/results/11844224 Created attachment 367236 [details]
Patch
Comment on attachment 367236 [details] Patch Clearing flags on attachment: 367236 Committed r244195: <https://trac.webkit.org/changeset/244195> All reviewed patches have been landed. Closing bug. |