Bug 77334

Summary: [chromium] MediaStream API: Fix a few memory leaks
Product: WebKit Reporter: Tommy Widenflycht <tommyw>
Component: WebKit APIAssignee: Tommy Widenflycht <tommyw>
Status: RESOLVED FIXED    
Severity: Normal CC: fishd, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch none

Description Tommy Widenflycht 2012-01-30 05:32:33 PST
It seems that either the behaviour of WebPrivatePtr has changed or I misunderstood something, but I fixed a memory leak in WebMediaStreamDescriptor and safeguarded WebMediaStreamSource.
Comment 1 Tommy Widenflycht 2012-01-30 05:35:56 PST
Created attachment 124535 [details]
Patch
Comment 2 WebKit Review Bot 2012-01-30 05:37:38 PST
Please wait for approval from fishd@chromium.org before submitting because this patch contains changes to the Chromium public API.
Comment 3 Darin Fisher (:fishd, Google) 2012-01-30 09:27:24 PST
Comment on attachment 124535 [details]
Patch

Sorry for not catching this in the initial code review!
Comment 4 WebKit Review Bot 2012-01-30 10:36:33 PST
Comment on attachment 124535 [details]
Patch

Clearing flags on attachment: 124535

Committed r106256: <http://trac.webkit.org/changeset/106256>
Comment 5 WebKit Review Bot 2012-01-30 10:36:38 PST
All reviewed patches have been landed.  Closing bug.