[webkit-reviews] review denied: [Bug 15055] Divide by 0 in ProgressTracker::incrementProgress : [Attachment 16204] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sat Sep 29 20:22:08 PDT 2007


Maciej Stachowiak <mjs at apple.com> has denied Marvin Decker
<marv.decker at gmail.com>'s request for review:
Bug 15055: Divide by 0 in ProgressTracker::incrementProgress
http://bugs.webkit.org/show_bug.cgi?id=15055

Attachment 16204: Patch
http://bugs.webkit.org/attachment.cgi?id=16204&action=edit

------- Additional Comments from Maciej Stachowiak <mjs at apple.com>
The fix looks good to me (I can understand why the remainingBytes estimate
could be wrong, since resources might not specify their length). But do you
have a test case that can trigger this? I believe it could be done as an http
test. r- to consider providing a test case, but if that is not practical just
reflag for review and I'll approve it.



More information about the webkit-reviews mailing list