[webkit-reviews] review denied: [Bug 101568] [BlackBerry] need to call closePopup at setValueAndClosePopup : [Attachment 172981] Patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Thu Nov 8 07:29:51 PST 2012
Rob Buis <rwlbuis at gmail.com> has denied Chris.Guan
<chris.guan at torchmobile.com.cn>'s request for review:
Bug 101568: [BlackBerry] need to call closePopup at setValueAndClosePopup
https://bugs.webkit.org/show_bug.cgi?id=101568
Attachment 172981: Patch
https://bugs.webkit.org/attachment.cgi?id=172981&action=review
------- Additional Comments from Rob Buis <rwlbuis at gmail.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=172981&action=review
Code looks good but ChangeLog should be better.
> Source/WebKit/blackberry/ChangeLog:10
> + Internally reviewed by Charles Wei.
Please also state PR and internal reviewer.
I think the description could be better, since in the PR it states there is a
freeze/crash which I can't see in this description.
More information about the webkit-reviews
mailing list