No inline/precise breakpoints in for-loop header distributed over multiple lines
Reported by
account-...@dlehmann.eu,
May 9 2017
|
||||
Issue descriptionUserAgent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:53.0) Gecko/20100101 Firefox/53.0 Steps to reproduce the problem: 1. Open example.html in Chromium 58 with new "more preicse"/inline breakpoints feature 2. Open DevTools > Debugger 3. Set breakpoint in line 7 by clicking on gutter 4. Set breakpoint in line 11 by clicking on gutter 5. Set breakpoint in line 17 by clicking on gutter What is the expected behavior? at 4.: Should show "inline breakpoint markers" for loop initialization and loop condition in line 11, as it is done for line 7. at 5.: Should show "inline breakpoint markers" for loop condition and increment in line 17, as it is done for line 7. What went wrong? - When for-loop header is on a single line, "precise" breakpoints can be set by clicking on markers inside the line. - However, when the for-loop header is distributed over multiple lines (by adding a line break after the loop variable initialization or after loop condition), one cannot set inline breakpoints since those markers are not shown. Did this work before? N/A Chrome version: 58.0.3029.81 Channel: n/a OS Version: Ubuntu 16.10 (64-bit) Flash Version: Shockwave Flash 25.0 r0 See attached screen capture for video of bug in Chromium 58 on Ubuntu
,
May 12 2017
Tested the issue on ubuntu 14.04 and 16.04 , mac os 10.12.3 and windows 7 using chrome latest stable M58 #58.0.3029.110 and M60 #60.0.3097.0 issue is reproduced. Issue is seen from M40 #40.0.2172.0 , where no line has "inline breakpoint markers" and is a non-regression issue , as it never worked before. Marking it as untraiged for further inpust on this. Thanks!
,
May 16 2017
Thanks for the report. kozy@, could you please take a look?
,
Jun 6 2017
was fixed by https://chromium-review.googlesource.com/c/472247 and should work in M61 (latest Canary). |
||||
►
Sign in to add a comment |
||||
Comment 1 by ligim...@chromium.org
, May 10 2017Labels: Needs-Triage-M58 Needs-Bisect