[webkit-reviews] review denied: [Bug 14437] CRASH: RTÉ video crashes Safari : [Attachment 15371] Tweaked initial patch

[bugzilla-daemon at webkit.org]

Darin Adler <darin at apple.com> has denied Anders Carlsson <andersca at apple.com>'s
request for review:
Bug 14437: CRASH: RTÉ video crashes Safari
http://bugs.webkit.org/show_bug.cgi?id=14437

Attachment 15371: Tweaked initial patch
http://bugs.webkit.org/attachment.cgi?id=15371&action=edit

------- Additional Comments from Darin Adler <darin at apple.com>
I just talked this over with Anders. It's clear that in cases where the JSValue
is an object, we can't return the object, because we can't run its toString
method in the right context -- the context is gone because the frame is
destroyed.

But in the simple cases where the JSValue is not an object, it seems there's no
reason we can't return the value. So I suggest we make a version of toString
that works on non-object JSValue, and use that if m_frame is nil. That way the
common cases of plain old strings and such will work fine.

That seems slightly better than always returning nil. review- until we at least
consider that option.