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]
Comment on attachment 33760 [details]
> 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:
So r+ if you make this change to make it equivalent to before.
Created attachment 33761 [details]
Assigning to myself while we remove the remaining client.
Comment on attachment 33761 [details]
Clearing review flag on attachment: 33761
Transmitting file data ...
Committed revision 46840.
All reviewed patches have been landed. Closing bug.