Title: [132655] trunk/Source/WebCore
Revision
132655
Author
[email protected]
Date
2012-10-26 07:49:20 -0700 (Fri, 26 Oct 2012)

Log Message

[v8] Memory instrumentation: don't count memory of WrapperTypeInfo
https://bugs.webkit.org/show_bug.cgi?id=100517

Reviewed by Alexander Pavlov.

* bindings/v8/V8PerIsolateData.cpp: when estimating bindings memory size skip
pointers to WrapperTypeInfo objects as they are static fields and belong to
the data segment.
(WTF):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (132654 => 132655)


--- trunk/Source/WebCore/ChangeLog	2012-10-26 14:43:32 UTC (rev 132654)
+++ trunk/Source/WebCore/ChangeLog	2012-10-26 14:49:20 UTC (rev 132655)
@@ -1,3 +1,15 @@
+2012-10-26  Yury Semikhatsky  <[email protected]>
+
+        [v8] Memory instrumentation: don't count memory of WrapperTypeInfo
+        https://bugs.webkit.org/show_bug.cgi?id=100517
+
+        Reviewed by Alexander Pavlov.
+
+        * bindings/v8/V8PerIsolateData.cpp: when estimating bindings memory size skip
+        pointers to WrapperTypeInfo objects as they are static fields and belong to
+        the data segment.
+        (WTF):
+
 2012-10-26  Kent Tamura  <[email protected]>
 
         Refactor Localizer-related classes.

Modified: trunk/Source/WebCore/bindings/v8/V8PerIsolateData.cpp (132654 => 132655)


--- trunk/Source/WebCore/bindings/v8/V8PerIsolateData.cpp	2012-10-26 14:43:32 UTC (rev 132654)
+++ trunk/Source/WebCore/bindings/v8/V8PerIsolateData.cpp	2012-10-26 14:49:20 UTC (rev 132655)
@@ -31,6 +31,15 @@
 #include <wtf/MemoryInstrumentationHashMap.h>
 #include <wtf/MemoryInstrumentationVector.h>
 
+namespace WTF {
+
+// WrapperTypeInfo are statically allocated, don't count them.
+template<> struct SequenceMemoryInstrumentationTraits<WebCore::WrapperTypeInfo*> {
+    template <typename I> static void reportMemoryUsage(I, I, MemoryClassInfo&) { }
+};
+
+}
+
 namespace WebCore {
 
 V8PerIsolateData::V8PerIsolateData(v8::Isolate* isolate)
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to