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

Issue 16866016: Revert hotfix for EVDA tearing. (Closed)

Created:
7 years, 6 months ago by sheu
Modified:
7 years, 6 months ago
CC:
chromium-reviews, joi+watch-content_chromium.org, darin-cc_chromium.org, jam, apatrick_chromium, feature-media-reviews_chromium.org, Pawel Osciak
Base URL:
https://chromium.googlesource.com/chromium/src.git@git-svn
Visibility:
Public.

Description

Revert hotfix for EVDA tearing. The EVDA tearing issue, caused by synchronization problems with the Mali EGL driver, has been fixed on the Mali driver side. Revert the hotfix here, allowing us to free up another two output buffers on the pipeline. BUG=chromium:225563 TEST=local build, run on CrOS snow Change-Id: I80f21275219050a0d32b746e2638d056ce9fac4f Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=206245

Patch Set 1 #

Total comments: 2

Patch Set 2 : 27c368ed Copyright notice. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -12 lines) Patch
M content/common/gpu/media/exynos_video_decode_accelerator.h View 1 1 chunk +0 lines, -6 lines 0 comments Download
M content/common/gpu/media/exynos_video_decode_accelerator.cc View 1 1 chunk +1 line, -6 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
sheu
PTAL.
7 years, 6 months ago (2013-06-13 20:44:20 UTC) #1
Ami GONE FROM CHROMIUM
LGTM % lolwats https://chromiumcodereview.appspot.com/16866016/diff/1/content/common/gpu/media/exynos_video_decode_accelerator.cc File content/common/gpu/media/exynos_video_decode_accelerator.cc (right): https://chromiumcodereview.appspot.com/16866016/diff/1/content/common/gpu/media/exynos_video_decode_accelerator.cc#newcode1 content/common/gpu/media/exynos_video_decode_accelerator.cc:1: // Copyright (c) 2012 The Chromium ...
7 years, 6 months ago (2013-06-13 20:56:47 UTC) #2
sheu
On 2013/06/13 20:56:47, Ami Fischman wrote: > LGTM % lolwats > > https://chromiumcodereview.appspot.com/16866016/diff/1/content/common/gpu/media/exynos_video_decode_accelerator.cc > File ...
7 years, 6 months ago (2013-06-13 20:57:44 UTC) #3
sheu
On 2013/06/13 20:57:44, sheu wrote: > The precommit verification script changed. It wants "Chromium OS ...
7 years, 6 months ago (2013-06-13 21:00:19 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/sheu@chromium.org/16866016/3002
7 years, 6 months ago (2013-06-13 21:45:02 UTC) #5
commit-bot: I haz the power
7 years, 6 months ago (2013-06-14 00:33:20 UTC) #6
Message was sent while issue was closed.
Change committed as 206245

Powered by Google App Engine
This is Rietveld 408576698