V8 interceptors need a truly empty value, rather than undefined. Sad.
Created attachment 27668 [details] V8HTMLPlugInCustom returns undefined, which fools interceptors, v1 WebCore/ChangeLog | 13 +++++++++++++ .../v8/custom/V8HTMLPlugInElementCustom.cpp | 16 ++++++++-------- 2 files changed, 21 insertions(+), 8 deletions(-)
Comment on attachment 27668 [details] V8HTMLPlugInCustom returns undefined, which fools interceptors, v1 We're talking about something prettier, and involving more pink ponies and less scary undocumented subtitles in #webkit. No biscuit for this patch!
Created attachment 27674 [details] V8HTMLPlugInCustom returns undefined, which fools interceptors, v2 WebCore/ChangeLog | 13 +++++++++++++ .../v8/custom/V8HTMLPlugInElementCustom.cpp | 16 ++++++++-------- 2 files changed, 21 insertions(+), 8 deletions(-)
Comment on attachment 27674 [details] V8HTMLPlugInCustom returns undefined, which fools interceptors, v2 LGTM.
Landed as http://trac.webkit.org/changeset/41017.