[webkit-reviews] review requested: [Bug 50080] [Qt] Support a QNetworkAccessManager affined to a different thread. : [Attachment 76070] Patch v2
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Thu Dec 9 08:38:11 PST 2010
Jocelyn Turcotte <jocelyn.turcotte at nokia.com> has asked for review:
Bug 50080: [Qt] Support a QNetworkAccessManager affined to a different thread.
https://bugs.webkit.org/show_bug.cgi?id=50080
Attachment 76070: Patch v2
https://bugs.webkit.org/attachment.cgi?id=76070&action=review
------- Additional Comments from Jocelyn Turcotte <jocelyn.turcotte at nokia.com>
This version:
- Renames NAMManipulator to QtNAMThreadSafeProxy and ReplyManipulator to
QtNetworkReplyThreadSafeProxy
- Adds a comment for the dummy variables, though if you have a better idea I
would prefer since it generates a warning for unused variable on GCC.
- Fixes style issues
More information about the webkit-reviews
mailing list