Replace scoped_ptr with unique_ptr in webrtc/modules/audio_device/

Replace scoped_ptr with unique_ptr in webrtc/modules/audio_device/

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

[kwiberg-webrtc, 2016-02-24 10:44:49]

Patchset #1 (id:1) has been deleted

[kwiberg-webrtc, 2016-02-24 10:45:12]

Patchset #1 (id:20001) has been deleted

[kwiberg-webrtc, 2016-02-24 10:45:43]

Patchset #1 (id:40001) has been deleted

[kwiberg-webrtc, 2016-02-24 10:55:57]

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

[henrika_webrtc, 2016-02-24 10:59:50]

I am sure this is for a good cause and part of a larger effort, but still, what
are the main benefits?

[kwiberg-webrtc, 2016-02-24 12:04:30]

On 2016/02/24 10:59:50, henrika_webrtc wrote:
> I am sure this is for a good cause and part of a larger effort, but still,
what
> are the main benefits?

scoped_ptr is a home-grown type that's very very close to behaving just like
unique_ptr. The motivation for the change is basically just that it's better for
everyone if we don't use a home-grown type where a standard type will do, since
that means there's less maintenance and fewer project-specific things to learn
about and keep track of.

And yes, this CL is part of a larger effort---I've landed about a dozen of them
thus far, with a few dozen more to go.

[henrika_webrtc, 2016-02-24 12:06:51]

Thanks and LGTM. I will use std::uniqe_ptr in the future then.

[kwiberg-webrtc, 2016-02-24 12:12:01]

On 2016/02/24 12:06:51, henrika_webrtc wrote:
> Thanks and LGTM. I will use std::uniqe_ptr in the future then.

Yes, at some point soon I'll do an announcement. Thought I'd finish more of the
conversion first, though, since there's a small amount of inconvenience if you
end up having to convert between the two.

[kwiberg-webrtc, 2016-02-24 12:12:13]

The CQ bit was checked by kwiberg@webrtc.org

[commit-bot: I haz the power, 2016-02-24 12:12:20]

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

[commit-bot: I haz the power, 2016-02-24 13:00:41]

Committed patchset #1 (id:60001)

[commit-bot: I haz the power, 2016-02-24 13:00:50]

Description was changed from

==========
Replace scoped_ptr with unique_ptr in webrtc/modules/audio_device/

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

to

==========
Replace scoped_ptr with unique_ptr in webrtc/modules/audio_device/

BUG=webrtc:5520

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

[commit-bot: I haz the power, 2016-02-24 13:00:51]

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