Open Bug 277713 Opened 20 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)

enhancement

Tracking

()

People

(Reporter: Biesinger, Unassigned)

References

Details

Attachments

(1 file)

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 patchSplinter 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
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
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: