Closed
Bug 733783
Opened 13 years ago
Closed 12 years ago
GCLI needs commands to support breakpoint stepping
Categories
(DevTools :: Console, defect, P2)
DevTools
Console
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 16
People
(Reporter: jwalker, Assigned: vporof)
References
Details
(Whiteboard: [gclicommands])
Attachments
(2 files, 1 obsolete file)
(deleted),
patch
|
past
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
Details | Diff | Splinter Review |
See notes on bug 683503.
Comment 1•13 years ago
|
||
Something to consider is emulating the relevant commands for GDB:
http://gcc.gnu.org/onlinedocs/gcc-4.1.2/gnat_ugn_unw/Introduction-to-GDB-Commands.html
We could also consider expanding the scope of the GDB emulation to add command aliases for all GDB commands supported by our debugger.
Reporter | ||
Updated•13 years ago
|
Target Milestone: Firefox 15 → Firefox 16
Assignee | ||
Updated•13 years ago
|
Assignee: nobody → vporof
Status: NEW → ASSIGNED
Assignee | ||
Comment 3•13 years ago
|
||
We'll need new strings for stepping because I took them from somewhere.
Attachment #636656 -
Flags: review?(jwalker)
Reporter | ||
Comment 4•12 years ago
|
||
Comment on attachment 636656 [details] [diff] [review]
v1
Review of attachment 636656 [details] [diff] [review]:
-----------------------------------------------------------------
I do think we need to add some tests for these commands. We've skipped it in the past and got bitten.
It's not hard, because there is a framework to help.
Examples:
- browser/devtools/commandline/test/browser_gcli_commands.js
- browser/devtools/commandline/test/browser_gcli_edit.js
There is even documentation (which I should perhaps move to MDN):
- browser/devtools/commandline/test/head.js
See the doc comments for DeveloperToolbarTest.checkInputStatus()
and DeveloperToolbarTest.exec()
Thanks.
Attachment #636656 -
Flags: review?(jwalker)
Comment 5•12 years ago
|
||
Unfortunately I need to add a breakpoint and this process is async ... so blocked by bug 773313
Comment 6•12 years ago
|
||
Works now so we should be fine to land depending on review.
Attachment #636656 -
Attachment is obsolete: true
Attachment #641947 -
Flags: review?(past)
Updated•12 years ago
|
Attachment #641947 -
Attachment is patch: true
Comment 7•12 years ago
|
||
Comment on attachment 641947 [details] [diff] [review]
Patch with working test
Review of attachment 641947 [details] [diff] [review]:
-----------------------------------------------------------------
The only issue I found was a window leak I got when I ran the test. I believe using the registerCleanupFunction idea from the debugger mochitests would fix it.
::: browser/devtools/commandline/test/resources_dbg.html
@@ +27,5 @@
> + function stepOverMe() {
> + output.value = "step over";
> + }
> +
> + function checkTestPaused() {
Nit: this function looks unused.
Attachment #641947 -
Flags: review?(past) → review+
Comment 8•12 years ago
|
||
Updated•12 years ago
|
Whiteboard: [gclicommands] → [gclicommands][land-in-fx-team]
Reporter | ||
Comment 9•12 years ago
|
||
Whiteboard: [gclicommands][land-in-fx-team] → [gclicommands][fixed-in-fx-team]
Reporter | ||
Updated•12 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Comment 10•12 years ago
|
||
Whiteboard: [gclicommands][fixed-in-fx-team] → [gclicommands]
Comment 11•12 years ago
|
||
Commented in the wrong bug this morning.
> The skipped statements are executed, but not stepped through
Could you explain the meaning of this string?
Side note: copy&paste of l10n comments (clearly wrong, e.g. "A very short string used to describe thefunction of the dbg step over command.") is kind of worse than having no comments at all.
Comment 12•12 years ago
|
||
> > The skipped statements are executed, but not stepped through
> Could you explain the meaning of this string?
I can give it a shot ... it is easier to understand when compared with Step In.
Step In and Step Over differ in only one respect, the way they handle function calls. Either command instructs the debugger to execute the next line of code. If the line contains a function call, Step Into executes only the call itself, then halts at the first line of code inside the function. Step Over executes the entire function, then halts at the first line outside the function. Use Step Into if you want to look inside the function call. Use Step Over if you want to avoid stepping into functions.
Bug 774603 has been created so that we can fix any errors in gclicommands.properties, please log any other issues in that bug.
Blocks: 774603
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•