Closed
Bug 525005
Opened 16 years ago
Closed 16 years ago
Stats widget polls every six seconds forever
Categories
(www.mozilla.org :: General, defect)
www.mozilla.org
General
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: morgamic, Assigned: davedash)
References
()
Details
Attachments
(2 files)
|
765 bytes,
patch
|
morgamic
:
review+
|
Details | Diff | Splinter Review |
|
666.79 KB,
image/png
|
Details |
http://www.mozilla.com/en-US/firefox/personal.html
Doesn't seem necessary to poll every six seconds until the end of time. Should put a limit on it between 2 and 3 (more than 1 in case the first check doesn't take).
| Reporter | ||
Updated•16 years ago
|
Assignee: nobody → dd
| Assignee | ||
Comment 1•16 years ago
|
||
The every six seconds was a typo on my part. It was meant to be every 60 seconds. I increased that time, and also have it only check once - polling is unnecessary for something that doesn't update that drastically.
Thanks
-d
Attachment #408912 -
Flags: review?(morgamic)
| Reporter | ||
Comment 2•16 years ago
|
||
Comment on attachment 408912 [details] [diff] [review]
v1
Looks good -- can you check this in?
Attachment #408912 -
Flags: review?(morgamic) → review+
| Reporter | ||
Comment 3•16 years ago
|
||
Also - thanks for doing this, Dave.
| Assignee | ||
Comment 4•16 years ago
|
||
------------------------------------------------------------------------
r55826 | ddash@mozilla.com | 2009-11-10 10:05:23 -0800 (Tue, 10 Nov 2009) | 2 li
[bug 525005] Fixed polling to stop after 1 retry, and retry only after 60s.
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Comment 5•16 years ago
|
||
Comment 6•16 years ago
|
||
Verified FIXED on http://www-trunk.stage.mozilla.com/en-US/firefox/personal.html
Status: RESOLVED → VERIFIED
Updated•13 years ago
|
Component: www.mozilla.org/firefox → www.mozilla.org
Updated•13 years ago
|
Component: www.mozilla.org → General
Product: Websites → www.mozilla.org
You need to log in
before you can comment on or make changes to this bug.
Description
•