Commit Graph

233 Commits

Author SHA1 Message Date
kjellander@webrtc.org
0240e8e90f Wider TSAN suppression for issue 300
On some machines, this test has still been failing, so I'm widening the
suppression to resolve this.

BUG=300
TEST=passing linux_tsan trybot.

Review URL: https://webrtc-codereview.appspot.com/992004

git-svn-id: http://webrtc.googlecode.com/svn/trunk@3280 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-12-13 13:02:29 +00:00
kjellander@webrtc.org
d2bcde2e4e Suppressing TSan warnings for system_wrappers_unittests
This CL makes system_wrappers_unittests pass on Lucid (it passed on Precise
without them for me).

BUG=300
TEST=Try job on linux_tsan
TBR=henrike

Review URL: https://webrtc-codereview.appspot.com/976007

git-svn-id: http://webrtc.googlecode.com/svn/trunk@3263 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-12-11 21:39:18 +00:00
turaj@webrtc.org
42259e7ebc VoE Changes to enable dual_streaming.
TEST=added new unit-test

This CL depends on issue 933015 http://webrtc-codereview.appspot.com/933015/
which is under review. Should be committed after issue 933015 is committed.
Committed: https://code.google.com/p/webrtc/source/detail?r=3231
Review URL: https://webrtc-codereview.appspot.com/970005

git-svn-id: http://webrtc.googlecode.com/svn/trunk@3257 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-12-11 02:15:12 +00:00
phoglund@webrtc.org
3bb42ef0d6 Made e2e audio quality test write its results to perf.
The https://chromereviews.googleplex.com/5573026/ patch will mark the test step as perf-printing - this cl will make the test actually print perf lines.

TEST=Ran locally.

Review URL: https://webrtc-codereview.appspot.com/933036

git-svn-id: http://webrtc.googlecode.com/svn/trunk@3252 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-12-10 10:08:00 +00:00
phoglund@webrtc.org
78bec2dcbe Fixed bug where we would rewrite *deref_ptr = ...; to // deref_ptr = ...;
BUG=

Review URL: https://webrtc-codereview.appspot.com/929036

git-svn-id: http://webrtc.googlecode.com/svn/trunk@3219 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-12-03 08:48:07 +00:00
phoglund@webrtc.org
0f8286fd75 Added last (?) suppressions for known issues.
BUG=1152

Review URL: https://webrtc-codereview.appspot.com/933027

git-svn-id: http://webrtc.googlecode.com/svn/trunk@3180 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-11-28 14:21:12 +00:00
phoglund@webrtc.org
e3b2bc6c40 Will now fix old src-relative paths so we go to webrtc/ paths.
We used to write paths relative to src, e.g. starting with video_engine/ etc, but now it should be webrtc/video_engine/. This script will now get that right.

BUG=

Review URL: https://webrtc-codereview.appspot.com/970006

git-svn-id: http://webrtc.googlecode.com/svn/trunk@3154 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-11-23 09:09:59 +00:00
phoglund@webrtc.org
3ec52c0689 Adding mflodman's reformat script with some fixes.
BUG=

Review URL: https://webrtc-codereview.appspot.com/966024

git-svn-id: http://webrtc.googlecode.com/svn/trunk@3143 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-11-21 13:28:52 +00:00
kjellander@webrtc.org
0f4185f8bb Making valgrind wrapper script work with test arguments
This is needed to be able to run memcheck/tsan/asan with command line arguments in the way the bots are executing the script (using --build_dir and --test arguments).

BUG=none
TEST=local execution

Review URL: https://webrtc-codereview.appspot.com/938015

git-svn-id: http://webrtc.googlecode.com/svn/trunk@3106 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-11-15 15:19:28 +00:00
tina.legrand@webrtc.org
1f8c02a4de Getting Opus from Chrome
This CL replaces http://review.webrtc.org/921006/

BUG=issue982

TBR=andrew@webrtc.org

Review URL: https://webrtc-codereview.appspot.com/933006

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2993 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-10-25 12:37:08 +00:00
phoglund@webrtc.org
8d408455d9 Generalized suppressions to cover issues found on bot.
Turns out the bot is running a different version of libpthread, probably because I'm on gPrecise / gcc 4.6 whereas the bot is on 4.4? Anyway, I've generalized that stuff now.

BUG=
TEST=Ran voe & vie_auto_test under valgrind locally.

Review URL: https://webrtc-codereview.appspot.com/908004

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2973 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-10-23 09:21:16 +00:00
andrew@webrtc.org
14b43beb7c Move src/ -> webrtc/
TBR=niklas.enbom@webrtc.org

