RESOLVED INVALID 122832
[GStreamer] use a smart pointer for ::setStreamVolumeElement() 
https://bugs.webkit.org/show_bug.cgi?id=122832
Summary [GStreamer] use a smart pointer for ::setStreamVolumeElement()
Philippe Normand
Reported 2013-10-15 07:43:54 PDT
SSIA
Attachments
patch (1.82 KB, patch)
2013-10-15 08:30 PDT, Philippe Normand 		eflews.bot: commit-queue-
DetailsFormatted DiffDiff
patch (2.61 KB, patch)
2013-10-15 08:40 PDT, Philippe Normand 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Philippe Normand
Comment 1 2013-10-15 08:30:06 PDT
Created attachment 214260 [details] patch
EFL EWS Bot
Comment 2 2013-10-15 08:33:23 PDT
Comment on attachment 214260 [details] patch Attachment 214260 [details] did not pass efl-ews (efl): Output: http://webkit-queues.appspot.com/results/4099038
EFL EWS Bot
Comment 3 2013-10-15 08:34:49 PDT
Comment on attachment 214260 [details] patch Attachment 214260 [details] did not pass efl-wk2-ews (efl-wk2): Output: http://webkit-queues.appspot.com/results/4099039
kov's GTK+ EWS bot
Comment 4 2013-10-15 08:39:43 PDT
Comment on attachment 214260 [details] patch Attachment 214260 [details] did not pass gtk-ews (gtk): Output: http://webkit-queues.appspot.com/results/4099042
Philippe Normand
Comment 5 2013-10-15 08:40:37 PDT
Created attachment 214264 [details] patch
Darin Adler
Comment 6 2013-10-15 13:21:38 PDT
Comment on attachment 214264 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=214264&action=review > Source/WebCore/ChangeLog:10 > + Use a smart pointer all the way from call site to member variable storage. Why?
Philippe Normand
Comment 7 2013-10-16 08:09:01 PDT
(In reply to comment #6) > (From update of attachment 214264 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=214264&action=review > > > Source/WebCore/ChangeLog:10 > > + Use a smart pointer all the way from call site to member variable storage. > > Why? Oh well I thought it'd simply be cleaner but I can't find other reasons and there's no ref leak in the current code. So I think I'll end up not doing this change.
Note You need to log in before you can comment on or make changes to this bug.