[webkit-reviews] review denied: [Bug 83349] [BlackBerry] We shouldn't set empty Content-Length & Content-Type to request's headers. : [Attachment 135979] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Apr 11 22:52:42 PDT 2012


George Staikos <staikos at kde.org> has denied Jason
<jason.liu at torchmobile.com.cn>'s request for review:
Bug 83349: [BlackBerry] We shouldn't set empty Content-Length & Content-Type to
 request's headers.
https://bugs.webkit.org/show_bug.cgi?id=83349

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

------- Additional Comments from George Staikos <staikos at kde.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=135979&action=review


I agree with Leo's comments.

> Source/WebCore/platform/network/blackberry/NetworkJob.cpp:587
>      if ((method != "GET") && (method != "HEAD")) {

There are extra parentheses here


More information about the webkit-reviews mailing list