Breakpoint jumps over function declaration
Reported by
account-...@dlehmann.eu,
Nov 14 2017
|
||
Issue descriptionUserAgent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Ubuntu Chromium/59.0.3071.109 Chrome/59.0.3071.109 Safari/537.36 Steps to reproduce the problem: 1. Open attached example.html 2. Open DevTools > Sources > select example.html 3. Reload 4. Set breakpoint in line 7 (buggy case) 5. Set breakpoint in line 11 (correct case) What is the expected behavior? - Breakpoint in line 7 (step 4.) should slide to the same location as the breakpoint in line 11 (step 5.), namely the function body that is following. What went wrong? - Breakpoint in line 7 "jumps over" function, that is, it is slided to line 16 (variable declaration) instead of line 13. - Unclear why breakpoint in line 7 should behave differently from line 11 (both comment lines, both are followed by a function declaration). Did this work before? N/A Chrome version: 59.0.3071.109 Channel: stable OS Version: Ubuntu 16.04 Flash Version: - See also attached screencapture for video demonstration of bug.
,
Nov 14 2017
That's right, I basically wanted to find out "from which line on" the breakpoint is added to the function body (the correct behavior). Apparently line 9, 10, and 11 work as expected (the breakpoint is set in the function), but earlier lines 6, 7, and 8 are having the problem (the breakpoint is set at the variable declaration after the function).
,
Nov 30 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/525ebafbc5297af32a97ab9d4739d7119e274029 commit 525ebafbc5297af32a97ab9d4739d7119e274029 Author: Alexey Kozyatinskiy <kozyatinskiy@chromium.org> Date: Thu Nov 30 00:19:49 2017 [DevTools] ignore empty lines when searching for breakpoint location R=lushnikov@chromium.org Bug: chromium:784852 Change-Id: If45209a8caf48068b6c612ccfd169035af5e63f1 Reviewed-on: https://chromium-review.googlesource.com/794752 Commit-Queue: Aleksey Kozyatinskiy <kozyatinskiy@chromium.org> Reviewed-by: Andrey Lushnikov <lushnikov@chromium.org> Cr-Commit-Position: refs/heads/master@{#520325} [modify] https://crrev.com/525ebafbc5297af32a97ab9d4739d7119e274029/third_party/WebKit/LayoutTests/http/tests/devtools/sources/debugger/source-frame-inline-breakpoint-decorations-expected.txt [modify] https://crrev.com/525ebafbc5297af32a97ab9d4739d7119e274029/third_party/WebKit/LayoutTests/http/tests/devtools/sources/debugger/source-frame-inline-breakpoint-decorations.js [modify] https://crrev.com/525ebafbc5297af32a97ab9d4739d7119e274029/third_party/WebKit/Source/devtools/front_end/sources/JavaScriptSourceFrame.js
,
Nov 30 2017
|
||
►
Sign in to add a comment |
||
Comment 1 by l...@chromium.org
, Nov 14 2017Status: Assigned (was: Unconfirmed)