[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