[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