Closed
Bug 1095635
Opened 10 years ago
Closed 10 years ago
Provide a command line option to dump gecko stdout to the console for debugging marionette
Categories
(Remote Protocol :: Marionette, defect)
Remote Protocol
Marionette
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla36
People
(Reporter: chmanchester, Assigned: chmanchester)
References
Details
Attachments
(1 file, 1 obsolete file)
I have something for this sitting in my patch queue for working on marionette. If others would find it convenient we could add it as a commandline option.
Comment 1•10 years ago
|
||
Does it mean that it will print out the dump() calls from within Firefox? If that is the case I would really be interested in given that at the moment it is like a shut in the dark to figure out what's wrong in a js code block passed to execute script.
Assignee | ||
Comment 2•10 years ago
|
||
(In reply to Henrik Skupin (:whimboo) from comment #1) > Does it mean that it will print out the dump() calls from within Firefox? If > that is the case I would really be interested in given that at the moment it > is like a shut in the dark to figure out what's wrong in a js code block > passed to execute script. I think it would, but also lots of other debug output from the server, so maybe not the best mechanism for trying to debug tests. We should do what we can to make that better, but may have a solution elsewhere.
Assignee | ||
Comment 3•10 years ago
|
||
We'll want anything here to work in concert with the approach in bug 1084565.
Depends on: 1084565
Assignee | ||
Comment 4•10 years ago
|
||
Attachment #8523916 -
Flags: review?(ahalberstadt)
Assignee | ||
Comment 5•10 years ago
|
||
/r/723 - Bug 1095635 - Accept '-' to correspond to stdout for marionette's --gecko-log option. Pull down this commit: hg pull review -r f7e630149f51ce2cb039a4e4730cf0a0572dc7c9
Assignee | ||
Comment 6•10 years ago
|
||
https://reviewboard.mozilla.org/r/721/#review313 ::: testing/marionette/mach_commands.py (Diff revision 1) > + @CommandArgument('--gecko-log', If we don't think this is generally useful I'll keep it to myself, but this is what I had in mind.
Updated•10 years ago
|
Attachment #8523916 -
Flags: review?(ahalberstadt) → review+
Comment 7•10 years ago
|
||
https://reviewboard.mozilla.org/r/721/#review325 Makes sense to me!
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → cmanchester
Assignee | ||
Comment 8•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/ebb36c4fc37f
https://hg.mozilla.org/mozilla-central/rev/ebb36c4fc37f
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
Assignee | ||
Comment 10•9 years ago
|
||
Attachment #8523916 -
Attachment is obsolete: true
Attachment #8618572 -
Flags: review+
Assignee | ||
Comment 11•9 years ago
|
||
Updated•1 year ago
|
Product: Testing → Remote Protocol
You need to log in
before you can comment on or make changes to this bug.
Description
•