[webkit-reviews] review granted: [Bug 219820] Remove WKDownloadDelegate.didWriteData : [Attachment 416086] Patch

[bugzilla-daemon at webkit.org]

James Savage <james.savage at apple.com> has granted Alex Christensen
<achristensen at apple.com>'s request for review:
Bug 219820: Remove WKDownloadDelegate.didWriteData
https://bugs.webkit.org/show_bug.cgi?id=219820

Attachment 416086: Patch

https://bugs.webkit.org/attachment.cgi?id=416086&action=review




--- Comment #2 from James Savage <james.savage at apple.com> ---
Comment on attachment 416086
  --> https://bugs.webkit.org/attachment.cgi?id=416086
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=416086&action=review

> Source/WebKit/UIProcess/API/Cocoa/WKDownload.mm:221
> +	   downloadProgress = [NSProgress progressWithTotalUnitCount:-1];

Presumably -1 is a sentinel for undetermined/indeterminate progress? Why was
this 100 originally?