|
|
Created:
3 years, 7 months ago by aleloi Modified:
3 years, 6 months ago Reviewers:
AleBzk CC:
webrtc-reviews_webrtc.org, yujie_mao (webrtc), kwiberg-webrtc, Andrew MacDonald, aleloi, hlundin-webrtc, tterriberry_mozilla.com, audio-team_agora.io, qiang.lu, niklas.enbom, peah-webrtc, minyue-webrtc, the sun, AleBzk, aluebs-webrtc, bjornv1 Target Ref:
refs/heads/master Project:
webrtc Visibility:
Public. |
DescriptionChange existing aec dump tests to use webrtc::AecDump.
Currently the debug dump functionality of WebRTC (a log of all
AudioProcessing operations) was tested by the following tests:
1. ApmTest.VerifyDebugDump* which configures and runs AudioProcessing
from a debug dump, and verifies that the same debug dump is
recorded.
2. DebugDumpTest.* which is a comprehensive test of the debug dump
operations. AudioProcessing configuration is changed, and the dump
is scanned for the change.
3. ApmTest::{DebugDump, DebugDumpFromFileHandle} that verify that
debug dumping can be started and files written.
This CL replaces the debug dump mechanism in all these tests to
webrtc::AecDump. Some of the tests are adapted to the chenges of the
new API to AecDump {Start,Stop}DebugRecording: the old functions
signal errors when a file cannot be opened. With AecDump, the
AecDumpFactory instead returns a nullptr.
The CL also changes audioproc_f to use AecDump.
BUG=webrtc:7404
Review-Url: https://codereview.webrtc.org/2864373002
Cr-Commit-Position: refs/heads/master@{#18605}
Committed: https://chromium.googlesource.com/external/webrtc/+/f4dd191b288830be57787b547c05094652c2171c
Patch Set 1 #Patch Set 2 : Mini-change, forgot about DCHECK. #
Messages
Total messages: 73 (69 generated)
The CQ bit was checked by aleloi@webrtc.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: Try jobs failed on following builders: presubmit on master.tryserver.webrtc (JOB_FAILED, http://build.chromium.org/p/tryserver.webrtc/builders/presubmit/builds/16838)
The CQ bit was checked by aleloi@webrtc.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...
The CQ bit was checked by aleloi@webrtc.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...
The CQ bit was checked by aleloi@webrtc.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: Try jobs failed on following builders: mac_baremetal on master.tryserver.webrtc (JOB_FAILED, http://build.chromium.org/p/tryserver.webrtc/builders/mac_baremetal/builds/21332)
The CQ bit was checked by aleloi@webrtc.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: Try jobs failed on following builders: win_dbg on master.tryserver.webrtc (JOB_FAILED, http://build.chromium.org/p/tryserver.webrtc/builders/win_dbg/builds/18775)
The CQ bit was checked by aleloi@webrtc.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...
The CQ bit was checked by aleloi@webrtc.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...
The CQ bit was checked by aleloi@webrtc.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...
Patchset #7 (id:120001) has been deleted
Patchset #6 (id:100001) has been deleted
Patchset #5 (id:80001) has been deleted
Patchset #4 (id:60001) has been deleted
Patchset #3 (id:40001) has been deleted
Patchset #2 (id:20001) has been deleted
Patchset #1 (id:1) has been deleted
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: Try jobs failed on following builders: mac_rel on master.tryserver.webrtc (JOB_FAILED, http://build.chromium.org/p/tryserver.webrtc/builders/mac_rel/builds/25694)
The CQ bit was checked by aleloi@webrtc.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...
Description was changed from ========== Non-virtual AecDump with tests. BUG=Add later. ========== to ========== Non-virtual AecDump with tests. BUG=Add later. ==========
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: Try jobs failed on following builders: linux_ubsan on master.tryserver.webrtc (JOB_FAILED, http://build.chromium.org/p/tryserver.webrtc/builders/linux_ubsan/builds/12291) linux_ubsan_vptr on master.tryserver.webrtc (JOB_FAILED, http://build.chromium.org/p/tryserver.webrtc/builders/linux_ubsan_vptr/builds...)
The CQ bit was checked by aleloi@webrtc.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: This issue passed the CQ dry run.
The CQ bit was checked by aleloi@webrtc.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...
Description was changed from ========== Non-virtual AecDump with tests. BUG=Add later. ========== to ========== AecDump with tests. BUG=Add later. ==========
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: This issue passed the CQ dry run.
The CQ bit was checked by aleloi@webrtc.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: This issue passed the CQ dry run.
Patchset #1 (id:140001) has been deleted
Patchset #4 (id:220001) has been deleted
Patchset #1 (id:160001) has been deleted
Patchset #2 (id:200001) has been deleted
Patchset #1 (id:180001) has been deleted
The CQ bit was checked by aleloi@webrtc.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: This issue passed the CQ dry run.
The CQ bit was checked by aleloi@webrtc.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...
Description was changed from ========== AecDump with tests. BUG=Add later. ========== to ========== Change existing aec dump tests to use webrtc::AecDump. Currently the debug dump functionality of WebRTC (a log of all AudioProcessing operations) was tested by the following tests: 1. ApmTest.VerifyDebugDump* which configures and runs AudioProcessing from a debug dump, and verifies that the same debug dump is recorded. 2. DebugDumpTest.* which is a comprehensive test of the debug dump operations. AudioProcessing configuration is changed, and the dump is scanned for the change. 3. ApmTest::{DebugDump, DebugDumpFromFileHandle} that verify that debug dumping can be started and files written. This CL replaces the debug dump mechanism in all these tests to webrtc::AecDump. Some of the tests are adapted to the chenges of the new API to AecDump {Start,Stop}DebugRecording: the old functions signal errors when a file cannot be opened. With AecDump, the AecDumpFactory instead returns a nullptr. The CL also changes audioproc_f to use AecDump. BUG=webrtc:7404 ==========
aleloi@webrtc.org changed reviewers: + alessiob@webrtc.org
Hi! Can you PTAL on this CL? It's cleanup from another API change.
Patchset #1 (id:240001) has been deleted
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: Try jobs failed on following builders: linux_msan on master.tryserver.webrtc (JOB_FAILED, http://build.chromium.org/p/tryserver.webrtc/builders/linux_msan/builds/21081)
LGTM
The CQ bit was checked by aleloi@webrtc.org
The patchset sent to the CQ was uploaded after l-g-t-m from alessiob@webrtc.org Link to the patchset: https://codereview.webrtc.org/2864373002/#ps280001 (title: "Mini-change, forgot about DCHECK.")
CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...
CQ is committing da patch. Bot data: {"patchset_id": 280001, "attempt_start_ts": 1497515600002480, "parent_rev": "af66f2ca8a3609184518af0d5083f9ee492ec20b", "commit_rev": "f4dd191b288830be57787b547c05094652c2171c"}
Message was sent while issue was closed.
Description was changed from ========== Change existing aec dump tests to use webrtc::AecDump. Currently the debug dump functionality of WebRTC (a log of all AudioProcessing operations) was tested by the following tests: 1. ApmTest.VerifyDebugDump* which configures and runs AudioProcessing from a debug dump, and verifies that the same debug dump is recorded. 2. DebugDumpTest.* which is a comprehensive test of the debug dump operations. AudioProcessing configuration is changed, and the dump is scanned for the change. 3. ApmTest::{DebugDump, DebugDumpFromFileHandle} that verify that debug dumping can be started and files written. This CL replaces the debug dump mechanism in all these tests to webrtc::AecDump. Some of the tests are adapted to the chenges of the new API to AecDump {Start,Stop}DebugRecording: the old functions signal errors when a file cannot be opened. With AecDump, the AecDumpFactory instead returns a nullptr. The CL also changes audioproc_f to use AecDump. BUG=webrtc:7404 ========== to ========== Change existing aec dump tests to use webrtc::AecDump. Currently the debug dump functionality of WebRTC (a log of all AudioProcessing operations) was tested by the following tests: 1. ApmTest.VerifyDebugDump* which configures and runs AudioProcessing from a debug dump, and verifies that the same debug dump is recorded. 2. DebugDumpTest.* which is a comprehensive test of the debug dump operations. AudioProcessing configuration is changed, and the dump is scanned for the change. 3. ApmTest::{DebugDump, DebugDumpFromFileHandle} that verify that debug dumping can be started and files written. This CL replaces the debug dump mechanism in all these tests to webrtc::AecDump. Some of the tests are adapted to the chenges of the new API to AecDump {Start,Stop}DebugRecording: the old functions signal errors when a file cannot be opened. With AecDump, the AecDumpFactory instead returns a nullptr. The CL also changes audioproc_f to use AecDump. BUG=webrtc:7404 Review-Url: https://codereview.webrtc.org/2864373002 Cr-Commit-Position: refs/heads/master@{#18605} Committed: https://chromium.googlesource.com/external/webrtc/+/f4dd191b288830be57787b547... ==========
Message was sent while issue was closed.
Committed patchset #2 (id:280001) as https://chromium.googlesource.com/external/webrtc/+/f4dd191b288830be57787b547... |