Bug 1291483 part 1: Use 'MakeUnique' instead of 'new' to allocate gfxFont::mGlyphChangeObservers. r?jfkthame draft
authorDaniel Holbert <dholbert@cs.stanford.edu>
Thu, 09 Mar 2017 16:38:56 -0800
changeset 496217 c0ef95e091eb91ca9dac031d93c492f6afdddb72
parent 496010 56fbe9964a0bccccb6b75a2be4190f52f7a0a502
child 496218 d7eab3f8b2bde895cc2a323208886fc021e5d06d
push id48561
push userdholbert@mozilla.com
push dateFri, 10 Mar 2017 00:39:29 +0000
reviewersjfkthame
bugs1291483
milestone55.0a1
Bug 1291483 part 1: Use 'MakeUnique' instead of 'new' to allocate gfxFont::mGlyphChangeObservers. r?jfkthame MozReview-Commit-ID: 7FO1HAukp5j
gfx/thebes/gfxFont.cpp
--- a/gfx/thebes/gfxFont.cpp
+++ b/gfx/thebes/gfxFont.cpp
@@ -3843,18 +3843,18 @@ gfxFont::AddSizeOfIncludingThis(MallocSi
     aSizes->mFontInstances += aMallocSizeOf(this);
     AddSizeOfExcludingThis(aMallocSizeOf, aSizes);
 }
 
 void
 gfxFont::AddGlyphChangeObserver(GlyphChangeObserver *aObserver)
 {
     if (!mGlyphChangeObservers) {
-        mGlyphChangeObservers.reset(
-            new nsTHashtable<nsPtrHashKey<GlyphChangeObserver>>);
+        mGlyphChangeObservers =
+            MakeUnique<nsTHashtable<nsPtrHashKey<GlyphChangeObserver>>>();
     }
     mGlyphChangeObservers->PutEntry(aObserver);
 }
 
 void
 gfxFont::RemoveGlyphChangeObserver(GlyphChangeObserver *aObserver)
 {
     NS_ASSERTION(mGlyphChangeObservers, "No observers registered");