[webkit-reviews] review denied: [Bug 24324] Make DOMTimer clamp adjustable via settings : [Attachment 28225] patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Mar 5 14:08:11 PST 2009


Dave Hyatt <hyatt at apple.com> has denied Mike Belshe <mike at belshe.com>'s request
for review:
Bug 24324: Make DOMTimer clamp adjustable via settings
https://bugs.webkit.org/show_bug.cgi?id=24324

Attachment 28225: patch
https://bugs.webkit.org/attachment.cgi?id=28225&action=review

------- Additional Comments from Dave Hyatt <hyatt at apple.com>
I think you need a comment explaining why a #define is being used.  It's ok to
mention that Chromium has no clamp in the comment.  We need to make sure people
know why this oddity is here, since otherwise they might just yank the code.

Also need a ChangeLog comment.


More information about the webkit-reviews mailing list