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

Issue 452933006: vp8 fix: re-allocate denoiser for spatial resize. (Closed)

Created:
6 years, 4 months ago by marpan
Modified:
6 years, 4 months ago
Reviewers:
Johann, Tom Finegan
CC:
chromium-reviews, wwcv, jzern, fgalligan1
Visibility:
Public.

Description

vp8 fix: re-allocate denoiser for spatial resize. Change has been committed upstream, see: https://gerrit.chromium.org/gerrit/#/c/71097/ https://gerrit.chromium.org/gerrit/#/c/71121/ BUG=http://code.google.com/p/chromium/issues/detail?id=392112 R=johannkoenig@google.com TBR=tomfinegan@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=288801

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -0 lines) Patch
M source/libvpx/vp8/encoder/onyx_if.c View 1 chunk +8 lines, -0 lines 0 comments Download

Messages

Total messages: 7 (0 generated)
marpan
6 years, 4 months ago (2014-08-11 19:22:49 UTC) #1
Johann
It looks like you have https://gerrit.chromium.org/gerrit/#/c/71121/1/vp8/encoder/onyx_if.c,unified but not https://gerrit.chromium.org/gerrit/#/c/71097/5/vp8/encoder/onyx_if.c,unified
6 years, 4 months ago (2014-08-11 21:19:11 UTC) #2
Johann
On 2014/08/11 21:19:11, Johann wrote: > It looks like you have > https://gerrit.chromium.org/gerrit/#/c/71121/1/vp8/encoder/onyx_if.c,unified > but ...
6 years, 4 months ago (2014-08-11 21:19:33 UTC) #3
marpan
This change has the net effect of both changes. 71121 was another fix based on ...
6 years, 4 months ago (2014-08-11 21:35:29 UTC) #4
Johann
LGTM. My mistake. Didn't even scroll down.
6 years, 4 months ago (2014-08-11 21:38:00 UTC) #5
marpan
Committed patchset #1 manually as r288801 (presubmit successful).
6 years, 4 months ago (2014-08-11 21:47:21 UTC) #6
Tom Finegan
6 years, 4 months ago (2014-08-12 02:01:43 UTC) #7
Message was sent while issue was closed.
On 2014/08/11 21:47:21, marpan wrote:
> Committed patchset #1 manually as r288801 (presubmit successful).

lgtm

Powered by Google App Engine
This is Rietveld 408576698