webrtc/modules/audio_processing: Use RTC_DCHECK() instead of assert()

webrtc/modules/audio_processing: Use RTC_DCHECK() instead of assert()

Committed: https://crrev.com/9e2be5f292c574ca88e145ebb15b0ffd63f9b87c
Cr-Commit-Position: refs/heads/master@{#14211}

[kwiberg-webrtc, 2016-09-12 09:09:52]

Patchset #1 (id:1) has been deleted

[kwiberg-webrtc, 2016-09-12 09:11:10]

kwiberg@webrtc.org changed reviewers:
+ aleloi@webrtc.org

[aleloi, 2016-09-12 10:04:56]

lgtm with nit.

https://codereview.webrtc.org/2320053003/diff/20001/webrtc/modules/audio_proc...
File webrtc/modules/audio_processing/agc/agc_manager_direct.cc (right):

https://codereview.webrtc.org/2320053003/diff/20001/webrtc/modules/audio_proc...
webrtc/modules/audio_processing/agc/agc_manager_direct.cc:13: #include <cassert>
This should be removed as well.

[kwiberg-webrtc, 2016-09-13 11:15:00]

New patch set uploaded.

https://codereview.webrtc.org/2320053003/diff/20001/webrtc/modules/audio_proc...
File webrtc/modules/audio_processing/agc/agc_manager_direct.cc (right):

https://codereview.webrtc.org/2320053003/diff/20001/webrtc/modules/audio_proc...
webrtc/modules/audio_processing/agc/agc_manager_direct.cc:13: #include <cassert>
On 2016/09/12 10:04:56, aleloi wrote:
> This should be removed as well.

Thanks.

[kwiberg-webrtc, 2016-09-13 11:15:09]

The CQ bit was checked by kwiberg@webrtc.org

[kwiberg-webrtc, 2016-09-13 11:15:10]

The patchset sent to the CQ was uploaded after l-g-t-m from aleloi@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2320053003/#ps40001 (title:
"Remove #include <cassert>")

[commit-bot: I haz the power, 2016-09-13 11:15:17]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2016-09-13 11:16:44]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-13 11:16:45]

Try jobs failed on following builders:
  mac_compile_dbg on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/mac_compile_dbg/builds/...)
  mac_gyp_dbg on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/mac_gyp_dbg/builds/554)
  mac_gyp_rel on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/mac_gyp_rel/builds/559)

[kwiberg-webrtc, 2016-09-13 11:24:30]

The CQ bit was checked by kwiberg@webrtc.org

[kwiberg-webrtc, 2016-09-13 11:24:31]

The patchset sent to the CQ was uploaded after l-g-t-m from aleloi@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2320053003/#ps60001 (title:
"rebase")

[commit-bot: I haz the power, 2016-09-13 11:24:36]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2016-09-13 13:24:59]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-13 13:25:00]

Try jobs failed on following builders:
  android_arm64_rel on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)
  android_dbg on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)
  android_rel on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)
  win_clang_dbg on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)
  win_dbg on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)

[kwiberg-webrtc, 2016-09-14 12:03:34]

The CQ bit was checked by kwiberg@webrtc.org

[commit-bot: I haz the power, 2016-09-14 12:03:44]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2016-09-14 12:23:26]

Committed patchset #3 (id:60001)

[commit-bot: I haz the power, 2016-09-14 12:23:35]

Description was changed from

==========
webrtc/modules/audio_processing: Use RTC_DCHECK() instead of assert()
==========

to

==========
webrtc/modules/audio_processing: Use RTC_DCHECK() instead of assert()

Committed: https://crrev.com/9e2be5f292c574ca88e145ebb15b0ffd63f9b87c
Cr-Commit-Position: refs/heads/master@{#14211}
==========

[commit-bot: I haz the power, 2016-09-14 12:23:36]

Patchset 3 (id:??) landed as
https://crrev.com/9e2be5f292c574ca88e145ebb15b0ffd63f9b87c
Cr-Commit-Position: refs/heads/master@{#14211}