Replace scoped_ptr with unique_ptr in webrtc/modules/audio_processing/vad/

Replace scoped_ptr with unique_ptr in webrtc/modules/audio_processing/vad/

BUG=webrtc:5520
Committed: https://crrev.com/dabf07f4776e3f0658fecd890da0bd55ce5fe6ff
Cr-Commit-Position: refs/heads/master@{#11657}

[kwiberg-webrtc, 2016-02-16 13:27:27]

Description was changed from

==========
Replace scoped_ptr with unique_ptr in webrtc/modules/audio_processing/vad/

BUG=webrtc:5520
==========

to

==========
Replace scoped_ptr with unique_ptr in webrtc/modules/audio_processing/vad/

BUG=webrtc:5520
==========

[kwiberg-webrtc, 2016-02-16 13:27:28]

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

[peah-webrtc, 2016-02-17 11:54:02]

https://codereview.webrtc.org/1699003002/diff/1/webrtc/modules/audio_processi...
File webrtc/modules/audio_processing/vad/common.h (right):

https://codereview.webrtc.org/1699003002/diff/1/webrtc/modules/audio_processi...
webrtc/modules/audio_processing/vad/common.h:15: 
Why is this include needed in the common file? Is it not sufficient to add
includes in the other files?

[kwiberg-webrtc, 2016-02-17 12:13:19]

https://codereview.webrtc.org/1699003002/diff/1/webrtc/modules/audio_processi...
File webrtc/modules/audio_processing/vad/common.h (right):

https://codereview.webrtc.org/1699003002/diff/1/webrtc/modules/audio_processi...
webrtc/modules/audio_processing/vad/common.h:15: 
On 2016/02/17 11:54:01, peah-webrtc wrote:
> Why is this include needed in the common file? Is it not sufficient to add
> includes in the other files?

We get size_t from there. Some other change in this CL made it so that stddef.h
was no longer included before vad/common.h by everyone, which triggered the
latent bug.

[peah-webrtc, 2016-02-17 12:32:28]

On 2016/02/17 12:13:19, kwiberg-webrtc wrote:
>
https://codereview.webrtc.org/1699003002/diff/1/webrtc/modules/audio_processi...
> File webrtc/modules/audio_processing/vad/common.h (right):
> 
>
https://codereview.webrtc.org/1699003002/diff/1/webrtc/modules/audio_processi...
> webrtc/modules/audio_processing/vad/common.h:15: 
> On 2016/02/17 11:54:01, peah-webrtc wrote:
> > Why is this include needed in the common file? Is it not sufficient to add
> > includes in the other files?
> 
> We get size_t from there. Some other change in this CL made it so that
stddef.h
> was no longer included before vad/common.h by everyone, which triggered the
> latent bug.

lgtm

[kwiberg-webrtc, 2016-02-17 12:44:03]

The CQ bit was checked by kwiberg@webrtc.org

[commit-bot: I haz the power, 2016-02-17 12:44:21]

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

[commit-bot: I haz the power, 2016-02-17 14:44:51]

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

[commit-bot: I haz the power, 2016-02-17 14:44:52]

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

[kwiberg-webrtc, 2016-02-17 14:59:37]

The CQ bit was checked by kwiberg@webrtc.org

[commit-bot: I haz the power, 2016-02-17 14:59:44]

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

[commit-bot: I haz the power, 2016-02-17 15:59:50]

Description was changed from

==========
Replace scoped_ptr with unique_ptr in webrtc/modules/audio_processing/vad/

BUG=webrtc:5520
==========

to

==========
Replace scoped_ptr with unique_ptr in webrtc/modules/audio_processing/vad/

BUG=webrtc:5520
==========

[commit-bot: I haz the power, 2016-02-17 15:59:50]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-02-17 15:59:58]

Description was changed from

==========
Replace scoped_ptr with unique_ptr in webrtc/modules/audio_processing/vad/

BUG=webrtc:5520
==========

to

==========
Replace scoped_ptr with unique_ptr in webrtc/modules/audio_processing/vad/

BUG=webrtc:5520

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

[commit-bot: I haz the power, 2016-02-17 15:59:59]

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