[webkit-reviews] review requested: [Bug 28665] Inspector: Throws an Error on "null" : [Attachment 38447] [PATCH] One Possible Solution
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Sun Aug 23 00:04:03 PDT 2009
Joseph Pecoraro <joepeck at webkit.org> has asked for review:
Bug 28665: Inspector: Throws an Error on "null"
https://bugs.webkit.org/show_bug.cgi?id=28665
Attachment 38447: [PATCH] One Possible Solution
https://bugs.webkit.org/attachment.cgi?id=38447&action=review
------- Additional Comments from Joseph Pecoraro <joepeck at webkit.org>
No other falsey value causes wrapObject() to return a null value (NaN,
undefined, 0, ''). This solution tackles null before calling wrapObject.
Another solution could be checking the result of wrapObject (wrapper). And yet
another solution could handle null differently inside
InspectorController.wrapObject(). This approach seemed simplest.
Pavel, would there be a more appropriate solution?
More information about the webkit-reviews
mailing list