sk_make_sp<T>()

sk_make_sp<T>()

because
    auto atm = sk_make_sp<SkAdvancedTypefaceMetrics>();
is better than
    sk_sp<SkAdvancedTypefaceMetrics> atm(new SkAdvancedTypefaceMetrics);

GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1753973002

Committed: https://skia.googlesource.com/skia/+/217c0b3f137d824413e806759bc25378abbe8c18

[hal.canary, 2016-03-01 22:51:16]

Description was changed from

==========
sk_make_sp<T>()
==========

to

==========
sk_make_sp<T>()
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[hal.canary, 2016-03-01 22:52:10]

halcanary@google.com changed reviewers:
+ mtklein@google.com, reed@google.com

[hal.canary, 2016-03-01 22:52:10]

PTAL

[mtklein, 2016-03-01 23:00:03]

mtklein@google.com changed reviewers:
- mtklein@google.com

[mtklein, 2016-03-01 23:00:04]

impl seems fine to me.  taking myself out of any naming bikeshed...

[hal.canary, 2016-03-01 23:16:06]

Description was changed from

==========
sk_make_sp<T>()
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
sk_make_sp<T>()

because
    auto atm = sk_make_sp<SkAdvancedTypefaceMetrics>();
is better than
    sk_sp<SkAdvancedTypefaceMetrics> atm(new SkAdvancedTypefaceMetrics);

GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[hal.canary, 2016-03-01 23:16:46]

On 2016/03/01 at 23:00:04, mtklein wrote:
> impl seems fine to me.  taking myself out of any naming bikeshed...

I don't care about names either, as long as it is short.

[reed1, 2016-03-02 01:37:00]

reed@google.com changed reviewers:
+ bungeman@google.com

[reed1, 2016-03-02 01:37:01]

+bunge

[bungeman-skia, 2016-03-02 15:06:53]

lgtm

[reed1, 2016-03-02 15:29:23]

api lgtm

[hal.canary, 2016-03-02 15:43:46]

The CQ bit was checked by halcanary@google.com

[commit-bot: I haz the power, 2016-03-02 15:43:50]

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

[commit-bot: I haz the power, 2016-03-02 16:06:24]

Description was changed from

==========
sk_make_sp<T>()

because
    auto atm = sk_make_sp<SkAdvancedTypefaceMetrics>();
is better than
    sk_sp<SkAdvancedTypefaceMetrics> atm(new SkAdvancedTypefaceMetrics);

GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
sk_make_sp<T>()

because
    auto atm = sk_make_sp<SkAdvancedTypefaceMetrics>();
is better than
    sk_sp<SkAdvancedTypefaceMetrics> atm(new SkAdvancedTypefaceMetrics);

GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...

Committed:
https://skia.googlesource.com/skia/+/217c0b3f137d824413e806759bc25378abbe8c18
==========

[commit-bot: I haz the power, 2016-03-02 16:06:25]

Committed patchset #1 (id:1) as
https://skia.googlesource.com/skia/+/217c0b3f137d824413e806759bc25378abbe8c18