Add back SkAutoTDelete::detach() for Android temporarily.

Add back SkAutoTDelete::detach() for Android temporarily.

TBR=

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

Committed: https://skia.googlesource.com/skia/+/cc864c336d19e3223f603af5f711a00ad721f647

[mtklein_C, 2016-03-17 00:13:27]

Description was changed from

==========
Add back SkAutoTDelete::detach() for Android temporarily.

BUG=skia:
==========

to

==========
Add back SkAutoTDelete::detach() for Android temporarily.

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

[mtklein_C, 2016-03-17 00:15:05]

Description was changed from

==========
Add back SkAutoTDelete::detach() for Android temporarily.

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

to

==========
Add back SkAutoTDelete::detach() for Android temporarily.

TBR=

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

[mtklein_C, 2016-03-17 00:15:07]

The CQ bit was checked by mtklein@chromium.org

[commit-bot: I haz the power, 2016-03-17 00:15:18]

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

[commit-bot: I haz the power, 2016-03-17 00:33:58]

Description was changed from

==========
Add back SkAutoTDelete::detach() for Android temporarily.

TBR=

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

to

==========
Add back SkAutoTDelete::detach() for Android temporarily.

TBR=

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

Committed:
https://skia.googlesource.com/skia/+/cc864c336d19e3223f603af5f711a00ad721f647
==========

[commit-bot: I haz the power, 2016-03-17 00:33:59]

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

[scroggo, 2016-03-17 12:37:47]

scroggo@google.com changed reviewers:
+ djsollen@google.com, scroggo@google.com

[scroggo, 2016-03-17 12:37:48]

lgtm

https://codereview.chromium.org/1811483004/diff/1/include/private/SkTemplates.h
File include/private/SkTemplates.h (right):

https://codereview.chromium.org/1811483004/diff/1/include/private/SkTemplates...
include/private/SkTemplates.h:102: // Need to update
graphics/BitmapRegionDecoder.cpp.
Looks like we also need to update Shader.cpp

[mtklein, 2016-03-17 12:39:13]

> Looks like we also need to update Shader.cpp

That's an SkAutoTUnref, right?

[scroggo, 2016-03-17 13:08:27]

On 2016/03/17 12:39:13, mtklein wrote:
> > Looks like we also need to update Shader.cpp
> 
> That's an SkAutoTUnref, right?

D'oh. Yes. And I see you fixed with crrev.com/1812843002