Upstream changes to V8 bindings for supporting nested workers.
Created attachment 29777 [details] Proposed Patch
Comment on attachment 29777 [details] Proposed Patch Style looks good! :) > + if (result.IsEmpty()) { > + v8::Local<v8::Object> object = toV8(type, type, impl); > + if (!object.IsEmpty()) > + static_cast<Worker*>(impl)->ref(); > + result = v8::Persistent<v8::Object>::New(object); > + V8Proxy::SetJSWrapperForDOMObject(impl, result); > + } > + return result; Would it be a bit more clear to inverse this: if (!result.IsEmpty()) return result; ... > + return PassRefPtr<EventListener>(); Can just return 0;
Created attachment 30042 [details] Proposed Patch All fixed. Thanks.
Comment on attachment 30042 [details] Proposed Patch Land it! :)
Assign to levin for landing.
http://trac.webkit.org/changeset/43328