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

Issue 17470002: Update encryption mechanism in libvpx (Closed)

Created:
7 years, 6 months ago by wwcv
Modified:
7 years, 6 months ago
CC:
chromium-reviews, fgalligan1, jpet1
Base URL:
https://src.chromium.org/chrome/trunk/src/
Visibility:
Public.

Description

Update encryption mechanism in libvpx "This allows code calling the library can choose an arbitrary encryption algorithm." https://gerrit.chromium.org/gerrit/#/c/58562/ Pull to update encryption: https://codereview.chromium.org/17009012/ Pull to fix win64 warnings: https://codereview.chromium.org/17451020/ Update README.chromium: https://codereview.chromium.org/17493003/ Another win64 warning: https://codereview.chromium.org/16835006/ README.chromium: https://codereview.chromium.org/17267003/ Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=207944

Patch Set 1 #

Patch Set 2 : #

Patch Set 3 : #

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

Messages

Total messages: 7 (0 generated)
wwcv
Waiting for the bots to come back clean so this can go in tomorrow morning. ...
7 years, 6 months ago (2013-06-20 00:21:31 UTC) #1
scherkus (not reviewing)
lgtm w/ CL description nit future reference: try to have a single reviewer for these ...
7 years, 6 months ago (2013-06-20 00:24:30 UTC) #2
wwcv
On 2013/06/20 00:24:30, scherkus wrote: > lgtm w/ CL description nit > > future reference: ...
7 years, 6 months ago (2013-06-20 00:33:04 UTC) #3
scherkus (not reviewing)
On 2013/06/20 00:33:04, wwcv wrote: > On 2013/06/20 00:24:30, scherkus wrote: > > lgtm w/ ...
7 years, 6 months ago (2013-06-20 00:38:54 UTC) #4
jpet1
lgtm
7 years, 6 months ago (2013-06-20 16:12:10 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/johannkoenig@chromium.org/17470002/40001
7 years, 6 months ago (2013-06-21 17:09:11 UTC) #6
commit-bot: I haz the power
7 years, 6 months ago (2013-06-21 22:50:10 UTC) #7
Message was sent while issue was closed.
Change committed as 207944

Powered by Google App Engine
This is Rietveld 408576698