Changeset 141390 in webkit

Timestamp:

Jan 31, 2013 12:52:58 AM (11 years ago)

Author:

haraken@chromium.org

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):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• bindings/v8/V8GCController.cpp (modified) (2 diffs)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2013-01-31  Kentaro Hara  <haraken@chromium.org>
+
+        [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  <yurys@chromium.org>
 

trunk/Source/WebCore/bindings/v8/V8GCController.cpp

@@ -333 +333 @@
 void V8GCController::minorGCPrologue()
 {
+    TRACE_EVENT_BEGIN0("v8", "GC");
+
     if (isMainThreadOrGCThread() && m_edenNodes) {
         for (size_t i = 0; i < m_edenNodes->size(); i++) {
@@ -375 +377 @@
 void V8GCController::minorGCEpilogue()
 {
+    TRACE_EVENT_END0("v8", "GC");
 }