Bug 211558 - Web Inspector: Sources: Breakpoints status not remembered/restored correctly
Summary: Web Inspector: Sources: Breakpoints status not remembered/restored correctly
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (show other bugs)
Version: Safari 13
Hardware: All All
: P2 Normal
Assignee: Devin Rousso
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2020-05-07 03:35 PDT by Valentin Gjorgjioski
Modified: 2020-05-07 16:02 PDT (History)
4 users (show)

See Also:


Attachments
Video recording with bug description on STP 105 (9.11 MB, video/quicktime)
2020-05-07 12:55 PDT, Valentin Gjorgjioski
no flags Details
Patch (1.80 KB, patch)
2020-05-07 13:31 PDT, Devin Rousso
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Valentin Gjorgjioski 2020-05-07 03:35:47 PDT
1. Open web inspector -> Sources. 
2. Press Command+Y or Click the "Disable All Breakpoints"
3. Close the web inspector
4. Open the web inspector - Expecting All breakpoints to be disabled, but they are not. 

Related: 
https://stackoverflow.com/questions/61010199/breakpoints-in-safari-dev-tools-get-enabled-automatically/61654892#61654892
https://discussions.apple.com/thread/251235650?replyId=251235650021
Comment 1 Devin Rousso 2020-05-07 09:38:50 PDT
I am unable to reproduce this in STP 105.  Can you attach a video, or more detailed steps to reproduce?
Comment 2 Valentin Gjorgjioski 2020-05-07 12:55:59 PDT
Created attachment 398785 [details]
Video recording with bug description on STP 105
Comment 3 Valentin Gjorgjioski 2020-05-07 13:12:33 PDT
Please see my last comment https://bugs.webkit.org/show_bug.cgi?id=211559#c10 on bug #211559.
Comment 4 Devin Rousso 2020-05-07 13:31:15 PDT
Created attachment 398790 [details]
Patch
Comment 5 Joseph Pecoraro 2020-05-07 14:15:37 PDT
Comment on attachment 398790 [details]
Patch

Nice. r=me
Comment 6 EWS 2020-05-07 16:01:37 PDT
Committed r261340: <https://trac.webkit.org/changeset/261340>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 398790 [details].
Comment 7 Radar WebKit Bug Importer 2020-05-07 16:02:22 PDT
<rdar://problem/62997044>