Issue 10961059: Correctly decrement OmxVideoDecodeAccelerator::output_buffers_at_component_ during Reset/Destroy. - Code Review

Chromium Code Reviews

chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out

(117)

My Issues | Starred Open | Closed | All

Issue 10961059: Correctly decrement OmxVideoDecodeAccelerator::output_buffers_at_component_ during Reset/Destroy. (Closed)

Created:
8 years, 3 months ago by Ami GONE FROM CHROMIUM

Modified:
8 years, 2 months ago

Reviewers:
Pawel Osciak

CC:
chromium-reviews, joi+watch-content_chromium.org, darin-cc_chromium.org, jam, apatrick_chromium, feature-media-reviews_chromium.org, marcheu, ArunM

Base URL:
svn://svn.chromium.org/chrome/trunk/src

Visibility:
Public.

More Reviews

Description

Correctly decrement OmxVideoDecodeAccelerator::output_buffers_at_component_ during Reset/Destroy. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=158285

Patch Set 1 #

Created: 8 years, 3 months ago

Download [raw] [tar.bz2]

		Unified diffs	Side-by-side diffs	Delta from patch set	Stats (+8 lines, -8 lines)			Patch
	M	content/common/gpu/media/omx_video_decode_accelerator.cc	View		2 chunks	+8 lines, -8 lines	0 comments	Download

Messages

Total messages: 7 (0 generated)

Expand Messages | Collapse Messages

Ami GONE FROM CHROMIUM

Pawel: please review. Arun: can you test that this fixes your issue?

8 years, 3 months ago (2012-09-22 02:42:09 UTC) #1

Arun M

Yes, its fixed. On 2012/09/22 02:42:09, Ami Fischman wrote: > Pawel: please review. > Arun: ...

8 years, 2 months ago (2012-09-24 05:48:54 UTC) #3

Arun M

Do we have to merge this to R23? On 2012/09/24 05:48:54, Arun M wrote: > ...

8 years, 2 months ago (2012-09-24 11:56:47 UTC) #4

Ami GONE FROM CHROMIUM

I don't think so since this is dcheck-only so irrelevant for users. -a On Sep ...

8 years, 2 months ago (2012-09-24 14:17:52 UTC) #5

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/fischman@chromium.org/10961059/1

8 years, 2 months ago (2012-09-24 14:18:11 UTC) #6

Change committed as 158285

Expand Messages | Collapse Messages

Powered by Google App Engine

This is Rietveld 408576698