Open Bug 277713 Opened 18 years ago Updated 3 years ago

infinite loop (slow script) warning should also be displayed in JS Console

Categories

(Core :: DOM: Core & HTML, enhancement, P5)

Product:
Core
Component:
DOM: Core & HTML
Type:
enhancement

Tracking

()

People

(Reporter: Biesinger, Unassigned)

References

Details

Attachments

(1 file)

patch
7.09 KB, patch
jst
: review-
Details | Diff | Splinter Review 
the "Slow script" warning dialog should also be shown in the JS Console,
especially to show a file name/line number. the patch I'm going to attach also
shows the function name.
Attached patch patch — — Splinter Review
Attachment #170782 - Flags: review?(jst)
Severity: normal → enhancement
Status: NEW → ASSIGNED
OS: Linux → All
Hardware: PC → All
Target Milestone: --- → mozilla1.8beta
Comment on attachment 170782 [details] [diff] [review]
patch

This seems like an aweful lot of code to print a message on the console which
is really only printed when we either execute malicious code or when a web
developer forgets to end a loop. Other than that this would be really really
rare, so I can't really justify adding all this code for that. How about just a
call to PrintWarningOnConsole() (currently file-static in nsDOMClassInfo.cpp)?
Attachment #170782 - Flags: review?(jst) → review-
QA Contact: ian → general
Duplicate of this bug: 257646
Assignee: cbiesinger → nobody
Target Milestone: mozilla1.8beta1 → ---
https://bugzilla.mozilla.org/show_bug.cgi?id=1472046

Move all DOM bugs that haven't been updated in more than 3 years and has no one currently assigned to P5.

If you have questions, please contact :mdaly.
Priority: -- → P5
No assignee, updating the status.
Status: ASSIGNED → NEW
No assignee, updating the status.
No assignee, updating the status.
No assignee, updating the status.
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.