The default bug view has changed. See this FAQ.

Revert the update check frequency increase

RESOLVED FIXED in Firefox 12

Status

()

Firefox
General
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: Ehsan, Assigned: Ehsan)

Tracking

Trunk
Firefox 17
x86
Mac OS X
Points:
---

Firefox Tracking Flags

(firefox12 fixed, firefox13 fixed, firefox14+ fixed, firefox15+ fixed, firefox16+ fixed)

Details

(Whiteboard: [qa-])

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
We need to revert the code bug 707020 adds after the testing of bug 481815 is finished.
Although the testing is not finished, we can probably revert this.
(Assignee)

Comment 2

5 years ago
Created attachment 593950 [details] [diff] [review]
Patch (v1)

[Approval Request Comment]
Regression caused by (bug #): No regression, it's just something that we wanted on nightly but not on Aurora.
User impact if declined: Updates checked for more than necessary.
Testing completed (on m-c, etc.): This reverts us to the situation we were in before bug 702020.
Risk to taking this patch (and alternatives if risky): None.
String changes made by this patch: None.
Attachment #593950 - Flags: review?(netzen)
Attachment #593950 - Flags: approval-mozilla-aurora?
Comment on attachment 593950 [details] [diff] [review]
Patch (v1)

Review of attachment 593950 [details] [diff] [review]:
-----------------------------------------------------------------

This reverts it back to the correct old values for Aurora. Looks good.
Attachment #593950 - Flags: review?(netzen) → review+
Sorry I should say:
This reverts it back to the correct old values for Nightly on the Aurora branch. Looks good.
(Assignee)

Comment 5

5 years ago
Alex, can we take this on Aurora sooner rather than later please?  Thanks!  :-)

Comment 6

5 years ago
Comment on attachment 593950 [details] [diff] [review]
Patch (v1)

[Triage Comment]
Minor interval change - low risk. Approved for Aurora.
Attachment #593950 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
(Assignee)

Comment 7

5 years ago
https://hg.mozilla.org/releases/mozilla-aurora/rev/111665833e7d
status-firefox12: --- → fixed
Ehsan: can we close this bug now?
(Assignee)

Comment 9

5 years ago
(In reply to :Gavin Sharp (use gavin@gavinsharp.com for email) from comment #8)
> Ehsan: can we close this bug now?

No, I've missed this on Aurora 13, which means that this will be on Beta13 tomorrow.  We need to take the patch on Aurora and Beta again tomorrow, but I'd like to hold off on landing this on central until the work in bug 307181 is finished.
(Assignee)

Comment 10

5 years ago
Comment on attachment 593950 [details] [diff] [review]
Patch (v1)

[Approval Request Comment]
Regression caused by (bug #): No regression, it's just something that we wanted on nightly but not on Aurora.
User impact if declined: Updates checked for more than necessary.
Testing completed (on m-c, etc.): This reverts us to the situation we were in before bug 702020.
Risk to taking this patch (and alternatives if risky): None.
String changes made by this patch: None.
Attachment #593950 - Flags: approval-mozilla-beta?
Attachment #593950 - Flags: approval-mozilla-aurora?
Attachment #593950 - Flags: approval-mozilla-aurora+
Comment on attachment 593950 [details] [diff] [review]
Patch (v1)

[Triage Comment]
Approving for Aurora 13 since we may still be able to land this before tomorrow's merge. Feel free to carry over to Beta 13 after the uplift.
Attachment #593950 - Flags: approval-mozilla-beta?
Attachment #593950 - Flags: approval-mozilla-aurora?
Attachment #593950 - Flags: approval-mozilla-aurora+
(Assignee)

Comment 12

5 years ago
(In reply to Alex Keybl [:akeybl] from comment #11)
> Comment on attachment 593950 [details] [diff] [review]
> Patch (v1)
> 
> [Triage Comment]
> Approving for Aurora 13 since we may still be able to land this before
> tomorrow's merge. Feel free to carry over to Beta 13 after the uplift.

Sorry if I was vague.  I do not want this on central, which is why I asked for approval on Aurora.  I'll land this on Aurora right now, and will re-request Aurora approval once again after the merge.
(Assignee)

Comment 13

5 years ago
https://hg.mozilla.org/releases/mozilla-aurora/rev/2e0d201168b3
status-firefox13: --- → fixed
status-firefox14: --- → affected
tracking-firefox14: --- → ?
(Assignee)

Comment 14

5 years ago
Comment on attachment 593950 [details] [diff] [review]
Patch (v1)

Requesting approval for the new Aurora.
Attachment #593950 - Flags: approval-mozilla-aurora+ → approval-mozilla-aurora?
Comment on attachment 593950 [details] [diff] [review]
Patch (v1)

[Triage comment]
approved for the new aurora.
Attachment #593950 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
(Assignee)

Comment 16

5 years ago
http://hg.mozilla.org/releases/mozilla-aurora/rev/9a9581a8a539
status-firefox14: affected → fixed
status-firefox15: --- → affected
tracking-firefox15: --- → ?

Updated

5 years ago
tracking-firefox14: ? → +
tracking-firefox15: ? → +
Do we want to do anything here before June 5th?
(Assignee)

Comment 18

5 years ago
We need to land this on Aurora after the uplift.  On central, I'd still like to keep this maybe for a couple of more weeks.
(Assignee)

Comment 19

5 years ago
Ported to the new aurora as well: http://hg.mozilla.org/releases/mozilla-aurora/rev/898a1bd48058
status-firefox15: affected → fixed
(Assignee)

Comment 20

5 years ago
Landed for Firefox 16 on Aurora:

http://hg.mozilla.org/releases/mozilla-aurora/rev/8386a8f7289a

Also, time to stop this on Nightly as well:

https://hg.mozilla.org/integration/mozilla-inbound/rev/b4028e63e58b
status-firefox16: --- → fixed
tracking-firefox16: --- → +
Target Milestone: --- → Firefox 17
https://hg.mozilla.org/mozilla-central/rev/b4028e63e58b
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED

Updated

5 years ago
Whiteboard: [qa-]
You need to log in before you can comment on or make changes to this bug.