Changed the delay estimator to be built using C++

Changed the delay estimator to be built using C++

BUG=webrtc:5724
NOPRESUBMIT=true

Committed: https://crrev.com/bdb7af692f48d4cf2a2e7d13c053facdc57df99a
Cr-Commit-Position: refs/heads/master@{#12336}

[peah-webrtc, 2016-04-11 15:48:40]

Patchset #2 (id:20001) has been deleted

[peah-webrtc, 2016-04-11 15:49:44]

Description was changed from

==========
Changed the delay estimator to be built using C++
Changed file endings to .cc and updated the buildfiles accordingly.

BUG=webrtc:5724
==========

to

==========
Changed the delay estimator to be built using C++

BUG=webrtc:5724
==========

[peah-webrtc, 2016-04-11 15:50:25]

peah@webrtc.org changed reviewers:
+ henrik.lundin@webrtc.org, ivoc@webrtc.org

[peah-webrtc, 2016-04-11 15:57:13]

The CQ bit was checked by peah@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2016-04-11 15:57:18]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1878613002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1878613002/40001

[commit-bot: I haz the power, 2016-04-11 16:13:31]

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

[commit-bot: I haz the power, 2016-04-11 16:13:32]

Dry run: Try jobs failed on following builders:
  presubmit on tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/presubmit/builds/4717)

[hlundin-webrtc, 2016-04-12 06:54:12]

LGTM with one comment.

https://codereview.webrtc.org/1878613002/diff/40001/webrtc/modules/audio_proc...
File webrtc/modules/audio_processing/aecm/aecm_core.cc (right):

https://codereview.webrtc.org/1878613002/diff/40001/webrtc/modules/audio_proc...
webrtc/modules/audio_processing/aecm/aecm_core.cc:21: #include
"webrtc/system_wrappers/include/cpu_features_wrapper.h"
Wrong order of includes.

[ivoc, 2016-04-12 07:26:45]

LGTM.

[peah-webrtc, 2016-04-12 11:53:40]

https://codereview.webrtc.org/1878613002/diff/40001/webrtc/modules/audio_proc...
File webrtc/modules/audio_processing/aecm/aecm_core.cc (right):

https://codereview.webrtc.org/1878613002/diff/40001/webrtc/modules/audio_proc...
webrtc/modules/audio_processing/aecm/aecm_core.cc:21: #include
"webrtc/system_wrappers/include/cpu_features_wrapper.h"
On 2016/04/12 06:54:12, hlundin-webrtc wrote:
> Wrong order of includes.

Done.

[peah-webrtc, 2016-04-12 11:53:48]

The CQ bit was checked by peah@webrtc.org

[peah-webrtc, 2016-04-12 11:53:48]

The patchset sent to the CQ was uploaded after l-g-t-m from ivoc@webrtc.org,
henrik.lundin@webrtc.org
Link to the patchset: https://codereview.webrtc.org/1878613002/#ps80001 (title:
"Rebase with latest master")

[commit-bot: I haz the power, 2016-04-12 11:53:59]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1878613002/80001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1878613002/80001

[commit-bot: I haz the power, 2016-04-12 12:00:06]

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

[commit-bot: I haz the power, 2016-04-12 12:00:07]

Try jobs failed on following builders:
  presubmit on tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/presubmit/builds/4743)

[peah-webrtc, 2016-04-12 13:15:46]

Description was changed from

==========
Changed the delay estimator to be built using C++

BUG=webrtc:5724
==========

to

==========
Changed the delay estimator to be built using C++

BUG=webrtc:5724
NOPRESUBMIT=true
==========

[peah-webrtc, 2016-04-12 13:15:59]

The CQ bit was checked by peah@webrtc.org

[commit-bot: I haz the power, 2016-04-12 13:16:03]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1878613002/80001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1878613002/80001

[commit-bot: I haz the power, 2016-04-12 13:54:31]

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

[commit-bot: I haz the power, 2016-04-12 13:54:33]

Try jobs failed on following builders:
  win_baremetal on tryserver.webrtc (JOB_TIMED_OUT, no build URL)

[peah-webrtc, 2016-04-12 21:47:18]

The CQ bit was checked by peah@webrtc.org

[commit-bot: I haz the power, 2016-04-12 21:47:23]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1878613002/80001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1878613002/80001

[commit-bot: I haz the power, 2016-04-12 21:47:47]

Description was changed from

==========
Changed the delay estimator to be built using C++

BUG=webrtc:5724
NOPRESUBMIT=true
==========

to

==========
Changed the delay estimator to be built using C++

BUG=webrtc:5724
NOPRESUBMIT=true
==========

[commit-bot: I haz the power, 2016-04-12 21:47:48]

Committed patchset #4 (id:80001)

[commit-bot: I haz the power, 2016-04-12 21:47:53]

Description was changed from

==========
Changed the delay estimator to be built using C++

BUG=webrtc:5724
NOPRESUBMIT=true
==========

to

==========
Changed the delay estimator to be built using C++

BUG=webrtc:5724
NOPRESUBMIT=true

Committed: https://crrev.com/bdb7af692f48d4cf2a2e7d13c053facdc57df99a
Cr-Commit-Position: refs/heads/master@{#12336}
==========

[commit-bot: I haz the power, 2016-04-12 21:47:54]

Patchset 4 (id:??) landed as
https://crrev.com/bdb7af692f48d4cf2a2e7d13c053facdc57df99a
Cr-Commit-Position: refs/heads/master@{#12336}