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

Issue 848963004: vp9: sync threads after a longjmp (Closed)

Created:
5 years, 11 months ago by Johann
Modified:
5 years, 11 months ago
Reviewers:
Tom Finegan
CC:
wwcv, jzern, fgalligan1
Base URL:
https://chromium.googlesource.com/chromium/deps/libvpx.git@m40-2214
Target Ref:
refs/heads/m40-2214
Visibility:
Public.

Description

vp9: sync threads after a longjmp Synchronize all threads immediately as a subsequent decode call may cause a resize invalidating some allocations. fixes one aspect of crbug.com/437655 https://gerrit.chromium.org/gerrit/73143 BUG=437655 R=tomfinegan@chromium.org Committed: 715db78484f91a05fbd0584c0ed6e9d2ffc9be06

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+11 lines, -0 lines) Patch
M source/libvpx/vp9/decoder/vp9_decoder.c View 1 chunk +11 lines, -0 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Johann
Committed patchset #1 (id:1) manually as 715db78484f91a05fbd0584c0ed6e9d2ffc9be06 (presubmit successful).
5 years, 11 months ago (2015-01-13 23:50:12 UTC) #1
Tom Finegan
5 years, 11 months ago (2015-01-13 23:51:59 UTC) #2
Message was sent while issue was closed.
On 2015/01/13 23:50:12, Johann wrote:
> Committed patchset #1 (id:1) manually as
> 715db78484f91a05fbd0584c0ed6e9d2ffc9be06 (presubmit successful).

lgtm

Powered by Google App Engine
This is Rietveld 408576698