Closed Bug 975895 Opened 11 years ago Closed 6 years ago

add console.assert() method to devtools/Console.jsm API

Categories

(DevTools :: Console, defect, P3)

defect

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: zombie, Unassigned)

References

Details

(Whiteboard: [polish-backlog][difficulty=easy])

bug 760193 added console.assert() method to the public web console API in Firefox 28. i'm not sure who is the primary audience for the Console.jsm API; i believe there are other, preferred methods for doing assertions in internal browser code, but at least according to some pages on mdn [1], Console.jsm is advertized to addon authors, so API parity with the apparent standard [2] seems like a good idea. https://developer.mozilla.org/en-US/docs/Tools/Browser_Console#Messages_from_add-ons https://getfirebug.com/wiki/index.php/Console.assert as a bonus, addon SDK uses devtools/Console.jsm directly to inject a global `console` object into every jetpack addon, so this fix would automatically work there too.
Whiteboard: [good first bug]
Thanks for the bug report. This issue is being fixed in bug 965860 where the console API is being rewritten in C++, and unified for both addons and DOM content scripts.
Depends on: 965860
Whiteboard: [good first bug]
Depends on: 988636
Chrome does this, would be nice to see it sooner rather than later.
Whiteboard: [devedition-40][parity]
Setting devedition-40 bugs to p3, filter on FB20EAC4-3FC3-48D9-B15F-0587C3987C25
Priority: -- → P3
Note, this is just for Console.jsm; not the console used in content (which has console.assert already)
(In reply to Jordan Santell [:jsantell] [@jsantell] from comment #4) > Note, this is just for Console.jsm; not the console used in content (which > has console.assert already) d'oh, I kind of don't care then.
Component: Developer Tools → Developer Tools: Console
Whiteboard: [devedition-40][parity] → [polish-backlog][parity]
Blocks: 1199386
Whiteboard: [polish-backlog][parity] → [polish-backlog]
Whiteboard: [polish-backlog] → [polish-backlog][difficulty=easy]
No longer blocks: 1199386
I'm not sure why Ember wants console.assert in Console.jsm which is only relevant to the browser toolbox. Removing from the list as its not on their current wish list AFAIK.
No longer blocks: ember
Product: Firefox → DevTools

Since we plan to remove Console.jsm at some point (Bug 1430810), and given that there's no activity for a while, I think we can close this.

Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.