[webkit-reviews] review granted: [Bug 13029] Permit NPAPI plug-ins to see HTTP response headers : [Attachment 13710] Proposed solution, take 4

[bugzilla-daemon at webkit.org]

Darin Adler <darin at apple.com> has granted Deneb Meketa <dmeketa at adobe.com>'s
request for review:
Bug 13029: Permit NPAPI plug-ins to see HTTP response headers
http://bugs.webkit.org/show_bug.cgi?id=13029

Attachment 13710: Proposed solution, take 4
http://bugs.webkit.org/attachment.cgi?id=13710&action=edit

------- Additional Comments from Darin Adler <darin at apple.com>
+	 char statusStr[10];
+	 sprintf(statusStr, "%d", [httpResponse statusCode]);

To avoid getting the security "cops" on our backs, lets use snprintf here
instead of plain old sprintf.

+    if (obj && (browser->version >= NPVERS_HAS_RESPONSE_HEADERS)) {
+	 notifyStream(obj, stream->url, stream->headers);
+    }

No braces around a single-line if statement.

I'm going to say r=me, but I'd like to see a version with snprintf.