Revert of free -> reset (patchset #3 id:40001 of https://codereview.chromium.org/1811723002/ )

Reason for revert:
Suspect for Win10 failures.

Original issue's description:
> 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

TBR=reed@google.com,mtklein@google.com,mtklein@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:

Review URL: https://codereview.chromium.org/1813843002

1 files changed, 2 insertions, 1 deletions

diff --git a/src/core/SkAdvancedTypefaceMetrics.h b/src/core/SkAdvancedTypefaceMetrics.h
index 424e5f3f51..92655d21b2 100644
--- a/src/core/SkAdvancedTypefaceMetrics.h
+++ b/src/core/SkAdvancedTypefaceMetrics.h
@@ -28,12 +28,13 @@ public:
     T*        get() const { return fPtr; }
     T* operator->() const { return fPtr; }
 
-    void reset(T* ptr = nullptr) {
+    void reset(T* ptr) {
         if (ptr != fPtr) {
             delete fPtr;
             fPtr = ptr;
         }
     }
+    void free() { this->reset(nullptr); }
     T* release() {
         T* ptr = fPtr;
         fPtr = nullptr;