Closed Bug 1355084 Opened 8 years ago Closed 7 years ago

Add debugDumpJSStack to the deprecation warnings

Categories

(Core :: SQLite and Embedded Database Bindings, enhancement, P3)

Product:
Core
Component:
SQLite and Embedded Database Bindings
Type:
enhancement

Tracking

()

Status:
RESOLVED FIXED
Tracking Flags:
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: 7 years ago
Depends on: 1355561
Resolution: --- → FIXED
Product: Toolkit → Core
You need to log in before you can comment on or make changes to this bug.