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

Issue 9187016: Update libvpx revision in DEPS (Closed)

Created:
8 years, 11 months ago by Roland McGrath
Modified:
8 years, 11 months ago
Reviewers:
jkoleszar
CC:
chromium-reviews
Visibility:
Public.

Description

Update libvpx revision in DEPS This brings in these changes: http://codereview.chromium.org/8501011 r109236 Suppress libvpx warning http://codereview.chromium.org/8666017 r113567 Build libvpx on non-Mac POSIX platforms also, by reusing the linux config. http://codereview.chromium.org/9185015 r117274 Compile libvpx code as PIC for all Linux targets BUG= http://code.google.com/p/chromium/issues/detail?id=109318 TEST= sizes test should show chrome-textrel=0 R=jkoleszar@google.com Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=117287

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M DEPS View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 2 (0 generated)
Roland McGrath
8 years, 11 months ago (2012-01-11 20:21:29 UTC) #1
jkoleszar
8 years, 11 months ago (2012-01-11 20:33:18 UTC) #2
On 2012/01/11 20:21:29, Roland McGrath wrote:

LGTM

Powered by Google App Engine
This is Rietveld 408576698