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

Issue 10826131: Enable WebM encrypted playback test. (Closed)

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

Description

Enable WebM encrypted playback test. - bear-320x240-encrypted.webm was updated to match the decryption code in http://codereview.chromium.org/10824136/. BUG=138261 TEST=media_unittests Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=150173

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -6 lines) Patch
M media/filters/pipeline_integration_test.cc View 2 chunks +4 lines, -6 lines 0 comments Download

Messages

Total messages: 9 (0 generated)
xhwang
lgtm
8 years, 4 months ago (2012-08-03 20:45:46 UTC) #1
xhwang
On 2012/08/03 20:45:46, xhwang wrote: > lgtm BTW, do you want to close http://codereview.chromium.org/10805073/ ?
8 years, 4 months ago (2012-08-03 20:54:13 UTC) #2
fgalligan1
On 2012/08/03 20:54:13, xhwang wrote: > On 2012/08/03 20:45:46, xhwang wrote: > > lgtm > ...
8 years, 4 months ago (2012-08-03 21:36:47 UTC) #3
xhwang
On 2012/08/03 21:36:47, fgalligan1 wrote: > On 2012/08/03 20:54:13, xhwang wrote: > > On 2012/08/03 ...
8 years, 4 months ago (2012-08-03 22:42:38 UTC) #4
fgalligan
OK Should I dcommit then? On Fri, Aug 3, 2012 at 3:42 PM, <xhwang@chromium.org> wrote: ...
8 years, 4 months ago (2012-08-03 22:44:19 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/fgalligan@chromium.org/10826131/1
8 years, 4 months ago (2012-08-03 22:45:54 UTC) #6
commit-bot: I haz the power
Try job failure for 10826131-1 (retry) on win_rel for step "runhooks" (clobber build). It's a ...
8 years, 4 months ago (2012-08-03 23:01:45 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/fgalligan@chromium.org/10826131/1
8 years, 4 months ago (2012-08-06 18:51:57 UTC) #8
commit-bot: I haz the power
8 years, 4 months ago (2012-08-06 23:25:25 UTC) #9
Change committed as 150173

Powered by Google App Engine
This is Rietveld 408576698