free -> reset

free -> reset

The C++ standard library uses ".reset()" where we sometimes write ".free()".
We also use ".reset()" quite a lot.  This standardizes on ".reset()".

This is one more step towards dropping SkAutoTDelete in favor of the standard
std::unique_ptr.

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

Committed: https://skia.googlesource.com/skia/+/0e3738db89e86035ed5d4f629bf58b817b1e5274

Committed: https://skia.googlesource.com/skia/+/852f15da7ceb53cfb49b9f728baa6dbc53b27694

[mtklein_C, 2016-03-16 21:07:11]

Description was changed from

==========
free -> reset

The C++ standard library uses ".reset()" where we sometimes write ".free()".
We also use ".reset()" quite a lot.  This standardizes on ".reset()".

This is one more step towards dropping SkAutoTDelete in favor of the standard
std::unique_ptr.

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

to

==========
free -> reset

The C++ standard library uses ".reset()" where we sometimes write ".free()".
We also use ".reset()" quite a lot.  This standardizes on ".reset()".

This is one more step towards dropping SkAutoTDelete in favor of the standard
std::unique_ptr.

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

[mtklein, 2016-03-16 21:08:41]

The CQ bit was checked by mtklein@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-03-16 21:08:44]

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

[commit-bot: I haz the power, 2016-03-16 21:10:54]

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

[commit-bot: I haz the power, 2016-03-16 21:10:55]

Dry run: Try jobs failed on following builders:
  Build-Mac-Clang-x86_64-Release-Trybot on client.skia.compile (JOB_FAILED,
http://build.chromium.org/p/client.skia.compile/builders/Build-Mac-Clang-x86_...)
  Build-Ubuntu-GCC-Arm7-Debug-Android-Trybot on client.skia.compile (JOB_FAILED,
http://build.chromium.org/p/client.skia.compile/builders/Build-Ubuntu-GCC-Arm...)

[mtklein, 2016-03-16 21:17:36]

The CQ bit was checked by mtklein@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-03-16 21:17:45]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1811723002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1811723002/20001

[mtklein, 2016-03-16 21:30:16]

The CQ bit was checked by mtklein@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-03-16 21:30:25]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1811723002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1811723002/40001

[mtklein_C, 2016-03-16 21:30:54]

The CQ bit was unchecked by mtklein@chromium.org

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

mtklein@chromium.org changed reviewers:
+ reed@google.com
- mtklein@google.com

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

Similar but less interesting than the last detach -> release CL.  Getting close.

[reed1, 2016-03-17 01:57:28]

lgtm

[mtklein, 2016-03-17 12:32:33]

The CQ bit was checked by mtklein@google.com

[commit-bot: I haz the power, 2016-03-17 12:32:44]

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

[commit-bot: I haz the power, 2016-03-17 12:36:20]

Description was changed from

==========
free -> reset

The C++ standard library uses ".reset()" where we sometimes write ".free()".
We also use ".reset()" quite a lot.  This standardizes on ".reset()".

This is one more step towards dropping SkAutoTDelete in favor of the standard
std::unique_ptr.

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

to

==========
free -> reset

The C++ standard library uses ".reset()" where we sometimes write ".free()".
We also use ".reset()" quite a lot.  This standardizes on ".reset()".

This is one more step towards dropping SkAutoTDelete in favor of the standard
std::unique_ptr.

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/+/0e3738db89e86035ed5d4f629bf58b817b1e5274
==========

[commit-bot: I haz the power, 2016-03-17 12:36:21]

Committed patchset #3 (id:40001) as
https://skia.googlesource.com/skia/+/0e3738db89e86035ed5d4f629bf58b817b1e5274

[bungeman-skia, 2016-03-17 16:53:47]

A revert of this CL (patchset #3 id:40001) has been created in
https://codereview.chromium.org/1813843002/ by bungeman@google.com.

The reason for reverting is: Suspect for Win10 failures..

[mtklein, 2016-03-17 17:16:20]

gonna tick this to reland when the tree opens

[mtklein, 2016-03-17 17:16:27]

Description was changed from

==========
free -> reset

The C++ standard library uses ".reset()" where we sometimes write ".free()".
We also use ".reset()" quite a lot.  This standardizes on ".reset()".

This is one more step towards dropping SkAutoTDelete in favor of the standard
std::unique_ptr.

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/+/0e3738db89e86035ed5d4f629bf58b817b1e5274
==========

to

==========
free -> reset

The C++ standard library uses ".reset()" where we sometimes write ".free()".
We also use ".reset()" quite a lot.  This standardizes on ".reset()".

This is one more step towards dropping SkAutoTDelete in favor of the standard
std::unique_ptr.

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/+/0e3738db89e86035ed5d4f629bf58b817b1e5274
==========

[mtklein, 2016-03-17 17:16:34]

The CQ bit was checked by mtklein@google.com

[commit-bot: I haz the power, 2016-03-17 17:16:39]

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

[commit-bot: I haz the power, 2016-03-17 17:51:29]

Description was changed from

==========
free -> reset

The C++ standard library uses ".reset()" where we sometimes write ".free()".
We also use ".reset()" quite a lot.  This standardizes on ".reset()".

This is one more step towards dropping SkAutoTDelete in favor of the standard
std::unique_ptr.

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/+/0e3738db89e86035ed5d4f629bf58b817b1e5274
==========

to

==========
free -> reset

The C++ standard library uses ".reset()" where we sometimes write ".free()".
We also use ".reset()" quite a lot.  This standardizes on ".reset()".

This is one more step towards dropping SkAutoTDelete in favor of the standard
std::unique_ptr.

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/+/0e3738db89e86035ed5d4f629bf58b817b1e5274

Committed:
https://skia.googlesource.com/skia/+/852f15da7ceb53cfb49b9f728baa6dbc53b27694
==========

[commit-bot: I haz the power, 2016-03-17 17:51:30]

Committed patchset #3 (id:40001) as
https://skia.googlesource.com/skia/+/852f15da7ceb53cfb49b9f728baa6dbc53b27694