[webkit-reviews] review granted: [Bug 99784] [chromium] Implement full-featured image cache : [Attachment 176325] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Nov 27 12:35:54 PST 2012


Stephen White <senorblanco at chromium.org> has granted Hin-Chung Lam
<hclam at google.com>'s request for review:
Bug 99784: [chromium] Implement full-featured image cache
https://bugs.webkit.org/show_bug.cgi?id=99784

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

------- Additional Comments from Stephen White <senorblanco at chromium.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=176325&action=review


OK.  r=me

> Source/WebCore/platform/graphics/chromium/ImageDecodingStore.cpp:138
> +    // entries quickly, it also owns the cached images.

Grammar nit:  comma splice.  (Both sides of the comma are complete sentences. 
Fix is to use a semicolon, or split into two sentences.)  Much too small to r-,
though.


More information about the webkit-reviews mailing list