Bug 1001419 - Use SizeOfExcludingThis for non-heap allocated member variable. r=njn
authorEric Rahm <erahm@mozilla.com>
Tue, 29 Apr 2014 14:47:56 -0700
changeset 181284 e0b5e968c38fa50b8e63423afe8592bb38ec0f2c
parent 181283 977cf6a69034d62d9c992fd7271470f49ac871c7
child 181285 734b0e34e002dc218c64abc4aa50f2d780b2a6d5
push id26699
push usercbook@mozilla.com
push dateFri, 02 May 2014 12:30:59 +0000
treeherdermozilla-central@66ea09d0c951 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersnjn
bugs1001419
milestone32.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1001419 - Use SizeOfExcludingThis for non-heap allocated member variable. r=njn
content/media/MediaResource.h
--- a/content/media/MediaResource.h
+++ b/content/media/MediaResource.h
@@ -417,17 +417,17 @@ public:
                   MallocSizeOf aMallocSizeOf) const MOZ_OVERRIDE
   {
     // Might be useful to track in the future:
     // - mChannel
     // - mURI (possibly owned, looks like just a ref from mChannel)
     // Not owned:
     // - mDecoder
     size_t size = MediaResource::SizeOfExcludingThis(aMallocSizeOf);
-    size += mContentType.SizeOfIncludingThisIfUnshared(aMallocSizeOf);
+    size += mContentType.SizeOfExcludingThisIfUnshared(aMallocSizeOf);
 
     return size;
   }
 
   virtual size_t SizeOfIncludingThis(
                   MallocSizeOf aMallocSizeOf) const MOZ_OVERRIDE
   {
     return aMallocSizeOf(this) + SizeOfExcludingThis(aMallocSizeOf);