RESOLVED FIXED 27826
[V8] Remove bug-bait V8Proxy::context() 
https://bugs.webkit.org/show_bug.cgi?id=27826
Summary [V8] Remove bug-bait V8Proxy::context()
Adam Barth
Reported Thursday, July 30, 2009 7:37:35 AM UTC
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).
Attachments
Patch v1 (2.47 KB, patch)
2009-07-29 23:40 PDT, Adam Barth 		no flags
DetailsFormatted DiffDiff
Patch v1 (2.57 KB, patch)
2009-07-29 23:52 PDT, Adam Barth 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Adam Barth
Comment 1 Thursday, July 30, 2009 7:40:48 AM UTC
Created attachment 33760 [details] Patch v1
David Levin
Comment 2 Thursday, July 30, 2009 7:50:08 AM UTC
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.
Adam Barth
Comment 3 Thursday, July 30, 2009 7:52:47 AM UTC
Created attachment 33761 [details] Patch v1
Adam Barth
Comment 4 Thursday, July 30, 2009 9:44:57 PM UTC
Assigning to myself while we remove the remaining client.
Adam Barth
Comment 5 Thursday, August 6, 2009 9:27:11 AM UTC
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
Adam Barth
Comment 6 Thursday, August 6, 2009 9:27:17 AM UTC
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.