1050440 - Avoid repeated JS frames in ThreadStackHelper

Status: RESOLVED FIXED

(Core :: XPCOM, defect)

Description

[(inactive) Jim Chen [:jchen] [:darchons]]

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.

Comment 1

[(inactive) Jim Chen [:jchen] [:darchons]]

Attachment: Remove repeated js::RunScript frames in ThreadStackHelper (v1)

This patch makes us ignore the js:RunScript frames, which add no value over the actual JS file/line frames.

Comment 2

[(inactive) Jim Chen [:jchen] [:darchons]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/0a2d3711dd58

Comment 3

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/0a2d3711dd58