Bug 64320

Summary: Java Bridge leaks local references
Product: WebKit Reporter: Steve Block <steveblock>
Component: WebCore Misc.Assignee: Steve Block <steveblock>
Status: RESOLVED FIXED    
Severity: Normal CC: ap, steveblock, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch
none
Patch
none
Patch none

Steve Block
Reported 2011-07-11 15:32:20 PDT
In particular, see the JavaField constructors and convertNPVariantToJavaValue()
Attachments
Patch (4.68 KB, patch)
2011-07-11 15:41 PDT, Steve Block 		no flags
DetailsFormatted DiffDiff
Patch (4.72 KB, patch)
2011-07-12 01:36 PDT, Steve Block 		no flags
DetailsFormatted DiffDiff
Patch (3.10 KB, patch)
2011-08-09 07:42 PDT, Steve Block 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Steve Block
Comment 1 2011-07-11 15:41:16 PDT
Created attachment 100371 [details] Patch
WebKit Review Bot
Comment 2 2011-07-11 15:52:57 PDT
Comment on attachment 100371 [details] Patch Attachment 100371 [details] did not pass mac-ews (mac): Output: http://queues.webkit.org/results/9013481
Steve Block
Comment 3 2011-07-12 01:36:51 PDT
Created attachment 100449 [details] Patch
Steve Block
Comment 4 2011-08-09 07:42:46 PDT
Created attachment 103361 [details] Patch
Steve Block
Comment 5 2011-08-09 07:43:39 PDT
The relevant code in convertNPVariantToJavaValue() was moved to javaValueToJvalue() and then removed in http://trac.webkit.org/changeset/92659
WebKit Review Bot
Comment 6 2011-08-09 10:00:18 PDT
Comment on attachment 103361 [details] Patch Clearing flags on attachment: 103361 Committed r92689: <http://trac.webkit.org/changeset/92689>
WebKit Review Bot
Comment 7 2011-08-09 10:00:24 PDT
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.