Bug 107458

Summary: [GTK] Memory leak in FullscreenVideoController::hideHud()
Product: WebKit Reporter: Sudarsana Nagineni (babu) <naginenis>
Component: WebKitGTKAssignee: Sudarsana Nagineni (babu) <naginenis>
Status: RESOLVED FIXED    
Severity: Normal CC: eric.carlson, feature-media-reviews, gustavo, pnormand, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 107398    
Attachments:
Description Flags
Patch none

Sudarsana Nagineni (babu)
Reported 2013-01-21 09:17:00 PST
The value returned by gdk_cursor_new() is not adopted in FullscreenVideoController::hideHud().
Attachments
Patch (1.40 KB, patch)
2013-01-21 09:27 PST, Sudarsana Nagineni (babu) 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Sudarsana Nagineni (babu)
Comment 1 2013-01-21 09:27:20 PST
Created attachment 183798 [details] Patch
Philippe Normand
Comment 2 2013-01-21 09:43:15 PST
Unless this is really urgent I'd rather fix this as part of bug 107398 and mention you in the ChangeLog :)
Sudarsana Nagineni (babu)
Comment 3 2013-01-21 11:16:54 PST
(In reply to comment #2) > Unless this is really urgent I'd rather fix this as part of bug 107398 and mention you in the ChangeLog :) Thanks for looking, Philippe. Fine with me to handle this as part of that bug.
Philippe Normand
Comment 4 2013-01-22 00:01:06 PST
Setting bug dependency so I don't forgot to integrate this patch.
Philippe Normand
Comment 5 2013-02-01 02:27:45 PST
Fixed in r141566
Note You need to log in before you can comment on or make changes to this bug.