Bug 1308040 - Don't count painting time in GC telemetry (r=sfink)
authorBill McCloskey <billm@mozilla.com>
Thu, 20 Oct 2016 21:21:47 -0700
changeset 347045 54da3269c27693a8f8da5c09b23c5b5e62e30aa4
parent 347044 6964ddd8a364bd185a8b8eb23404b5eb202487f7
child 347046 1a44a71f1d221ec80b8cb636c94adac262a8a71b
push id10298
push userraliiev@mozilla.com
push dateMon, 14 Nov 2016 12:33:03 +0000
treeherdermozilla-aurora@7e29173b1641 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerssfink
bugs1308040
milestone52.0a1
Bug 1308040 - Don't count painting time in GC telemetry (r=sfink)
js/src/jsgc.cpp
--- a/js/src/jsgc.cpp
+++ b/js/src/jsgc.cpp
@@ -7724,15 +7724,19 @@ GCRuntime::requestInterruptCallback()
     }
 }
 
 void
 GCRuntime::invokeInterruptCallback(JSContext* cx)
 {
     interruptCallbackRequested = false;
 
+    stats.suspendPhases();
+
     JS::AutoAssertNoGC nogc(cx);
     JS::AutoAssertOnBarrier nobarrier(cx);
     JS::AutoSuppressGCAnalysis suppress(cx);
     for (JS::GCInterruptCallback callback : interruptCallbacks) {
         (*callback)(cx);
     }
-}
+
+    stats.resumePhases();
+}