Bug 77334 - [chromium] MediaStream API: Fix a few memory leaks
Summary: [chromium] MediaStream API: Fix a few memory leaks
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit API (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Tommy Widenflycht
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-01-30 05:32 PST by Tommy Widenflycht
Modified: 2012-01-30 10:36 PST (History)
2 users (show)

See Also:


Attachments
Patch (4.37 KB, patch)
2012-01-30 05:35 PST, Tommy Widenflycht
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
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.