Review URL: https://webrtc-codereview.appspot.com/915006

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2963 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-10-22 18:19:23 +00:00
tina.legrand@webrtc.org
a7d8387bdd Opus integration
First patch = delivery from August 22, 2012.

Review URL: https://webrtc-codereview.appspot.com/756005

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2945 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-10-18 10:00:52 +00:00
phoglund@webrtc.org
21ca3fab12 Extended suppressions to be more general and suppress missed errors.
BUG=

Review URL: https://webrtc-codereview.appspot.com/864015

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2930 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-10-15 21:02:56 +00:00
kjellander@webrtc.org
ccb52c2961 Improve error handling for barcode_tools
These changes make the execution abort earlier on an error (like a tool is not found) and makes it easier to figure out what's wrong.

Made build_zxing.py executable.

BUG=None
TEST=Local runs of the PyAuto test src/chrome/test/functional/webrtc_video_quality.py in Chromium.

Review URL: https://webrtc-codereview.appspot.com/840005

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2899 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-10-10 16:11:28 +00:00
phoglund@webrtc.org
ff33bdd9d7 Suppressed all voe_auto_test standard suite bugs and introduced a flag for excluding timing-dependent tests.
Also Suppressed FakeMediaProcess errors (bug 898) and took out a test and suppressed general errors (bug 332). Lastly, fixed memory leak in misc test.

BUG=898, 332
TEST=Ran voe_auto_test with repeat=10 through the whole standard suite, under valgrind. Ran without valgrind. Tested that the extended and standard tests still start and are reachable from the menu.

Review URL: https://webrtc-codereview.appspot.com/855009

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2898 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-10-10 15:41:03 +00:00
phoglund@webrtc.org
4743f938d6 Added more suppressions.
These tests are really flaky for sure. None of these showed up locally or in previous runs.

BUG=
TBR=kjellander@webrtc.org

Review URL: https://webrtc-codereview.appspot.com/863006

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2860 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-10-02 09:22:51 +00:00
phoglund@webrtc.org
321da3bc4f Added more suppressions found on the bot.
I will look over and split them into different bugs later - seems a bad idea to use one bug for everything.

BUG=
TBR=kjellander@webrtc.org

Review URL: https://webrtc-codereview.appspot.com/861005

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2859 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-10-02 08:54:45 +00:00
phoglund@webrtc.org
4fdc102f03 Added suppression for one missed error.
BUG=
TBR=kjellander@webrtc.org

Review URL: https://webrtc-codereview.appspot.com/862004

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2855 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-10-01 14:53:30 +00:00
phoglund@webrtc.org
8ff3ff1a8b Made ViE standard tests runnable under valgrind.
Ensured there are bugs for all open valgrind issues in the standard tests and suppressed the known issues. This way, we can get it running in continuous integration and keep new issues from entering.

Removed bad check in codec test, added suppressions.

Fixed simple memory leaks in tests.

BUG=Related to bug 329
TEST=Ran the vie_auto_test standard suite many times under valgrind to root out flakiness. Ran the standard suite without valgrind to ensure I didn't break anything.

Review URL: https://webrtc-codereview.appspot.com/843005

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2854 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-10-01 10:04:26 +00:00
kjellander@webrtc.org
d1e7a9a90c ThreadSanitizer and Dr Memory for Windows
Added new wrapper script webrtc_tests.bat for executing memory/threading tests on Windows.
Updated webrtc_tests.sh to include modifications in chrome_tests.sh that has happened since we copied it.

To setup TSAN for Windows, see http://www.chromium.org/developers/how-tos/using-valgrind/threadsanitizer
I did like this:
1. Added "third_party/tsan": "http://src.chromium.org/chrome/trunk/deps/third_party/tsan"
to custom_deps in my .gclient file
2. gclient sync
3. SET GYP_DEFINES=build_with_tool=tsan && gclient runhooks
4. Compiled.
5. Ran the test using the wrapper script (see below).

