[webkit-reviews] review denied: [Bug 85812] [BlackBerry] Delete previous buffer when new over-scroll image path is set : [Attachment 140564] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon May 7 12:22:32 PDT 2012


Rob Buis <rwlbuis at gmail.com> has denied  review:
Bug 85812: [BlackBerry] Delete previous buffer when new over-scroll image path
is set
https://bugs.webkit.org/show_bug.cgi?id=85812

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

------- Additional Comments from Rob Buis <rwlbuis at gmail.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=140564&action=review


Looks good, but please fix the period.

> Source/WebKit/blackberry/Api/BackingStore.cpp:1221
> +    // For now we fallback to solid color if sizes don't match, later we can
implement tiling

It is preferred to end these sentences with a period.


More information about the webkit-reviews mailing list