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

Issue 360053005: vp8 denoiser fix: free memory of denoiser->denoiser_state. (Closed)

Created:
6 years, 5 months ago by marpan1
Modified:
6 years, 5 months ago
CC:
chromium-reviews, wwcv, jzern, fgalligan1, Tom Finegan
Visibility:
Public.

Description

vp8 denoiser fix: free memory of denoiser->denoiser_state. Change has been committed upstream: https://gerrit.chromium.org/gerrit/#/c/70675/ BUG=https://code.google.com/p/webrtc/issues/detail?id=3524 R=johannkoenig@google.com, ronghuawu@chromium.org TBR=tomfinegan@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=280676

Patch Set 1 #

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

Messages

Total messages: 5 (0 generated)
marpan1
6 years, 5 months ago (2014-06-30 20:35:12 UTC) #1
Johann
LGTM (but you need a LGTM from a real committer)
6 years, 5 months ago (2014-06-30 20:56:42 UTC) #2
marpan1
6 years, 5 months ago (2014-06-30 21:21:01 UTC) #3
Ronghua Wu (Left Chromium)
rubberstamp LGTM as committer
6 years, 5 months ago (2014-06-30 21:35:32 UTC) #4
marpan1
6 years, 5 months ago (2014-06-30 21:53:18 UTC) #5
Message was sent while issue was closed.
Committed patchset #1 manually as r280676 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698