Title: [132605] trunk/Source/WebCore
Revision
132605
Author
[email protected]
Date
2012-10-26 03:50:46 -0700 (Fri, 26 Oct 2012)

Log Message

Memory instrumentation: do not report memory occupied by v8::String
https://bugs.webkit.org/show_bug.cgi?id=100487

Reviewed by Alexander Pavlov.

Pointers to v8::String are skipped when reporting memory usage of V8 bindings'
string cache.

* bindings/v8/V8Binding.cpp:
(WTF):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (132604 => 132605)


--- trunk/Source/WebCore/ChangeLog	2012-10-26 10:45:37 UTC (rev 132604)
+++ trunk/Source/WebCore/ChangeLog	2012-10-26 10:50:46 UTC (rev 132605)
@@ -1,3 +1,16 @@
+2012-10-26  Yury Semikhatsky  <[email protected]>
+
+        Memory instrumentation: do not report memory occupied by v8::String
+        https://bugs.webkit.org/show_bug.cgi?id=100487
+
+        Reviewed by Alexander Pavlov.
+
+        Pointers to v8::String are skipped when reporting memory usage of V8 bindings'
+        string cache.
+
+        * bindings/v8/V8Binding.cpp:
+        (WTF):
+
 2012-10-26  Mihnea Ovidenie  <[email protected]>
 
         [CSSRegions] Add support for auto-height regions with region-breaks

Modified: trunk/Source/WebCore/bindings/v8/V8Binding.cpp (132604 => 132605)


--- trunk/Source/WebCore/bindings/v8/V8Binding.cpp	2012-10-26 10:45:37 UTC (rev 132604)
+++ trunk/Source/WebCore/bindings/v8/V8Binding.cpp	2012-10-26 10:50:46 UTC (rev 132605)
@@ -61,6 +61,14 @@
 #include <wtf/text/StringHash.h>
 #include <wtf/text/WTFString.h>
 
+namespace WTF {
+
+template<> struct SequenceMemoryInstrumentationTraits<v8::String*> {
+    template <typename I> static void reportMemoryUsage(I, I, MemoryClassInfo&) { }
+};
+
+}
+
 namespace WebCore {
 
 v8::Handle<v8::Value> setDOMException(int exceptionCode, v8::Isolate* isolate)
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to