EOM
Created attachment 160458 [details] Patch
Comment on attachment 160458 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=160458&action=review > Source/WebCore/platform/graphics/GeneratorGeneratedImage.cpp:90 > + MemoryClassInfo info(memoryObjectInfo, this, MemoryInstrumentation::CachedResourceImage); Id rather move this method and the one above to GeneratedImage.cpp and include the latter in the build. You can fix GeneratedImage.cpp in a separate patch. > Source/WebCore/platform/graphics/Image.cpp:203 > + MemoryClassInfo info(memoryObjectInfo, this, MemoryInstrumentation::CachedResourceImage); Do we miss decodedSize() now?
Comment on attachment 160458 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=160458&action=review >> Source/WebCore/platform/graphics/Image.cpp:203 >> + MemoryClassInfo info(memoryObjectInfo, this, MemoryInstrumentation::CachedResourceImage); > > Do we miss decodedSize() now? Actually decodedSize is a sum of cached frames sizes. I reported it via BitmapImage instrumentation.
(In reply to comment #2) > (From update of attachment 160458 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=160458&action=review > > > Source/WebCore/platform/graphics/GeneratorGeneratedImage.cpp:90 > > + MemoryClassInfo info(memoryObjectInfo, this, MemoryInstrumentation::CachedResourceImage); > > Id rather move this method and the one above to GeneratedImage.cpp and include the latter in the build. You can fix GeneratedImage.cpp in a separate patch. Agree.
Committed r126974: <http://trac.webkit.org/changeset/126974>