[Webkit-unassigned] [Bug 107308] New: WebKitDownload notify::progress signal is not emitted when it should

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=107308

           Summary: WebKitDownload notify::progress signal is not emitted
                    when it should
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Unspecified
               URL: http://bugs.sugarlabs.org/ticket/4385
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: Normal
          Priority: P2
         Component: WebKit Gtk
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: humitos at gmail.com


Created an attachment (id=183512)
 --> (https://bugs.webkit.org/attachment.cgi?id=183512&action=review)
Test Case

Hi,

Working on Sugar's Browse Activity I found that "notify::progress" signal from WebKitDownload is not emitted when it should. The first time that you download a file using WebKitDownload this signal is emitted without any problem, but the second time that you download a file (it could be the same one) that signal is no emitted at all if the first download was successful. Instead, if the downloading progress was canceled at any time (let's say 20%) the "notify::progress" signal will be emitted when the progress property reaches the last value (in this case 20%).

I'm attaching a Test Case done in Python and I can confirm this behaviour using Epiphany that the first time that I click on a link to download a file the progress is showed and the second time it just says "Starting..." until the file is fully downloaded.

I went to the code and I saw this comment. Maybe it has something to do with this.

 - ./Source/WebKit/gtk/webkit/webkitdownload.cpp L325

    /**
     * WebKitDownload:progress:
     *
     * Determines the current progress of the download. Notice that,
     * although the progress changes are reported as soon as possible,
     * the emission of the notify signal for this property is
     * throttled, for the benefit of download managers. If you care
     * about every update, use WebKitDownload:current-size.
     *
     * Since: 1.1.2
     */

Thanks, and I hope it helps

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.