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

Issue 10806046: Update encrypted WebM test files in media/test/data. (Closed)

Created:
8 years, 5 months ago by fgalligan1
Modified:
8 years, 5 months ago
CC:
chromium-reviews, feature-media-reviews_chromium.org
Visibility:
Public.

Description

Update encrypted WebM test files in media/test/data. - The files are updated to work with decryptor changes from crbug.com/119845. BUG=132801 TEST=media_unittests Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=147940

Patch Set 1 #

Patch Set 2 : Rebase to master. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+-1 lines, --1 lines) Patch
M media/test/data/bear-320x240-encrypted.webm View 0 chunks +-1 lines, --1 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
fgalligan1
Is there anything else I need to do for a CL that only updates binary ...
8 years, 5 months ago (2012-07-20 22:41:16 UTC) #1
scherkus (not reviewing)
LGTM for test binary files like this it's OK to TBR them
8 years, 5 months ago (2012-07-21 01:14:45 UTC) #2
xhwang
We don't need media/test/data/vp8-encrypted-I-frame-320x240 file anymore once http://codereview.chromium.org/10800057/ is landed. Otherwise LGTM.
8 years, 5 months ago (2012-07-21 01:18:44 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/fgalligan@chromium.org/10806046/8001
8 years, 5 months ago (2012-07-23 18:58:27 UTC) #4
commit-bot: I haz the power
8 years, 5 months ago (2012-07-23 19:44:12 UTC) #5
Try job failure for 10806046-8001 (retry) on linux_rel for step "apply_issue".
It's a second try, previously, step "apply_issue" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&...

Powered by Google App Engine
This is Rietveld 408576698