Bug 211559 - Web Inspector: Sources: Hidden breakpoints status (enabled/disabled) is not consistent
Summary: Web Inspector: Sources: Hidden breakpoints status (enabled/disabled) is not c...
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (show other bugs)
Version: Safari 13
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-05-07 03:40 PDT by Valentin Gjorgjioski
Modified: 2020-05-07 13:10 PDT (History)
3 users (show)

See Also:


Attachments
Step 1 and Step 2 (119.52 KB, image/png)
2020-05-07 12:44 PDT, Valentin Gjorgjioski
no flags Details
Step 3 (150.62 KB, image/png)
2020-05-07 12:45 PDT, Valentin Gjorgjioski
no flags Details
Step 4 (292.15 KB, image/png)
2020-05-07 12:45 PDT, Valentin Gjorgjioski
no flags Details
Step 4 second part (127.41 KB, image/png)
2020-05-07 12:45 PDT, Valentin Gjorgjioski
no flags Details
Steps 5 and 6 (368.79 KB, image/png)
2020-05-07 12:46 PDT, Valentin Gjorgjioski
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Valentin Gjorgjioski 2020-05-07 03:40:58 PDT
1. Open Web Inspector -> Sources. 
2. Enable Breakpoints "All timeouts" "All Intervals"
3. Hide them (not to be visible) 
4. Although they are enabled, they are not taken into account. Maybe that is correct behaviour (logical). However: 

5. Enable breakpoints "All requests" 
6. Hide them (not to be visible)
7. Browser stops for every request (now they are taken into account). 

One of these (4 or 7) should be corrected for consistency. I would rather see number 7 fixed. When they are hidden, they shouldn't be taken into account. Or when user hide it, it should be disabled.
Comment 1 Devin Rousso 2020-05-07 09:37:21 PDT
(In reply to Valentin Gjorgjioski from comment #0)
> 3. Hide them (not to be visible) 
What do you mean when you say "hide them"?  It'd be helpful if you included more descriptive steps like "press delete" or "right click and select Delete Breakpoint" :)
Comment 2 Valentin Gjorgjioski 2020-05-07 10:50:28 PDT
Hide them is: Click on the plus next to Breakpoints and uncheck them (so they will not appear on the list anymore).
Comment 3 BJ Burg 2020-05-07 11:01:33 PDT
Your text labels for breakpoints does not match what's in the UI. I recommend you upload a screen recording that demonstrates the issue.
Comment 4 Valentin Gjorgjioski 2020-05-07 12:44:49 PDT
Created attachment 398777 [details]
Step 1 and Step 2
Comment 5 Valentin Gjorgjioski 2020-05-07 12:45:06 PDT
Created attachment 398778 [details]
Step 3
Comment 6 Valentin Gjorgjioski 2020-05-07 12:45:24 PDT
Created attachment 398779 [details]
Step 4
Comment 7 Valentin Gjorgjioski 2020-05-07 12:45:54 PDT
Created attachment 398780 [details]
Step 4 second part
Comment 8 Valentin Gjorgjioski 2020-05-07 12:46:15 PDT
Created attachment 398781 [details]
Steps 5 and 6
Comment 9 Valentin Gjorgjioski 2020-05-07 12:48:13 PDT
Hmmm, I have attached steps 1 to 6, but now I can't simulate Step7. The only thing I did I downloaded the STP 105 (and restarted Safari in meantime). However, Step 7 now I can't simulate neither in Safari, nor in STP, although it was there the last few days, and whole day today until I resolved this. 

Connected to this please see someone else asking the following, and the answer is by me:  

https://stackoverflow.com/questions/61010199/breakpoints-in-safari-dev-tools-get-enabled-automatically/61654892#61654892

And there is shown step 7, that (for some reason) now I can't reproduce anymore with my Safari. Here is a direct link to Step 7. 

https://i.stack.imgur.com/TGtim.png
Comment 10 Valentin Gjorgjioski 2020-05-07 13:10:58 PDT
The only thing I can imagine is that this option "All requests" was there in the list, and was enabled, but pane/graphics was not drawing / was hiding this option for some reason (maybe it was too small and wasn't shown?!). So after me clicking from the toolbar the pane redrew and I could see it. I have no other explanation. 

Another thing, it seems that bug #211558 appears ONLY when "All requests" is enabled bellow. Please see the video in that bug.  I had breakpoint "All requests" for that video enabled. This was not done intentionally, but now when I am writing this comment I just conclude that. 

So, since I could simulate these two bugs all the time the previous few days, and at least one other guy also asking on stack-overflow and on apple community, leads me to a conclusion, this group of breakpoints was indeed enabled but for some reason was not shown in the pane.