Chromium Code Reviews

Issue 2785363002: Fixing DTX unittest for AudioEncoderOpus. (Closed)

Created:
3 years, 8 months ago by minyue-webrtc
Modified:
3 years, 8 months ago
Reviewers:
squashingskak, kwiberg-webrtc
CC:
webrtc-reviews_webrtc.org, AleBzk, peah-webrtc, tlegrand-webrtc, tterriberry_mozilla.com, audio-team_agora.io, hlundin-webrtc, kwiberg-webrtc, minyue-webrtc
Target Ref:
refs/heads/master
Project:
webrtc
Visibility:
Public.

Description

Fixing DTX unittest for AudioEncoderOpus. BUG=None Review-Url: https://codereview.webrtc.org/2785363002 Cr-Commit-Position: refs/heads/master@{#17520} Committed: https://chromium.googlesource.com/external/webrtc/+/3153c6dce55c5fcefc48ae2f2e75e1e872e828f8

Patch Set 1 #

Unified diffs Side-by-side diffs Stats (+2 lines, -2 lines)
M webrtc/modules/audio_coding/codecs/opus/audio_encoder_opus_unittest.cc View 1 chunk +2 lines, -2 lines 0 comments

Messages

Total messages: 9 (5 generated)
minyue-webrtc
This is related to a comment in https://codereview.webrtc.org/2772773002/ PTAL.
3 years, 8 months ago (2017-03-30 22:56:45 UTC) #3
kwiberg-webrtc
lgtm
3 years, 8 months ago (2017-03-31 09:15:13 UTC) #4
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/2785363002/1
3 years, 8 months ago (2017-04-04 06:12:28 UTC) #6
commit-bot: I haz the power
3 years, 8 months ago (2017-04-04 06:34:43 UTC) #9
Message was sent while issue was closed. 
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/external/webrtc/+/3153c6dce55c5fcefc48ae2f2...

Powered by Google App Engine