Open Bug 726174 Opened 12 years ago Updated 2 years ago

fix slow-script dialog proper

Categories

(Core :: DOM: Core & HTML, defect, P3)

x86_64
Linux
defect

Tracking

()

People

(Reporter: luke, Unassigned)

References

Details

(Whiteboard: [Snappy:P2])

The slow-script dialog logic needs to be re-fixed.  Bug 699974 contained a flawed fix that needed to be backed out for the reason stated in bug 720289 comment 3.  To restate: it is possible (esp. with flash plugins) for reentrent code to permanently escape the slow-script dialog and has caused several reported hangs.

I'm not an expert in this area (clearly) but it seems that nsJSContext needs a stack, not a single field, for ctx->callbackTime so that an outer invocation is neither liberated nor hung by an inner invocation.
Will wait on chromehang(bug 712109) to determine severity of this
Whiteboard: [Snappy] → [Snappy:P2]
Priority: -- → P3
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.