Bug 26175 – [GTK] Download progress notification should be throttled, for the benefit of download managers

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKitGTK

[Gustavo Noronha (kov)]

This means less notifications for changes on the "progress" property. Someone who really wants to get all notifications can still connect to the "current-size" property.

[Gustavo Noronha (kov)]

Created attachment 30932 [details]
Throttle download speed.

 WebKit/gtk/ChangeLog                 |   16 ++++++++++++++++
 WebKit/gtk/webkit/webkitdownload.cpp |   26 ++++++++++++++++++++++----
 2 files changed, 38 insertions(+), 4 deletions(-)

[Xan Lopez]

Comment on attachment 30932 [details]
Throttle download speed.

Looks good to me. I wonder if current-size should be throttled too, in case it reports changes more often than needed (and uses too much CPU in doing so).

[Gustavo Noronha (kov)]

Landed as r44485.

About throttling current-size, my original patch did that, but the high CPU usage we experienced was not caused by the signal emissions (notice we already have signal emissions, and many function calls on that code path), but by the download manager updating the UI based on the emissions. I decided to not propose throttling the emissions of current-size and total-size, because then those properties will work for whoever wants to follow the progress more closely, while download managers can keep using the convenience property without having to do throttling themselves. Thoughts on that?