RESOLVED FIXED94766
[V8] SharedPersistent isn't needed---let's delete it 
https://bugs.webkit.org/show_bug.cgi?id=94766
Summary [V8] SharedPersistent isn't needed---let's delete it
Adam Barth
Reported 2012-08-22 19:04:52 PDT
[V8] SharedPersistent isn't needed---let's delete it
Attachments
builds but not tested (13.91 KB, patch)
2012-08-22 19:06 PDT, Adam Barth 		abarth: review-
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Adam Barth
Comment 1 2012-08-22 19:06:51 PDT
Created attachment 160062 [details] builds but not tested
Kentaro Hara
Comment 2 2012-08-22 19:11:45 PDT
Comment on attachment 160062 [details] builds but not tested View in context: https://bugs.webkit.org/attachment.cgi?id=160062&action=review > Source/WebCore/ChangeLog:9 > + The New/Dispose mechanism already does that for us. We can use use Nit: use use => use
Adam Barth
Comment 3 2012-08-22 20:01:04 PDT
Comment on attachment 160062 [details] builds but not tested I makes a crash.
Adam Barth
Comment 4 2012-08-22 23:41:08 PDT
I'm not exactly sure what's causing these crashes. Note: The patch has a memory leak because we need to add an "adopt" method to OwnHandle so that we don't call Persistent::New an extra time for the handle returned by createNewContext.
Anders Carlsson
Comment 5 2013-05-02 11:25:50 PDT
This has been fixed (by removing V8).
Note You need to log in before you can comment on or make changes to this bug.