[Webkit-unassigned] [Bug 32982] plugins/get-url-with-iframe-target.html fails on SnowLeopard (64-bit)

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Dec 28 00:03:58 PST 2009


https://bugs.webkit.org/show_bug.cgi?id=32982





--- Comment #2 from Eric Seidel <eric at webkit.org>  2009-12-28 00:03:58 PST ---
(From update of attachment 45548)
argument name seems superflous:
 73     static NPReason reasonForError(NSError *error);

Can't tell if this spacing is right or wrong, seems like spaces beofre * are
wrong here:
 77     static PassRefPtr<PluginRequest> create(uint32_t requestID,
NSURLRequest *request, NSString *frameName, bool allowPopups)

And here:
 88     PluginRequest(uint32_t requestID, NSURLRequest *request, NSString
*frameName, bool allowPopups)

Or shoudl they be " *" because this is a .mm file?

This looks sane.  Looks like a good change.  I still feel like you probably
want someone who's been in this code before to review it though.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list