Currently the event handlers are using V8Proxy::context(m_frame), which leads to the wrong context for event listeners from isolated worlds. Basically all clients of that function must die, but we're going to take it slow. Patch forthcoming.
Created attachment 33248 [details] patch
Comment on attachment 33248 [details] patch Actually, there might be some interaction with workers that I need to think though. Clearing the review flag for now.
Created attachment 33271 [details] Slightly better patch
Created attachment 33273 [details] patch with less trailing whitespace
Comment on attachment 33273 [details] patch with less trailing whitespace ok. I wonder if OwnHandler would rather live in v8 namespace? It seems like a useful primitive and not very WebCore-specific.
(In reply to comment #5) > ok. I wonder if OwnHandler would rather live in v8 namespace? It seems like a > useful primitive and not very WebCore-specific. I asked Ager about that, but he didn't seem too receptive. He hadn't seen use for the concept. We can demonstrate the use cases in the bindings and then ask again.
Sending LayoutTests/ChangeLog Adding LayoutTests/http/tests/security/isolatedWorld/click-event-expected.txt Adding LayoutTests/http/tests/security/isolatedWorld/click-event.html Sending LayoutTests/platform/gtk/Skipped Sending LayoutTests/platform/mac/Skipped Sending LayoutTests/platform/qt/Skipped Sending LayoutTests/platform/win/Skipped Sending WebCore/ChangeLog Sending WebCore/WebCore.gypi Adding WebCore/bindings/v8/OwnHandle.h Sending WebCore/bindings/v8/V8AbstractEventListener.cpp Sending WebCore/bindings/v8/V8AbstractEventListener.h Transmitting file data ............ Committed revision 46529. http://trac.webkit.org/changeset/46529