385733 - Remove "rate change limit" check from DownloadProgressListener

Status: RESOLVED FIXED

(Toolkit :: Downloads API, defect)

Description

[Ed Lee :Mardak]

Attachment: v1

rateChangeLimit is 0, so it'll end up changing every time anyway... so kill it! 

(needs bug 237693 checked in first)

Comment 1

[Shawn Wilsher :sdwilsh]

Is this already included in your other patch?

Comment 2

[Ed Lee :Mardak]

The attached patch is based on changes to bug 237693, so it isn't included in the patch for that bug.

Comment 3

[Shawn Wilsher :sdwilsh]

ah, so it also cannot land until then, correct?

Also, please remove the check for the update interval since the back-end already does this.

Comment 4

[Ed Lee :Mardak]

Attachment: v2

(In reply to comment #3)
> ah, so it also cannot land until then, correct?
Correct. (Updated to diff off the other bug's latest patch v6)

> Also, please remove the check for the update interval since the back-end
> already does this.
Done.

Comment 5

[Shawn Wilsher :sdwilsh]

Patch doesn't apply cleanly - please submit a new one.

Comment 6

[Ed Lee :Mardak]

Attachment: for checkin

Comment 7

[Shawn Wilsher :sdwilsh]

Checking in toolkit/mozapps/downloads/content/DownloadProgressListener.js;
new revision: 1.14; previous revision: 1.13