Electrolysis/Release Criteria/Slow Script: Difference between revisions

Electrolysis/Release Criteria/Slow Script (view source)

Revision as of 21:29, 22 February 2016

92 bytes added , 22 February 2016

Update with some more results.

Azhang

10

edits

@@ Line 33: / Line 33: @@
 * [PARENT] '''ProcessHangMonitor.reportHang''' is called when the '''ProcessHangMonitor''' receives a "process-hang-report" event (https://dxr.mozilla.org/mozilla-central/source/browser/modules/ProcessHangMonitor.jsm#66)
 * [PARENT] JS function '''ProcessHangMonitor.reportHang''' accumulates the SLOW_SCRIPT_NOTICE_COUNT histogram (https://dxr.mozilla.org/mozilla-central/source/browser/modules/ProcessHangMonitor.jsm#403)
-* I've checked that this is currently the only code path that results in SLOW_SCRIPT_NOTICE_COUNT being incremented.
+* :azhang checked that this is currently the only code path that results in SLOW_SCRIPT_NOTICE_COUNT being incremented.
 Have we tested the metric to make sure it's working properly in e10s?
 * There should be no differences in the slow script '''reporting''' mechanism between e10s and non-e10s; 1 call to ShowSlowScriptDialog means 1 addition to the histogram regardless of e10s setting. Manual testing supports this.
-* The timeout has changed from 2 to 10 in e10s builds (see https://hg.mozilla.org/mozilla-central/rev/63680efe6d55#l35.12 for before and https://hg.mozilla.org/mozilla-central/rev/5002e9c06365#l2.13 for after). :azhang is currently investigating this.
+* The timeout has changed from 2 to 10 in e10s builds (see https://hg.mozilla.org/mozilla-central/rev/63680efe6d55#l35.12 for before and https://hg.mozilla.org/mozilla-central/rev/5002e9c06365#l2.13 for after). However, the value is correct on all beta versions, from 45.0b1 to 45.0b8 - the timeouts are identical between e10s and non-e10s.