Issue 185393008: gpu: Return early GpuCommandBufferStub::OnWaitSyncPoint() if |sync_point| is 0.

Issue 185393008: gpu: Return early GpuCommandBufferStub::OnWaitSyncPoint() if |sync_point| is 0. (Closed)

Created:
6 years, 9 months ago by dshwang

Modified:
6 years, 9 months ago

Reviewers:
Zhenyao Mo, piman

CC:
chromium-reviews, joi+watch-content_chromium.org, piman+watch_chromium.org, jam, darin-cc_chromium.org

Base URL:
https://git.chromium.org/chromium/src.git@master

Visibility:
Public.

More Reviews

Description

gpu: Return early GpuCommandBufferStub::OnWaitSyncPoint() if |sync_point| is 0. WaitSyncPoint(0) does not do anything, so return early. Accelerated video code path frequently calls WaitSyncPoint(0). BUG=N/A Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=255271

Patch Set 1 #

Created: 6 years, 9 months ago

Download [raw] [tar.bz2]

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

Messages

Total messages: 9 (0 generated)

Expand Messages | Collapse Messages

dshwang

On 2014/03/05 22:11:13, piman wrote: > lgtm thank you for lgtm.

6 years, 9 months ago (2014-03-05 22:12:38 UTC) #3

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/dongseong.hwang@intel.com/185393008/1

6 years, 9 months ago (2014-03-05 22:14:46 UTC) #5

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/dongseong.hwang@intel.com/185393008/1

6 years, 9 months ago (2014-03-06 00:05:06 UTC) #6

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/dongseong.hwang@intel.com/185393008/1

6 years, 9 months ago (2014-03-06 02:48:55 UTC) #7

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/dongseong.hwang@intel.com/185393008/1

6 years, 9 months ago (2014-03-06 03:13:34 UTC) #8

Message was sent while issue was closed.

Change committed as 255271

Expand Messages | Collapse Messages