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

Issue 17466002: Add WebRTC test configurations for Android testing. (Closed)

Created:
7 years, 6 months ago by kjellander_chromium
Modified:
7 years, 4 months ago
Reviewers:
Isaac (away)
CC:
chromium-reviews, craigdh+watch_chromium.org, bulach+watch_chromium.org, yfriedman+watch_chromium.org, ilevy-cc_chromium.org, klundberg+watch_chromium.org, frankf+watch_chromium.org, hellner1
Visibility:
Public.

Description

Add WebRTC test configurations for Android testing. BUG=webrtc:1882 TEST=From a recent WebRTC checkout: source build/android/envsetup.sh gclient runhooks ninja -C out/Debug build/android/run_test.py -s webrtc I also successful ran builder and tester with the buildbot configuration in https://codereview.chromium.org/17390019 using a locally patched checkout. Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=216920

Patch Set 1 #

Patch Set 2 : Rebased #

Total comments: 4

Patch Set 3 : Fixed nit and added real tests to the suite. #

Patch Set 4 : Rebased #

Patch Set 5 : Fixed accidental char removal #

Unified diffs Side-by-side diffs Delta from patch set Stats (+25 lines, -2 lines) Patch
M build/android/buildbot/bb_device_steps.py View 1 2 3 2 chunks +8 lines, -2 lines 0 comments Download
M build/android/buildbot/bb_run_bot.py View 1 2 3 2 chunks +2 lines, -0 lines 0 comments Download
M build/android/pylib/gtest/gtest_config.py View 1 2 3 4 1 chunk +15 lines, -0 lines 0 comments Download

Messages

Total messages: 9 (0 generated)
kjellander_chromium
Finally we're getting close to run tests on Android devices!
7 years, 6 months ago (2013-06-19 14:15:25 UTC) #1
Isaac (away)
looks good so far!
7 years, 4 months ago (2013-07-31 06:27:23 UTC) #2
kjellander_chromium
On 2013/07/31 06:27:23, Isaac wrote: > looks good so far! +hellner So, what do I ...
7 years, 4 months ago (2013-07-31 18:48:36 UTC) #3
Isaac (away)
It looked like the CL was incomplete? If it is ready this LGTM. https://chromiumcodereview.appspot.com/17466002/diff/3001/build/android/buildbot/bb_device_steps.py File ...
7 years, 4 months ago (2013-07-31 21:49:08 UTC) #4
kjellander_chromium
Since http://webrtc-codereview.appspot.com/1872004/ is now landed, I added the real tests (I planned to do that ...
7 years, 4 months ago (2013-08-05 11:24:03 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/kjellander@chromium.org/17466002/14001
7 years, 4 months ago (2013-08-10 14:04:08 UTC) #6
commit-bot: I haz the power
Retried try job too often on chromium_presubmit for step(s) presubmit http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=chromium_presubmit&number=20002
7 years, 4 months ago (2013-08-10 14:18:57 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/kjellander@chromium.org/17466002/23001
7 years, 4 months ago (2013-08-10 19:28:44 UTC) #8
commit-bot: I haz the power
7 years, 4 months ago (2013-08-12 05:00:56 UTC) #9
Message was sent while issue was closed.
Change committed as 216920

Powered by Google App Engine
This is Rietveld 408576698