Closed Bug 504189 Opened 15 years ago Closed 15 years ago

Need to make slow script dialog delay higher On WinCE

Categories

(Firefox :: Settings UI, defect, P4)

ARM
Windows CE
defect

Tracking

()

RESOLVED FIXED

People

(Reporter: zpao, Assigned: zpao)

Details

(Whiteboard: [nv])

Attachments

(1 file)

On WINCE we're hitting these dialogs more often because the delay hasn't changed but the devices are slower, so we just need to set the delay to be higher
Priority: -- → P4
Summary: Need to make slow script dialog delay higher → Need to make slow script dialog delay higher On WinCE
Whiteboard: [nv]
Attached patch Patch v0.1Splinter Review
Since this is Firefox specific, I just set it in firefox.js, overriding the
pref in libpref/src/init/all.js. Easy enough to move but might affect other
products (though I think mobile overrides this pref on their own).
Assignee: nobody → paul
Status: NEW → ASSIGNED
Attachment #388746 - Flags: review?(vladimir)
pushed http://hg.mozilla.org/mozilla-central/rev/7d70d1bf80a2
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Just out of curiousity, what did we change this to? maps.google.com is throwing up a slow script warning using 20091105 and it basically temporarily paralyzed the browser.
(In reply to comment #3)
> Just out of curiousity, what did we change this to? maps.google.com is throwing
> up a slow script warning using 20091105 and it basically temporarily paralyzed
> the browser.

Looks like 20s. Maybe it should be even higher?
Perhaps - During testing I will see how often I see this and report back.

(In reply to comment #4)
> (In reply to comment #3)
> > Just out of curiousity, what did we change this to? maps.google.com is throwing
> > up a slow script warning using 20091105 and it basically temporarily paralyzed
> > the browser.
> 
> Looks like 20s. Maybe it should be even higher?
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: