Convert //services from scoped_ptr to std::unique_ptr

Convert //services from scoped_ptr to std::unique_ptr

BUG=554298
R=ben@chromium.org,jam@chromium.org

Committed: https://crrev.com/39fafebd8ff93ca8aac361926e6b3ef840302328
Cr-Commit-Position: refs/heads/master@{#388610}

[dcheng, 2016-04-20 22:51:36]

Description was changed from

==========
Convert //services from scoped_ptr to std::unique_ptr

BUG=554298
R=ben@chromium.org,erg@chromium.org
==========

to

==========
Convert //services from scoped_ptr to std::unique_ptr

BUG=554298
R=ben@chromium.org,jam@chromium.org
==========

[dcheng, 2016-04-20 22:51:36]

dcheng@chromium.org changed reviewers:
+ jam@chromium.org
- erg@chromium.org

[dcheng, 2016-04-20 22:52:01]

Incidentally, should //services have its own OWNERS file?

[jam, 2016-04-20 22:54:44]

lgtm

On 2016/04/20 22:52:01, dcheng wrote:
> Incidentally, should //services have its own OWNERS file?

I think the argument would be that no one person would be familiar enough with
reviews there, as each service is quite different. We should just ensure each
service has its own owners file.

[jam, 2016-04-20 22:55:55]

On 2016/04/20 22:54:44, jam wrote:
> lgtm
> 
> On 2016/04/20 22:52:01, dcheng wrote:
> > Incidentally, should //services have its own OWNERS file?
> 
> I think the argument would be that no one person would be familiar enough with
> reviews there, as each service is quite different. We should just ensure each
> service has its own owners file.

I can take care of this.

[dcheng, 2016-04-20 22:57:28]

On 2016/04/20 at 22:54:44, jam wrote:
> lgtm
> 
> On 2016/04/20 22:52:01, dcheng wrote:
> > Incidentally, should //services have its own OWNERS file?
> 
> I think the argument would be that no one person would be familiar enough with
reviews there, as each service is quite different. We should just ensure each
service has its own owners file.

Ah, so kind of like //components. All the current services look like things that
//mojo/OWNERS would be familiar with, so would it be appropriate to seed it with
that list?

[dcheng, 2016-04-20 22:57:32]

The CQ bit was checked by dcheng@chromium.org

[commit-bot: I haz the power, 2016-04-20 22:58:03]

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

[Ben Goodger (Google), 2016-04-20 23:12:39]

lgtm 2

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

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-22 19:27:59]

Description was changed from

==========
Convert //services from scoped_ptr to std::unique_ptr

BUG=554298
R=ben@chromium.org,jam@chromium.org
==========

to

==========
Convert //services from scoped_ptr to std::unique_ptr

BUG=554298
R=ben@chromium.org,jam@chromium.org

Committed: https://crrev.com/39fafebd8ff93ca8aac361926e6b3ef840302328
Cr-Commit-Position: refs/heads/master@{#388610}
==========

[commit-bot: I haz the power, 2016-04-22 19:28:00]

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