Issue 5336002: Remove tests for codecs we no longer plan on supporting.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Issue 5336002: Remove tests for codecs we no longer plan on supporting. (Closed)

Created:
10 years, 1 month ago by scherkus (not reviewing)

Modified:
9 years, 7 months ago

Reviewers:
petkov, fbarchard, Alpha Left Google

CC:
chromium-os-reviews_chromium.org, truty+cc_chromium.org, sosa+cc_chromium.org, seano+cc_chromium.org, ericli, petkov+cc_chromium.org

Base URL:
ssh://git@gitrw.chromium.org:9222/autotest.git@master

Visibility:
Public.

Description

Remove tests for codecs we no longer plan on supporting. Also removed tests that depended on other sites to provide content, as they could remove content at anytime and break the test. BUG=chromium-os:5890 TEST=audiovideo_FFMPEG should continue to run Change-Id: I152d8319634300216fa43fb8cb3ce738ef8a2077 Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=ade14cf

Patch Set 1 #

Total comments: 1

Created: 10 years, 1 month ago

Download [raw] [tar.bz2]

		Unified diffs	Side-by-side diffs	Delta from patch set	Stats (+1 line, -35 lines)			Patch
	M	client/site_tests/audiovideo_FFMPEG/audiovideo_FFMPEG.py	View		1 chunk	+1 line, -4 lines	0 comments	Download
	M	client/site_tests/audiovideo_FFMPEG/testcases	View		4 chunks	+0 lines, -31 lines	1 comment	Download

Messages

Total messages: 3 (0 generated)

Expand Messages | Collapse Messages

fbarchard

LGTM http://codereview.chromium.org/5336002/diff/1/client/site_tests/audiovideo_FFMPEG/testcases File client/site_tests/audiovideo_FFMPEG/testcases (left): http://codereview.chromium.org/5336002/diff/1/client/site_tests/audiovideo_FFMPEG/testcases#oldcode10 client/site_tests/audiovideo_FFMPEG/testcases:10: # in (crosbug.com/5890). may want to add TODO ...

10 years, 1 month ago (2010-11-24 01:59:00 UTC) #2

LGTM given that looks good to fbarchard and yourself.

Expand Messages | Collapse Messages