[webkit-reviews] review granted: [Bug 85575] [BlackBerry] Autofill backing store implementation upstream : [Attachment 140421] Patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Sun May 6 16:30:16 PDT 2012
Rob Buis <rwlbuis at gmail.com> has granted Jonathan Dong
<jonathan.dong at torchmobile.com.cn>'s request for review:
Bug 85575: [BlackBerry] Autofill backing store implementation upstream
https://bugs.webkit.org/show_bug.cgi?id=85575
Attachment 140421: Patch
https://bugs.webkit.org/attachment.cgi?id=140421&action=review
------- Additional Comments from Rob Buis <rwlbuis at gmail.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=140421&action=review
Looks good. I don't really see the need to move away from instance() concept,
but ok.
> Source/WebCore/platform/network/blackberry/AutofillBackingStore.cpp:185
> +
One newline too much.
More information about the webkit-reviews
mailing list