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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 2295283004: Remove unused audio_e2e_harness.cc and infrastructure. (Closed)

Created:
4 years, 3 months ago by the sun
Modified:
4 years, 3 months ago
Reviewers:
phoglund
CC:
webrtc-reviews_webrtc.org, tterriberry_mozilla.com
Target Ref:
refs/pending/heads/master
Project:
webrtc
Visibility:
Public.

Description

Remove unused audio_e2e_harness.cc and infrastructure. BUG=webrtc:4690 Committed: https://crrev.com/96f2c4df316769806d11e25cfbf698a5cf501584 Cr-Commit-Position: refs/heads/master@{#14165}

Patch Set 1 #

Patch Set 2 : misc #

Patch Set 3 : rebase #

Patch Set 4 : rebase #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -312 lines) Patch
M webrtc/tools/BUILD.gn View 1 2 1 chunk +0 lines, -22 lines 0 comments Download
M webrtc/tools/DEPS View 1 2 3 1 chunk +0 lines, -1 line 0 comments Download
D webrtc/tools/e2e_quality/audio/README View 1 chunk +0 lines, -27 lines 0 comments Download
D webrtc/tools/e2e_quality/audio/audio_e2e_harness.cc View 1 chunk +0 lines, -109 lines 0 comments Download
D webrtc/tools/e2e_quality/audio/daemon.conf View 1 chunk +0 lines, -1 line 0 comments Download
D webrtc/tools/e2e_quality/audio/default.pa View 1 chunk +0 lines, -6 lines 0 comments Download
D webrtc/tools/e2e_quality/audio/perf View 1 chunk +0 lines, -1 line 0 comments Download
D webrtc/tools/e2e_quality/audio/run_audio_test.py View 1 chunk +0 lines, -131 lines 0 comments Download
M webrtc/tools/tools.gyp View 1 2 1 chunk +0 lines, -14 lines 0 comments Download

Dependent Patchsets:

Issue 2319583005 Patch 1

Messages

Total messages: 16 (7 generated)
the sun
4 years, 3 months ago (2016-09-08 12:30:54 UTC) #2
phoglund
lgtm
4 years, 3 months ago (2016-09-08 13:58:35 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/2295283004/60001
4 years, 3 months ago (2016-09-08 14:11:15 UTC) #5
commit-bot: I haz the power
Try jobs failed on following builders: android_rel on master.tryserver.webrtc (JOB_FAILED, http://build.chromium.org/p/tryserver.webrtc/builders/android_rel/builds/16383)
4 years, 3 months ago (2016-09-08 15:15:07 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/2295283004/60001
4 years, 3 months ago (2016-09-09 07:25:43 UTC) #9
commit-bot: I haz the power
Try jobs failed on following builders: android_rel on master.tryserver.webrtc (JOB_FAILED, http://build.chromium.org/p/tryserver.webrtc/builders/android_rel/builds/16410)
4 years, 3 months ago (2016-09-09 07:38:04 UTC) #11
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/2295283004/60001
4 years, 3 months ago (2016-09-09 12:03:45 UTC) #13
commit-bot: I haz the power
Committed patchset #4 (id:60001)
4 years, 3 months ago (2016-09-09 13:51:21 UTC) #14
commit-bot: I haz the power
4 years, 3 months ago (2016-09-09 13:51:31 UTC) #16
Message was sent while issue was closed. 
Patchset 4 (id:??) landed as
https://crrev.com/96f2c4df316769806d11e25cfbf698a5cf501584
Cr-Commit-Position: refs/heads/master@{#14165}

Powered by Google App Engine
This is Rietveld 408576698