107308 2013-01-18 11:16:34 -0800 WebKitDownload notify::progress signal is not emitted when it should 2017-03-11 10:46:25 -0800 1 1 1 Unclassified WebKit WebKitGTK 528+ (Nightly build) Unspecified Linux UNCONFIRMED http://bugs.sugarlabs.org/ticket/4385 P2 Normal --- 0 humitos webkit-unassigned bugs-noreply humitos oldest_to_newest 810865 0 183512 humitos 2013-01-18 11:16:34 -0800 Created attachment 183512 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 183512 2013-01-18 11:16:34 -0800 2013-01-18 11:16:34 -0800 Test Case download_twice_same_file_with_webkit.py text/x-python 2064 humitos ZnJvbSBnaS5yZXBvc2l0b3J5IGltcG9ydCBHdGsKZnJvbSBnaS5yZXBvc2l0b3J5IGltcG9ydCBX ZWJLaXQKCgpkZWYgX19kZXN0cm95X2NiKHdpZGdldCwgZGF0YT1Ob25lKToKICAgIGdsb2JhbCBk b3dubG9hZAogICAgaWYgZG93bmxvYWQgaXMgbm90IE5vbmU6CiAgICAgICAgZG93bmxvYWQuY2Fu Y2VsKCkKICAgIEd0ay5tYWluX3F1aXQoKQoKCmRlZiBfX3Byb2dyZXNzX2NoYW5nZV9jYihkb3du bG9hZCwgc29tZXRoaW5nKToKICAgIHByb2dyZXNzID0gZG93bmxvYWQuZ2V0X3Byb2dyZXNzKCkK ICAgIHByaW50IHByb2dyZXNzCgoKZGVmIF9fc3RhdGVfY2hhbmdlX2NiKGRvd25sb2FkLCBncGFy YW1zcGVjKToKICAgIHN0YXRlID0gZG93bmxvYWQuZ2V0X3N0YXR1cygpCiAgICBpZiBzdGF0ZSA9 PSBXZWJLaXQuRG93bmxvYWRTdGF0dXMuU1RBUlRFRDoKICAgICAgICBwcmludCAnU1RBUlRFRCcK ICAgIGVsaWYgc3RhdGUgPT0gV2ViS2l0LkRvd25sb2FkU3RhdHVzLkZJTklTSEVEOgogICAgICAg IHByaW50ICdGSU5JU0hFRCcKICAgIGVsaWYgc3RhdGUgPT0gV2ViS2l0LkRvd25sb2FkU3RhdHVz LkNBTkNFTExFRDoKICAgICAgICBwcmludCAnQ0FOQ0VMTEVEJwogICAgZWxzZToKICAgICAgICBw cmludCAnT3RoZXIgc3RhdGU6ICVzJyAlIHN0YXRlCgoKZGVmIF9fZXJyb3JfY2IoZG93bmxvYWQs IGVycl9jb2RlLCBlcnJfZGV0YWlsLCByZWFzb24pOgogICAgcHJpbnQgJ0Vycm9yIGRvd25sb2Fk aW5nIFVSSSBjb2RlICVzLCBkZXRhaWwgJXM6ICVzJyAlIChlcnJfY29kZSwgZXJyX2RldGFpbCwg cmVhc29uKQoKCiMgV2UgbmVlZCB0byBrZWVwIHRoZSBkb3dubG9hZCBpbiBtZW1vcnkKZG93bmxv YWQgPSBOb25lCmRlZiBfX2NsaWNrZWRfc3RhcnRfYnV0dG9uKHdpZGdldCk6CiAgICBwcmludCAn QnV0dG9uIGNsaWNrZWQnCgogICAgdXJsID0gJ2h0dHA6Ly9kb3dubG9hZC5sYXB0b3Aub3JnL3hv LTEuNS9vcy9jYW5kaWRhdGUvMTIuMS4wLTIxLzIxMDIxbzEuemQnCiAgICBuciA9IFdlYktpdC5O ZXR3b3JrUmVxdWVzdCgpCiAgICBuci5zZXRfdXJpKHVybCkKICAgIGdsb2JhbCBkb3dubG9hZAog ICAgZG93bmxvYWQgPSBXZWJLaXQuRG93bmxvYWQobmV0d29ya19yZXF1ZXN0PW5yKQoKICAgIGRv d25sb2FkLmNvbm5lY3QoJ25vdGlmeTo6cHJvZ3Jlc3MnLCBfX3Byb2dyZXNzX2NoYW5nZV9jYikK ICAgIGRvd25sb2FkLmNvbm5lY3QoJ25vdGlmeTo6c3RhdHVzJywgX19zdGF0ZV9jaGFuZ2VfY2Ip CiAgICBkb3dubG9hZC5jb25uZWN0KCdlcnJvcicsIF9fZXJyb3JfY2IpCgogICAgcHJpbnQgZG93 bmxvYWQuZ2V0X3VyaSgpCiAgICBkb3dubG9hZC5zZXRfZGVzdGluYXRpb25fdXJpKCdmaWxlOi8v L3RtcC8yMTAyMW8xLnpkJykKICAgIGRvd25sb2FkLnN0YXJ0KCkKCgpkZWYgX19jbGlja2VkX2Nh bmNlbF9idXR0b24od2lkZ2V0KToKICAgIGdsb2JhbCBkb3dubG9hZAogICAgaWYgZG93bmxvYWQg aXMgbm90IE5vbmU6CiAgICAgICAgZG93bmxvYWQuY2FuY2VsKCkKICAgICAgICBwcmludCAnRG93 bmxvYWQgQ0FOQ0VMTEVEIScKCndpbmRvdyA9IEd0ay5XaW5kb3coKQp3aW5kb3cuc2V0X3RpdGxl KCdEb3dubG9hZCBhIGZpbGUgd2l0aCBXZWJLaXQgZXhhbXBsZScpCndpbmRvdy5zZXRfZGVmYXVs dF9zaXplKDMwMCwgNjApCndpbmRvdy5jb25uZWN0KCdkZXN0cm95JywgX19kZXN0cm95X2NiKQoK c3RhcnRfYnV0dG9uID0gR3RrLkJ1dHRvbignRG93bmxvYWQgZmlsZSEnKQpzdGFydF9idXR0b24u Y29ubmVjdCgnY2xpY2tlZCcsIF9fY2xpY2tlZF9zdGFydF9idXR0b24pCgpjYW5jZWxfYnV0dG9u ID0gR3RrLkJ1dHRvbignQ2FuY2VsIGRvd25sb2FkIScpCmNhbmNlbF9idXR0b24uY29ubmVjdCgn Y2xpY2tlZCcsIF9fY2xpY2tlZF9jYW5jZWxfYnV0dG9uKQoKdmJveCA9IEd0ay5Cb3gob3JpZW50 YXRpb249R3RrLk9yaWVudGF0aW9uLlZFUlRJQ0FMKQp2Ym94LmFkZChzdGFydF9idXR0b24pCnZi b3guYWRkKGNhbmNlbF9idXR0b24pCgp3aW5kb3cuYWRkKHZib3gpCndpbmRvdy5zaG93X2FsbCgp Ckd0ay5tYWluKCkK