470765 - Better diagnostics for "unresponsive script" - Load the script into a "source view" window

Status: RESOLVED WONTFIX

(Core :: JavaScript Engine, enhancement)

Description

[Ulrich Windl]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.1.19) Gecko/20081213 SUSE/1.1.14-0.1 SeaMonkey/1.1.14
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.1.19) Gecko/20081213 SUSE/1.1.14-0.1 SeaMonkey/1.1.14

Recently those "unresponsive scripts" seem to become more and more "popular". The browser only offers "Stop the Script" or "Continue Script" after a time-out. This is not very helpful to the user. Therefor I'd suggest to add one or more of the following:

1) Show the URI of the script that causes the error

2) Show the line of the script code that is currently executed

3) Load the script into a "source view" window

Eventually I hope these measures will help to find out the programmers of bad code, and in the long run those terrible code will vanish.

Reproducible: Always

Steps to Reproduce:
1. Load a site with slow scripting code (like sf.net, qa.openoffice.org, etc.)

Actual Results:  
You are not having fun with those sites. Scripts are very slow (e.g. try to search a specific bug in qa.openoffice.org). The pop-up isn't very helpful in dealing with the situation.

Expected Results:  
On the long run: Better web sites.
Soon: A way to debug bad scripts.

Comment 1

[Karsten Düsterloh]

Not sure how easy (1)-(3) are to implement, but it's a valid RfE.
We surely won't be able fix it in the SM 1.x series, though.

Comment 2

[Tony Mechelynck [:tonymec]]

Karsten: Wouldn't this RFE rather belong in Core::JavaScript Engine? Or else, in SeaMonkey::UI Design?

Comment 3

[Karsten Düsterloh]

At a minimum, we'd need hooks in the JS code to get the data, I suppose.

Comment 4

[Matthew Cline]

Isn't the all currently implemented, except for the "link to pop up a view-source window" part?  Should this bug be closed?

Comment 5

[Wayne Mery (:wsmwk)]

(In reply to comment #4)
> Isn't the all currently implemented, except for the "link to pop up a
> view-source window" part?  Should this bug be closed?

yeah, Load the script into a "source view" window is the only point remaining

Comment 6

[Aleksey Tsalolikhin]

This seems related to ticket 397394