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

Issue 16145002: Add more tests to Chromium WebRTC testers. (Closed)

Created:
7 years, 6 months ago by kjellander_chromium
Modified:
7 years, 6 months ago
CC:
chromium-reviews, cmp-cc_chromium.org, kjellander+cc_chromium.org, xusydoc+watch_chromium.org, ilevy-cc_chromium.org, phoglund_chromium
Visibility:
Public.

Description

Add more tests to Chromium WebRTC testers. These are tests that normally don't execute in the main waterfall (except content_unittest). The PyAuto tests are in the progress of being converted to browser_tests and Telemetry tests, but we'd like to get them up here now so we can retire our own machines running these tests. Overview of the new tests added: * PyAuto AppRTC: tests that the WebRTC reference application is working. * PyAuto quality: performs video and audio quality analysis to catch regressions. * content_unittests: subset of the main waterfall execution of this test. Used to catch regressions with recent WebRTC revisions in the chromium.webrtc.fyi waterfall. * manual browser_tests and content_browsertests: Tests that normally don't run in the main waterfall, as they require audio/video devices. Prerequisites: The quality tests needs ffmpeg 0.11.1 and zxing (https://code.google.com/p/zxing/) executables to be installed in the PATH of the buildbot tester. This will be done by: * Win: Copying ffmpeg and zxing into build_internal\tools * Mac: Installing ffmpeg into /usr/local/bin and copying zxing into ~/slavebin * Linux: Installing ffmpeg into /usr/bin and copying zxing into ~/slavebin TEST=local execution on Linux, Windows and Mac. BUG=244550 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=208672

Patch Set 1 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+87 lines, -39 lines) Patch
M masters/master.chromium.webrtc.fyi/master_linux_cfg.py View 1 chunk +9 lines, -1 line 0 comments Download
M masters/master.chromium.webrtc.fyi/master_mac_cfg.py View 1 chunk +9 lines, -1 line 0 comments Download
M masters/master.chromium.webrtc.fyi/master_win_cfg.py View 1 chunk +9 lines, -1 line 0 comments Download
M masters/master.chromium.webrtc/master_linux_cfg.py View 1 chunk +9 lines, -1 line 0 comments Download
M masters/master.chromium.webrtc/master_mac_cfg.py View 1 chunk +9 lines, -1 line 0 comments Download
M masters/master.chromium.webrtc/master_win_cfg.py View 1 chunk +9 lines, -2 lines 0 comments Download
M scripts/master/factory/chromium_factory.py View 2 chunks +33 lines, -32 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
kjellander_chromium
Add more tests to get a complete coverage similar to the one we have today ...
7 years, 6 months ago (2013-05-28 19:53:06 UTC) #1
Mike Stip (use stip instead)
lgtm
7 years, 6 months ago (2013-05-28 20:14:31 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/kjellander@chromium.org/16145002/4002
7 years, 6 months ago (2013-06-26 08:35:44 UTC) #3
commit-bot: I haz the power
7 years, 6 months ago (2013-06-26 08:36:39 UTC) #4
Message was sent while issue was closed.
Change committed as 208672

Powered by Google App Engine
This is Rietveld 408576698