Fix invalid console.trace() calls in inspector front

RESOLVED FIXED in Firefox 55


2 years ago
Last year


(Reporter: jdescottes, Assigned: anejaalisha, Mentored)



Firefox 55

Firefox Tracking Flags

(firefox55 fixed)



(4 attachments)

The inspector front contains two invalid calls to console.trace() :


`console.trace()` does not take any argument ( Therefore the strings passed to console.trace() in these 2 spots are never logged anywhere.

The goal of this bug is to split the calls in order to first call `console.warn` with the string, and then `console.trace` without any argument.

This should be a good first bug.

You can have a look at to get started!
Can I work on solving this bug?
And if I can, can you provide me with some more details about it?:)
Flags: needinfo?(jdescottes)
Thanks! Assigning the bug to you.

You should find some information to get started in my first comment.
The basic idea is to modify the two calls to console.trace from:

> console.trace("some message");


> console.warn("some message");
> console.trace();

The only issue here is that I don't have a scenario to trigger this issue.
Assignee: nobody → anejaalisha
Flags: needinfo?(jdescottes)
Attachment #8845665 - Attachment is patch: false
Attachment #8845665 - Attachment mime type: text/plain → image/png
I have resolved the bug. But I am pretty confused how to push changes/get them reviewed as earlier I have worked on treeherder/bedrock codebase where we create Pull Requests.

Attachments for your review:)
Attachment #8845664 - Attachment is patch: false
Attachment #8845664 - Attachment mime type: text/plain → image/png
The changes look ok, but you need to submit them as patch, screenshots won't do here :)

If you are using Git, follow the instructions at
If you are using mercurial, this page might be more appropriate:

I can help you if you are really stuck, but please give a shot to the documentation above and let me know which step you are blocked at.
I suppose I have created the patch following all instructions from here:
But I am really stuck as in how to move forward. How to checkout the patch which I worked?
I am also attaching the terminal screenshot.
Flags: needinfo?(jdescottes)
Posted image patch_commands.png
According to your screenshot you should have a Bug1345112.patch on your filesystem. 
Can you please attach this file here?
Flags: needinfo?(jdescottes)
Attachment #8846277 - Flags: review?(jdescottes)
Comment on attachment 8846277 [details] [diff] [review]

Review of attachment 8846277 [details] [diff] [review]:

The patch look good now :) 
Thanks for taking the time to go through this!
Attachment #8846277 - Flags: review?(jdescottes) → review+
Keywords: checkin-needed
Pushed by
Fix invalid console.trace() calls in inspector front. r=jdescottes
Keywords: checkin-needed
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 55
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.