Closed Bug 1050440 Opened 10 years ago Closed 10 years ago

Avoid repeated JS frames in ThreadStackHelper

Categories

(Core :: XPCOM, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla34

People

(Reporter: jchen, Assigned: jchen)

Details

Attachments

(1 file)

Recent changes in how JS pushes frames to the pseudostack made ThreadStackHelper repeat JS frames. e.g.,

> (content script)
> js::RunScript
> (content script)
> js::RunScript
> (content script)
> js::RunScript
> TImer::Fire
> Startup::XRE_Main

We should fix that.
This patch makes us ignore the js:RunScript frames, which add no value over the actual JS file/line frames.
Attachment #8470303 - Flags: review?(snorp)
Attachment #8470303 - Flags: review?(snorp) → review+
Assignee: nobody → nchen
Status: NEW → ASSIGNED
https://hg.mozilla.org/mozilla-central/rev/0a2d3711dd58
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla34
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: