Closed Bug 1355084 Opened 4 years ago Closed 3 years ago

Add debugDumpJSStack to the deprecation warnings

Categories

(Toolkit :: Storage, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED
Tracking Status
firefox55 --- affected

People

(Reporter: mak, Assigned: mak, Mentored)

References

Details

(Keywords: good-first-bug, Whiteboard: [good-first-bug][lang=cpp])

See the discussion from https://bugzilla.mozilla.org/show_bug.cgi?id=1007034#c29 and the suggestion at https://bugzilla.mozilla.org/show_bug.cgi?id=1007034#c34

The cpp native stack is not really useful if we crash for a deprecated API, we could also print out the js stack.

This should be trivial, good to verify the dev environment is well setup to make a build.
Depends on: 1007034
Marco, it appears as if you have already added in a JS stack dump, within the 365069 changeset.

Should this bug be marked as closed?
Matt, you are right, thank you for noticing that. This has been fixed in bug 1355561.
Assignee: nobody → mak77
Status: NEW → RESOLVED
Closed: 3 years ago
Depends on: 1355561
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.