To setup Dr Memory for Windows, see http://www.chromium.org/developers/how-tos/using-drmemory
I did like this:
1. Added "third_party/drmemory": "http://src.chromium.org/svn/trunk/deps/third_party/drmemory",
to custom_deps in my .gclient file (using the drmemory.DEPS as described on Chromium's wiki ends up in the wrong location)
2. gclient sync
3. SET GYP_DEFINES=build_with_tool=drmemory && gclient runhooks
4. Compiled.
5. Ran the test using the wrapper script (see below).


TEST=
On Windows: 
tools\valgrind-webrtc\webrtc_tests.bat --tool=tsan --test build\Debug\voice_engine_unittests.exe
tools\valgrind-webrtc\webrtc_tests.bat --tool=drmemory_light --test build\Debug\voice_engine_unittests.exe
tools\valgrind-webrtc\webrtc_tests.bat --tool=drmemory_full --test build\Debug\voice_engine_unittests.exe
On Linux: 
tools/valgrind-webrtc/webrtc_tests.sh --tool=memcheck --test out/Release/test_support_unittests
tools/valgrind-webrtc/webrtc_tests.sh --tool=tsan --test out/Release/test_support_unittests
tools/valgrind-webrtc/webrtc_tests.sh --tool=asan --test out/Release/test_support_unittests
Review URL: https://webrtc-codereview.appspot.com/845004

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2846 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-09-28 15:34:18 +00:00
kjellander@webrtc.org
1101278a17 Wider suppression filter for tz-warnings in TSAN
This catches the remaining warnings we see which it seems like the fun:__tz* suppression in Chromium's tools/valgrind/tsan/suppressions.txt file is aimed for.

BUG=884

Review URL: https://webrtc-codereview.appspot.com/841006

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2845 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-09-28 15:30:58 +00:00
phoglund@webrtc.org
e8ebbdad09 Removed branch coverage since it turned out to be essentially useless as a metric.
BUG=

Review URL: https://webrtc-codereview.appspot.com/812006

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2830 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-09-27 09:27:08 +00:00
kjellander@webrtc.org
b43f85ffd3 Support for being executed from runtests.py
This is also needed to make it possible to run unit tests easily using Chromium's buildbot source code.

BUG=None
TEST=tools/valgrind-webrtc/webrtc_tests.sh --test test_support_unittests --build_dir=out/Debug

Review URL: https://webrtc-codereview.appspot.com/784007

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2738 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-09-11 11:22:45 +00:00
kjellander@webrtc.org
b764d78831 Adding support for --test argument
BUG=Cannot use this from Chrome buildbot scripts.
TEST=Ran tools/valgrind-webrtc/webrtc_tests.sh --test out/Debug/test_support_unittests

Review URL: https://webrtc-codereview.appspot.com/784006

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2737 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-09-11 09:34:51 +00:00
kjellander@webrtc.org
ead3005c46 Making Java headless to save execution time
Is seems this only saves a few seconds, but it's a least less confusing for the user executing the test.

BUG=

Review URL: https://webrtc-codereview.appspot.com/761008

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2709 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-09-06 09:33:08 +00:00
vspasova@webrtc.org
992da29c75 A tool that crops the top pixel rows of a YUV video.
TEST=./tools/barcode_tools/yuv_cropper.py --width=640
--height=480 --crop_height=32
--yuv_file=<path_and_name_to_yuv_file>
--output_file=<path_and_name_to_output_file>

Review URL: https://webrtc-codereview.appspot.com/765005

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2692 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-08-31 09:47:47 +00:00
kjellander@webrtc.org
6307dbf468 Updates the PRESUBMIT.py to allow years from 2011 to the current year.
Since Chromium has moved to this policy, we should too.
Code is copied from /depot_tools/presubmit_canned_checks.py but modified for our purpose.

BUG=
TEST=Tested git cl presubmit with a modified .cc file with the 2011 header and one with the 2012.

Review URL: https://webrtc-codereview.appspot.com/770005

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2691 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-08-31 07:07:11 +00:00
vspasova@webrtc.org
1b0a02e12e Fixing a path and Ant invocation issue in build_zxing.py and delete_file issue in helper_functions.py
Review URL: https://webrtc-codereview.appspot.com/761006

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2688 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-08-30 12:56:38 +00:00
vspasova@webrtc.org
28655423fd Removed barcode_tools.gyp and added build_zxing.py
BUG=

Review URL: https://webrtc-codereview.appspot.com/735009

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2615 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-08-15 14:35:40 +00:00
vspasova@webrtc.org
8327c85bf5 Fixing a syntax error in barcode_tools DEPS.
BUG=

Review URL: https://webrtc-codereview.appspot.com/705011

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2614 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-08-15 11:52:46 +00:00
kjellander@webrtc.org
8ee2750962 Fixes after TBR comments in http://review.webrtc.org/675005
BUG=

Review URL: https://webrtc-codereview.appspot.com/717014

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2613 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-08-15 11:03:29 +00:00
vspasova@webrtc.org
400e7da51b Created a barcode encoder and decoder Python tools.
The barcode encoder generates barcodes as PNG files, than converts them to YUV files,
then stitches these into one YUV video file and finaly overlays this video over the
base video to produce a YUV video of the base with encoded barcodes in the upper part
of the video.

The decoder gets a YUV video with overlaid barcodes and first splits it into PNG files,
than decodes every file, and finally generates a stats file.

The encoder and decoder use Zxing and its Java command-line tools for the encoding and
decoding. They also use ffmpeg for the conversion between PNG and YUV and vice versa.

BUG=
TEST=
The tools could be used from trunk as:
./tools/barcode_tools/barcode_encoder.py --barcode_height=32 --base_frame_width=352
--base_frame_height=288 --base_yuv=<path_nad_name_to_base_file>
--output_yuv=<path_nad_name_to_output_file>

./tools/barcode_tools/barcode_decoder.py --yuv_file=<path_and_name_to_yuv_file>
--yuv_frame_width=352 --yuv=frame_height=288 --barcode_height=32
--stats_file=<path_and_name_of_stats_file>

By default the width of the barcodes is the same as the width of the base YUV frames.
This could be changes with the command-line option --barcode_width

Review URL: https://webrtc-codereview.appspot.com/679010

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2612 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-08-15 10:25:12 +00:00
phoglund@webrtc.org
b64dd31c3c Split bot status list in two rows so it fits better on the screen.
BUG=

Review URL: https://webrtc-codereview.appspot.com/709009

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2590 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-08-10 08:57:39 +00:00
phoglund@webrtc.org
8274bf26ca Added suppression for issue 716.
BUG=
TBR=kjellander@webrtc.org

Review URL: https://webrtc-codereview.appspot.com/719005

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2548 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-08-02 12:25:13 +00:00
andrew@webrtc.org
6f8db36e04 Reorganize voice_engine/.
The usual changes:
voice_engine/main/source -> voice_engine/
voice_engine/main/interface -> voice_engine/include
voice_engine/main/test -> voice_engine/test
Include path changes.

BUG=none
TEST=trybots

Review URL: https://webrtc-codereview.appspot.com/705004

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2535 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-07-27 21:49:28 +00:00
andrew@webrtc.org
d41f59a23f Fix Mac-gcc warnings.
Resolves:
- warning: allocating zero-element array
- warning: suggest a space before ‘;’ or explicit braces around empty
  body in ‘for’ statement

BUG=none
TEST=build on Mac-gcc, trybots

Review URL: https://webrtc-codereview.appspot.com/675006

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2519 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-07-16 17:05:47 +00:00
kjellander@webrtc.org
1bc6d3238c Only status from interesting bots are reported to the Dashboard
TBR=phoglund
BUG=none
TEST=Tested with local modifications on the live installation.

Review URL: https://webrtc-codereview.appspot.com/675005

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2496 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-07-09 09:43:30 +00:00
kjellander@webrtc.org
ba5a97db4a Moving buildbot to internal repository.
BUG=None
TEST=None

Review URL: https://webrtc-codereview.appspot.com/654007

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2428 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-06-20 07:55:41 +00:00
kjellander@webrtc.org
5f9f1db12a This change make PulseAudio only start for the tests on the LinuxLargeTests bot.
I put back the --daemonize flag too, since the audio_e2e_test didn't find the pulseaudio daemon otherwise.

TBR=phoglund@webrtc.org
BUG=None
TEST=Tested locally on LinuxLargeTests bot.

Review URL: https://webrtc-codereview.appspot.com/635009

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2395 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-06-12 08:07:30 +00:00
andrew@webrtc.org
1c284734cd Fix master's "Start PulseAudio" step.
TBR=kjellander@webrtc.org
BUG=none
TEST=manually on the LinuxLargeTests bot.

Review URL: https://webrtc-codereview.appspot.com/632010

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2393 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-06-12 06:15:45 +00:00
andrew@webrtc.org
0594916a21 Add audio_e2e_test to LinuxLargeTests.
Ensure pulseaudio is running on the bot.

BUG=none
TEST=audio_e2e_test
Review URL: https://webrtc-codereview.appspot.com/646004

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2392 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-06-12 03:32:18 +00:00
andrew@webrtc.org
9f6577b304 Restore default source in e2e test.
Also wait for VoE to start up.

TBR=kjellander@webrtc.org
BUG=issue502
TEST=audio_e2e_test

Review URL: https://webrtc-codereview.appspot.com/643006

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2391 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-06-12 03:25:03 +00:00
kjellander@webrtc.org
f08f52f136 Fixing issues with slaves.cfg on Windows.
This fix is needed for our own build slaves to work properly on Windows and is caused by the hacky way we created the Libvpx waterfall to avoid duplicating unnecessary Python code.

TBR=phoglund
BUG=None
TEST=Tested on Windows build slave.

Review URL: https://webrtc-codereview.appspot.com/639009

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2387 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-06-11 08:06:40 +00:00
phoglund@webrtc.org
f7d0c77b48 Added the bitrate estimator test to the trybots.
TBR=kjellander@webrtc.org
BUG=
TEST=

Review URL: https://webrtc-codereview.appspot.com/642005

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2378 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-06-07 09:28:12 +00:00
phoglund@webrtc.org
d2956d8cda Renamed test_bwe.
TBR=kjellander@webrtc.org
BUG=
TEST=

Review URL: https://webrtc-codereview.appspot.com/635007

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2376 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-06-07 08:36:35 +00:00
andrew@webrtc.org
f4c6aa2e81 Improve the reliablity of the audio e2e test.
- Use higher quality resampling.
- Add a longer delay before pacat recording.

TBR=kjellander@webrtc.org
BUG=issue502
TEST=manually

Review URL: https://webrtc-codereview.appspot.com/646005

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2374 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-06-07 04:00:15 +00:00
bjornv@webrtc.org
20e13edede New attempt to revert r2362, since drover failed.
TBR=stefan@webrtc.org
Review URL: https://webrtc-codereview.appspot.com/640005

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2368 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-06-05 13:07:56 +00:00
phoglund@webrtc.org
df373984e2 Renamed test_bwe.
BUG=
TEST=
TBR=kjellander@webrtc.org

Review URL: https://webrtc-codereview.appspot.com/642004

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2364 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-06-05 11:00:24 +00:00
kjellander@webrtc.org
343301ff73 Fixing release compilation on Linux and Mac trybots
BUG=None
TEST=None

Review URL: https://webrtc-codereview.appspot.com/620005

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2342 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-06-01 11:10:24 +00:00
kjellander@webrtc.org
c03df177c6 Enabling audio_coding_module_test on trybots
BUG=None
TEST=None

Review URL: https://webrtc-codereview.appspot.com/615005

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2341 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-06-01 11:09:39 +00:00
kjellander@webrtc.org
29c5a2335c Renamed to Network Emulator and improved error handling.
Changed default start port of the port-range to 32768.

BUG=None
TEST=Tested locally.

Review URL: https://webrtc-codereview.appspot.com/627004

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2338 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-06-01 08:42:17 +00:00
kjellander@webrtc.org
595749f7ec Network simulation script based on Dummynet.
This script only intends to support Mac and Linux so far. Additional coding and conditional checking will be required to support Windows.

BUG=None.
TEST=Tested on Linux and Mac.

Review URL: https://webrtc-codereview.appspot.com/606006

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2335 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-05-31 20:19:05 +00:00
kjellander@webrtc.org
2e84c112f5 Updating bitrate controller tests to test naming conventions.
The test is now named 'bitrate_controller_unittests'.
This CL also enables it on the bots. The test is excluded on ASAN since
it fails when compiled with projects generated with GYP_DEFINES='asan=1' (see issue 555).

BUG=None
TEST=bitrate_controller_unittests was tested in Debug+Release on Linux, Mac and Windows + TSAN and memcheck.

Review URL: https://webrtc-codereview.appspot.com/612004

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2333 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-05-31 13:55:01 +00:00
kjellander@webrtc.org
bb24b14c8a Libvpx waterfall additional changes. The CL http://review.webrtc.org/595005/ was not complete since it didn't put the libvpx waterfall at its own port.
Chrome bots are now using the correct scheduler.
Created separate watchlist e-mail for libvpx builds to avoid spamming WebRTC when these builds fail.

BUG=None
TEST=Tested on local master and slaves.

Review URL: https://webrtc-codereview.appspot.com/620004

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2331 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-05-31 10:42:51 +00:00
phoglund@webrtc.org
9259e7bd03 Added a step for restarting pulseaudio.
BUG=
TEST=

Review URL: https://webrtc-codereview.appspot.com/611007

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2322 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-05-30 09:07:02 +00:00
kjellander@webrtc.org
8a7a019b55 Syncing tests on try master with build master.
Also adding video_codecs_test_framework_integrationtests since it's executing fast enough and catches a lot.

BUG=None
TEST=None

Review URL: https://webrtc-codereview.appspot.com/605006

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2314 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-05-29 14:49:39 +00:00
kjellander@webrtc.org
fe87f8ce08 Libvpx buildbot waterfall.
I think I found a decent way to avoid duplicating the whole configuration for the libvpx waterfall. It's not perfect but it works. I still haven't figured out what the best way to manage the slaves would be however, since they will need to know which master to connect to, or they'll pick the default they're configured for in slaves.cfg, which is the WebRTC waterfall (can be overridden with the TESTING_MASTER property, but that's only to be used for development and testing.

BUG=None
TEST=Tested on local master and slaves.

Review URL: https://webrtc-codereview.appspot.com/595005

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2310 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-05-29 06:23:42 +00:00
kjellander@webrtc.org
b6e4cc776e Valgrind memcheck and TSAN script now uses Chrome+WebRTC suppression files.
Skeleton suppression files for future WebRTC suppressions are added and are included in addition to the ones Chrome are using and maintaining when our wrapper script executes.

Also added tweaked PRESUBMIT checks based on the Chrome code, that verifies
that suppressions are added correctly. I tested that they work by adding an invalid
suppression.

BUG=544
TEST=Tested running tools/valgrind-webrtc/webrtc_tests.sh --tool=tsan -t out/Debug/system_wrappers_unittests and it reports far less errors. Tested adding bad suppression and it was caught by the PRESUBMIT check.

Review URL: https://webrtc-codereview.appspot.com/601004

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2304 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-05-27 20:59:35 +00:00
kjellander@webrtc.org
15cd04a314 Enabling vp8_integrationtests on all platforms
BUG=None
TEST=Tested on Linux, Mac and Windows in Debug+Release

Review URL: https://webrtc-codereview.appspot.com/599004

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2299 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-05-25 09:46:19 +00:00
kjellander@webrtc.org
21c5bda17c Enabling udp_transport_unittests for TSAN
BUG=536
TEST=Tested running the test within TSAN on Linux.

Review URL: https://webrtc-codereview.appspot.com/591004

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2298 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-05-25 09:45:33 +00:00
phoglund@webrtc.org
3c580bed4e Made fuzz tests and chrome bots FYI: e.g, they will warn instead of making the whole build red if they fail.
BUG=
TEST=

Review URL: https://webrtc-codereview.appspot.com/589005

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2286 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-05-24 09:31:49 +00:00
andrew@webrtc.org
aa5f410db5 Add a README to e2e_quality.
TBR=kjellander@webrtc.org
BUG=issue502
TEST=none

Review URL: https://webrtc-codereview.appspot.com/590004

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2278 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-05-23 15:11:20 +00:00
andrew@webrtc.org
2b004655b5 Add comparison to e2e and clean up.
BUG=issue502
TEST=manual

Review URL: https://webrtc-codereview.appspot.com/577006

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2274 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-05-23 02:47:58 +00:00
phoglund@webrtc.org
bd48e71058 Adjusted the bloat build's start time.
BUG=
TEST=

Review URL: https://webrtc-codereview.appspot.com/566015

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2267 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-05-22 09:17:28 +00:00
kjellander@webrtc.org
08d9928443 ThreadSanitizer (TSAN) and AddressSanitizer (ASAN) bots
BUG=None
TEST=Tested on local master with the live slaves.

Review URL: https://webrtc-codereview.appspot.com/571011

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2258 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-05-18 14:33:06 +00:00
kjellander@webrtc.org
aee87f0c87 Updating to new Android trybot machines
BUG=None
TEST=None needed.

Review URL: https://webrtc-codereview.appspot.com/584005

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2257 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-05-18 09:28:58 +00:00
kjellander@webrtc.org
47b4041e92 Make the bot kill VCam properly
Fixing so that VCamManager.exe is properly killed and changed so that
the stepp is not failing if there's nothing to kill.

BUG=None
TEST=Tested successfully on live master and slaves.

Review URL: https://webrtc-codereview.appspot.com/568007

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2255 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-05-17 18:08:04 +00:00
kjellander@webrtc.org
e96b54f442 Adding Mac and Windows Large Tests bots.
This also means the old LinuxVideo bot is renamed to LinuxLargeTests.

BUG=None
TEST=Tested successfully with local buildbot master and remote slaves.

Review URL: https://webrtc-codereview.appspot.com/582005

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2251 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-05-16 12:02:13 +00:00
kjellander@webrtc.org
fbfbdbe9ea Making the old bots also trigger on source changes.
The old scheduler was missing the branch configuration, that was started being used as a consequence of adapting the Chromium Buildbot scripts.

TBR=phoglund
BUG=None
TEST=Tested on live master+slaves with manual edits.

Review URL: https://webrtc-codereview.appspot.com/576012

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2250 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-05-16 12:00:26 +00:00
phoglund@webrtc.org
440ff23969 Fixed some unexpected syntax for chrome bots.
BUG=
TEST=

Review URL: https://webrtc-codereview.appspot.com/569017

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2246 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-05-16 09:05:57 +00:00
kjellander@webrtc.org
c5ad6092fa Added Chrome+WebRTC bots based on Chrome scripts, for all platforms.
Also removed the old Linux Chrome and Chrome Bloat bots from utils.py.

Some of these changes may seem a bit confusing and messy but they must be considered a step on the way to moving away from our inheritance based Factories into the style Chrome users, where most configuration is in the .cfg files, factory setup is in the factories and step details are put into the commands-files.

BUG=None
TEST=Tested with local master and production slaves.

Review URL: https://webrtc-codereview.appspot.com/581006

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2238 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-05-15 09:29:22 +00:00
kjellander@webrtc.org
f6ec0a91a2 Making Coverity Static Analyzer run properly on Windows
BUG=None
TEST=Tested on main Coverity server and a Windows machine running Static Analyzer.

Review URL: https://webrtc-codereview.appspot.com/579004

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2235 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-05-14 14:18:53 +00:00
phoglund@webrtc.org
dedfe4db70 Updated tracker to filter the new chrome bots too.
BUG=
TEST=

Review URL: https://webrtc-codereview.appspot.com/562009

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2232 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-05-11 11:34:29 +00:00
turaj@webrtc.org
799980fcaa Revert 2224 - Revert 2213 - Moved the fuzz test under valgrind and added valgrind support to the video bot.
CL 2213 did not need to be reverted. 

BUG=
TEST=

Review URL: https://webrtc-codereview.appspot.com/579008

TBR=phoglund@webrtc.org
Review URL: https://webrtc-codereview.appspot.com/569013

TBR=turaj@webrtc.org
Review URL: https://webrtc-codereview.appspot.com/576010

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2227 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-05-11 00:41:48 +00:00
turaj@webrtc.org
c2fe91256c Revert 2213 - Moved the fuzz test under valgrind and added valgrind support to the video bot.
A series of CL:s by Patrik W. is breaking the auto-test. It started with CL 2211, but the later CL:s seems dependent on another. So I decided to go in reverse order and revert all of them.

BUG=
TEST=

Review URL: https://webrtc-codereview.appspot.com/579008

TBR=phoglund@webrtc.org
Review URL: https://webrtc-codereview.appspot.com/569013

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2224 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-05-10 22:55:57 +00:00
andrew@webrtc.org
b06db96840 Add a framework for audio end-to-end quality testing.
The quality comparison step is still to be done.

BUG=issue502
TEST=manual

Review URL: https://webrtc-codereview.appspot.com/577005

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2220 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-05-10 18:45:11 +00:00
phoglund@webrtc.org
80f7ac23b0 Moved the fuzz test under valgrind and added valgrind support to the video bot.
BUG=
TEST=

Review URL: https://webrtc-codereview.appspot.com/579008

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2213 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-05-10 14:43:10 +00:00
phoglund@webrtc.org
00a186665c Added link to bloat builds.
BUG=
TEST=

Review URL: https://webrtc-codereview.appspot.com/573004

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2210 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-05-10 07:54:47 +00:00
phoglund@webrtc.org
0b7ba967f5 Fixed comments and added new flags for the PyAuto test.
The ChromeBloat bot will now also be ignored for LKGRs.

BUG=
TEST=

Review URL: https://webrtc-codereview.appspot.com/559004

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2169 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-05-03 14:21:14 +00:00
kjellander@webrtc.org
1e1dd170e0 Disabling PTY for mv command on Android bot.
BUG=None
TEST=Tested on local master with the Android bot slave.

Review URL: https://webrtc-codereview.appspot.com/552004

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2158 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-05-02 07:45:27 +00:00
phoglund@webrtc.org
dbb7f91f36 The ChromeBloat bot will now also be ignored for LKGRs.
BUG=
TEST=

Review URL: https://webrtc-codereview.appspot.com/543004

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2156 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-05-02 06:32:30 +00:00
kjellander@webrtc.org
890aa0d2c0 Disabling HTTP status push on Build master since we're not using it
Also updated .gitignore

BUG=None
TEST=Tested on local master and production master.

Review URL: https://webrtc-codereview.appspot.com/539005

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2147 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-04-30 09:56:57 +00:00
kjellander@webrtc.org
be0ac630a1 Overriding tgrid URL for master web status
The change in r2143 changed so we're reusing Chromium scripts for setting up the web status. However Chrome doesn't use the tgrid page it seems (they put the Console page on that URL instead) but we need it for our LKGR parsing.

BUG=None
TEST=Tested on local master.

Review URL: https://webrtc-codereview.appspot.com/541004

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2146 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-04-30 09:15:57 +00:00
kjellander@webrtc.org
c6c4ffc5d8 Android trybots + fixing web status config
BUG=None
TEST=Tested on local master and slave.

Review URL: https://webrtc-codereview.appspot.com/514001

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2143 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-04-30 07:59:09 +00:00
phoglund@webrtc.org
61331137b6 Reduced the time from check-in to build significantly.
BUG=
TEST=

Review URL: https://webrtc-codereview.appspot.com/525001

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2123 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-04-26 07:32:07 +00:00
phoglund@webrtc.org
6a65cfbb0e Enabled the new PyAuto test on the build bot.
BUG=
TEST=

Review URL: https://webrtc-codereview.appspot.com/520003

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2116 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-04-25 10:11:55 +00:00
kjellander@webrtc.org
b28b43a66f Adding alwaysUseLatest parameter for GClient sync.
Removed confusing use of Subversion URL in places where it doesn't have
any effect.

BUG=None
TEST=Successful build with local master and the Chrome slave.

Review URL: https://webrtc-codereview.appspot.com/521001

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2103 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-04-24 07:48:21 +00:00
kjellander@webrtc.org
7401a1f393 Updating Chrome excludes to use the same as Chrome buildbots.
See the NEEDED_COMPONENTS constant of http://src.chromium.org/viewvc/chrome/trunk/tools/build/scripts/master/factory/chromium_factory.py?view=markup for more details.

BUG=None
TEST=None

Review URL: https://webrtc-codereview.appspot.com/513001

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2099 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-04-23 18:59:40 +00:00
kjellander@webrtc.org
55e4fffa5b LCOV fix in addition to r2095.
https://webrtc-codereview.appspot.com/515002 was not properly tested.

BUG=None
TEST=Tested on local master and slave, verified fix working in twistd.log on master.
TBR=phoglund

Review URL: https://webrtc-codereview.appspot.com/513002

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2097 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-04-23 18:22:20 +00:00
kjellander@webrtc.org
f3794f8e5a Fixed normal LCOV case
BUG=None
TEST=Tested on local master, with and without a test failure.

Review URL: https://webrtc-codereview.appspot.com/515002

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2095 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-04-23 15:42:56 +00:00
phoglund@webrtc.org
f6cd33dd89 Implemented bloat calculation. This will measure the binary size of Chrome+WebRTC components each weekend.
BUG=
TEST=

Review URL: https://webrtc-codereview.appspot.com/508005

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2088 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-04-23 09:27:57 +00:00
kjellander@webrtc.org
39946f1380 Skipping code coverage HTML generation on failed builds.
BUG=None, except that the dashboard shows confusing stats.
TEST=Tested with local master and slave.

Review URL: https://webrtc-codereview.appspot.com/508002

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2087 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-04-23 08:57:44 +00:00
kjellander@webrtc.org
0cac8be82c Fixing e-mail notification for buildbot master
BUG=None
TEST=None (same config works on try master)

Review URL: https://webrtc-codereview.appspot.com/501001

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2066 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-04-19 15:05:53 +00:00
kjellander@webrtc.org
76643d7c62 Enabling fastbuild in GYP define.
The fastbuild flag means debug symbols are not generated. This has no
impact on our tests as long as we don't need to debug with the built
binaries, i.e. is perfect for the buildbots. Compilation is 30% faster
or more.

BUG=None
TEST=Tested on local master and linux, mac and win slaves.

Review URL: https://webrtc-codereview.appspot.com/499001

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2062 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-04-19 07:30:39 +00:00
kjellander@webrtc.org
82d85aeb39 All errors are now printed to stderr instead of stdout.
This will make them easier to detect when running as a cron job.

BUG=None
TEST=None

Review URL: https://webrtc-codereview.appspot.com/482005

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2050 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-04-18 09:47:39 +00:00
phoglund@webrtc.org
c3eb178a0d Will now filter chrome-triggered builds.
BUG=
TEST=

Review URL: https://webrtc-codereview.appspot.com/492009

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2040 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-04-17 11:23:35 +00:00
phoglund@webrtc.org
c440d56e92 Rewired the oath2 symlink and updated tgrid_parser to the new Build Bot version's tgrid syntax.
Added back the gviz_api to the tools deps - will remove it from the main project DEPS in a different patch. Rewired those symlinks too.

Made the build status loader algorithm more scalable. It read in the whole database on each load which is probably unsustainable in the long run. Also, it will now forget bots that have offline for more than ~5 revisions.

Fixed a bug in the coverage tracker.

BUG=
TEST=

Review URL: https://webrtc-codereview.appspot.com/486002

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2039 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-04-17 08:49:10 +00:00
andrew@webrtc.org
336d52d817 Roll Chromium 122775:132375 (current Canary).
Hijacking the gyp inside_chromium_build finally came back to bite us.
Chrome started using it to control the path to the gold linker, which
broke our build. This change removes use of inside_chromium_build from
WebRTC.

If peerconnection rolls past this point, libjingle.gyp will need to be updated.

BUG=
TEST=build on Linux, Mac, Win

Review URL: https://webrtc-codereview.appspot.com/493007

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2038 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-04-17 01:19:27 +00:00
kjellander@webrtc.org
050f212881 Updating clean script for new buildbot configurations.
BUG=None
TEST=None

Review URL: https://webrtc-codereview.appspot.com/485008

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2033 4adac7df-926f-26a2-2b94-8c16560cd09d
2012-04-16 14:41:11 +00:00