Need to be able to print backtraces

RESOLVED FIXED in mozilla1.9beta4

Status

()

Core
JavaScript Engine
RESOLVED FIXED
10 years ago
10 years ago

People

(Reporter: (dormant account), Assigned: (dormant account))

Tracking

({regression})

Trunk
mozilla1.9beta4
x86
Mac OS X
regression
Points:
---
Bug Flags:
blocking1.9 +
in-testsuite -
in-litmus -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

10 years ago
The patch in bug 350650 caused the exception to be cleared before js_ReportErrorAgain calls the user-defined error handler.

Is there an alternative way of getting an error backtrace?
(Assignee)

Comment 1

10 years ago
Created attachment 306647 [details] [diff] [review]
Keep exn object around for the callback
Comment on attachment 306647 [details] [diff] [review]
Keep exn object around for the callback

This seems fine to me, only a couple of nits.

>Index: jsexn.c
>         /* Flag the error as an exception. */
>         reportp->flags |= JSREPORT_EXCEPTION;
>+        /* Pass the exception object */

Add a blank line between the line and the error. Also, end the sentence with a period.
Blake, can you pick nits, get approval, and land? Need to fix this regression for 1.9/fx3.

/be
Assignee: general → mrbkap
Flags: blocking1.9+
Keywords: regression
Created attachment 306786 [details] [diff] [review]
Updated to my nits

I don't know which release this should track, trying b4 first, though I don't think it would be too terrible for this to go in after the freeze.
Attachment #306647 - Attachment is obsolete: true
Attachment #306786 - Flags: review+
Attachment #306786 - Flags: approval1.9b4?
Comment on attachment 306786 [details] [diff] [review]
Updated to my nits

a=mconnor on behalf of drivers
Attachment #306786 - Flags: approval1.9b4? → approval1.9b4+

Updated

10 years ago
Assignee: mrbkap → tglek
Fix checked in. Thanks for the patch, Taras!
Status: NEW → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9beta4

Updated

10 years ago
Flags: in-testsuite-
Flags: in-litmus-
You need to log in before you can comment on or make changes to this bug.