D19857: [DownloadJob] Report total size only if known

Kai Uwe Broulik noreply at phabricator.kde.org
Mon Mar 18 15:24:52 GMT 2019


broulik created this revision.
broulik added a reviewer: Plasma.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
broulik requested review of this revision.

REVISION SUMMARY
  In case the download size isn't known, the browser reports `-1`

TEST PLAN
  It now shows "xyz MiB of 0 B" so we don't handle that in notifications but it's better than -1 B
  Or should I explicitly `setTotalAmount(0)` in this case?

REPOSITORY
  R856 Plasma Browser Integration

REVISION DETAIL
  https://phabricator.kde.org/D19857

AFFECTED FILES
  host/downloadjob.cpp

To: broulik, #plasma
Cc: plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190318/6cefea6e/attachment.html>


More information about the Plasma-devel mailing list