The zero-argument version of V8Proxy::context() is never the right way to get the context. We should remove this bug-bait function once we finish removing all the clients. (ScheduledAction is the only remaining client, see Bug 27703).
Created attachment 33760 [details] Patch v1
Comment on attachment 33760 [details] Patch v1 > Index: WebCore/bindings/v8/V8Proxy.cpp > + v8::Context::Scope contextScope(m_context); This is more than just a transform on this line. This line would be equivalent to what was there before: v8::Context::Scope contextScope(v8::Local<v8::Context>::New(m_context)); So r+ if you make this change to make it equivalent to before.
Created attachment 33761 [details] Patch v1
Assigning to myself while we remove the remaining client.
Comment on attachment 33761 [details] Patch v1 Clearing review flag on attachment: 33761 Sending WebCore/ChangeLog Sending WebCore/bindings/v8/V8Proxy.cpp Sending WebCore/bindings/v8/V8Proxy.h Transmitting file data ... Committed revision 46840. http://trac.webkit.org/changeset/46840
All reviewed patches have been landed. Closing bug.