Commit 3757446e authored by mrowe@apple.com's avatar mrowe@apple.com

<rdar://problem/9124537> Crashes during layout tests due to overrelease of AVFoundation objects.

Rubber-stamped by Eric Carlsson.

* platform/graphics/avfoundation/MediaPlayerPrivateAVFoundationObjC.mm:
(WebCore::MediaPlayerPrivateAVFoundationObjC::createContextVideoRenderer):
(WebCore::MediaPlayerPrivateAVFoundationObjC::setAsset):

git-svn-id: svn://svn.chromium.org/blink/trunk@80916 bbb929c8-8fbe-4397-9dbb-9b2b20218538
parent e8bd6b8c
2011-03-11 Mark Rowe <mrowe@apple.com>
Rubber-stamped by Eric Carlsson.
<rdar://problem/9124537> Crashes during layout tests due to overrelease of AVFoundation objects.
* platform/graphics/avfoundation/MediaPlayerPrivateAVFoundationObjC.mm:
(WebCore::MediaPlayerPrivateAVFoundationObjC::createContextVideoRenderer):
(WebCore::MediaPlayerPrivateAVFoundationObjC::setAsset):
2011-03-11 Chris Marrin <cmarrin@apple.com> 2011-03-11 Chris Marrin <cmarrin@apple.com>
Unreviewed. Unreviewed.
......
...@@ -188,7 +188,7 @@ void MediaPlayerPrivateAVFoundationObjC::createContextVideoRenderer() ...@@ -188,7 +188,7 @@ void MediaPlayerPrivateAVFoundationObjC::createContextVideoRenderer()
if (!m_avAsset || m_imageGenerator) if (!m_avAsset || m_imageGenerator)
return; return;
m_imageGenerator.adoptNS([AVAssetImageGenerator assetImageGeneratorWithAsset:m_avAsset.get()]); m_imageGenerator = [AVAssetImageGenerator assetImageGeneratorWithAsset:m_avAsset.get()];
[m_imageGenerator.get() setApertureMode:AVAssetImageGeneratorApertureModeCleanAperture]; [m_imageGenerator.get() setApertureMode:AVAssetImageGeneratorApertureModeCleanAperture];
[m_imageGenerator.get() setAppliesPreferredTrackTransform:YES]; [m_imageGenerator.get() setAppliesPreferredTrackTransform:YES];
...@@ -498,7 +498,7 @@ unsigned MediaPlayerPrivateAVFoundationObjC::totalBytes() const ...@@ -498,7 +498,7 @@ unsigned MediaPlayerPrivateAVFoundationObjC::totalBytes() const
void MediaPlayerPrivateAVFoundationObjC::setAsset(id asset) void MediaPlayerPrivateAVFoundationObjC::setAsset(id asset)
{ {
m_avAsset.adoptNS(asset); m_avAsset = asset;
} }
MediaPlayerPrivateAVFoundation::AVAssetStatus MediaPlayerPrivateAVFoundationObjC::assetStatus() const MediaPlayerPrivateAVFoundation::AVAssetStatus MediaPlayerPrivateAVFoundationObjC::assetStatus() const
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment