zhuangzhineng / chromedevtools

Automatically exported from code.google.com/p/chromedevtools
BSD 3-Clause "New" or "Revised" License
0 stars 0 forks source link

Breakpoint set by JavascriptVm#setBreakpoint won't work if WIP backend is used and 'column' parameter is set to '-1' as specified in javadoc #57

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
SDK/Debugger version: 0.3.2
Google Chrome/V8 Embedder + version: 16.0.912.75
OS + version: Ubuntu 11.10

What steps will reproduce the problem?
1. Connect to Chrome using SDK with WIP backend (wk97678).
2. Set a breakpoint on some line by calling JavascriptVm#setBreakpoint method 
with 'column' parameter set to Breakpoint.EMPTY_VALUE (-1), as specified in the 
javadoc. BreakpointCallback#success method will be called reporting that the 
breakpoint is successfully created.
3. Make Chrome to execute the script containing the breakpoint

What is the expected result?
Execution should be suspended on the breakpoint.

What happens instead?
The breakpoint is ignored. 

However if 'column' parameter is set to '0' the breakpoint will work properly. 
So if now 'column=0' is an intended way to set a breakpoint for the whole line 
it should at least be reflected in the javadoc.

Original issue reported on code.google.com by nikolay....@gmail.com on 19 Jan 2012 at 10:18

GoogleCodeExporter commented 9 years ago

Original comment by peter.ry...@gmail.com on 22 Jan 2012 at 2:07

GoogleCodeExporter commented 9 years ago
Fixed in HEAD

Original comment by peter.ry...@gmail.com on 31 Jan 2012 at 4:47

GoogleCodeExporter commented 9 years ago
Fixed in 0.3.4

Original comment by peter.ry...@gmail.com on 25 Feb 2012 at 12:56