Title: [141390] trunk/Source/WebCore
Revision
141390
Author
[email protected]
Date
2013-01-31 00:52:58 -0800 (Thu, 31 Jan 2013)

Log Message

[V8] Trace not only major DOM GCs but also minor DOM GCs
https://bugs.webkit.org/show_bug.cgi?id=108436

Reviewed by Adam Barth.

Currently only major DOM GCs are traced.
We should also trace minor DOM GCs.

No tests. No change in behavior.

* bindings/v8/V8GCController.cpp:
(WebCore::V8GCController::minorGCPrologue):
(WebCore::V8GCController::minorGCEpilogue):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (141389 => 141390)


--- trunk/Source/WebCore/ChangeLog	2013-01-31 08:49:47 UTC (rev 141389)
+++ trunk/Source/WebCore/ChangeLog	2013-01-31 08:52:58 UTC (rev 141390)
@@ -1,3 +1,19 @@
+2013-01-31  Kentaro Hara  <[email protected]>
+
+        [V8] Trace not only major DOM GCs but also minor DOM GCs
+        https://bugs.webkit.org/show_bug.cgi?id=108436
+
+        Reviewed by Adam Barth.
+
+        Currently only major DOM GCs are traced.
+        We should also trace minor DOM GCs.
+
+        No tests. No change in behavior.
+
+        * bindings/v8/V8GCController.cpp:
+        (WebCore::V8GCController::minorGCPrologue):
+        (WebCore::V8GCController::minorGCEpilogue):
+
 2013-01-31  Yury Semikhatsky  <[email protected]>
 
         Layout Test inspector-protocol/take-heap-snapshot.html crashes in the Debug mode

Modified: trunk/Source/WebCore/bindings/v8/V8GCController.cpp (141389 => 141390)


--- trunk/Source/WebCore/bindings/v8/V8GCController.cpp	2013-01-31 08:49:47 UTC (rev 141389)
+++ trunk/Source/WebCore/bindings/v8/V8GCController.cpp	2013-01-31 08:52:58 UTC (rev 141390)
@@ -332,6 +332,8 @@
 
 void V8GCController::minorGCPrologue()
 {
+    TRACE_EVENT_BEGIN0("v8", "GC");
+
     if (isMainThreadOrGCThread() && m_edenNodes) {
         for (size_t i = 0; i < m_edenNodes->size(); i++) {
             ASSERT(!m_edenNodes->at(i)->wrapper().IsEmpty());
@@ -374,6 +376,7 @@
 
 void V8GCController::minorGCEpilogue()
 {
+    TRACE_EVENT_END0("v8", "GC");
 }
 
 void V8GCController::majorGCEpilogue()
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to