Commit 97d592f7 authored by marcheu@chromium.org's avatar marcheu@chromium.org

OMX: Don't leave FreeOutputBuffers if MakeCurrent fails.

If it fails, we still want to free the OMX resources.

BUG=chrome-os-partner:12316
TEST=by hand: play a video, look for buffer leaks.

Change-Id: Ifcc4fe4248b72067e4592224b24fdb716ab4b230


Review URL: https://chromiumcodereview.appspot.com/10832263

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151376 0039d316-1c4b-4281-b951-d872f2087c98
parent a427e071
......@@ -722,9 +722,6 @@ void OmxVideoDecodeAccelerator::FreeOutputBuffers() {
// Calls to OMX to free buffers.
OMX_ERRORTYPE result;
if (!make_context_current_.Run())
return;
for (OutputPictureById::iterator it = pictures_.begin();
it != pictures_.end(); ++it) {
OMX_BUFFERHEADERTYPE* omx_buffer = it->second.omx_buffer_header;
......
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