Bug 1375219: Remove MozAfterPaint listener before calling await. r?mconley draft
authorKris Maglione <maglione.k@gmail.com>
Wed, 21 Jun 2017 12:13:44 -0700
changeset 598522 b3db60f86916bacff06eea839f8f22ce9a3c53bc
parent 598516 4bc36447fcf42277b0e13b42f9ec710536c3f051
child 634495 475493984f4f2da79cae306fd5fb8fafa8b93c21
push id65223
push usermaglione.k@gmail.com
push dateWed, 21 Jun 2017 19:19:28 +0000
reviewersmconley
bugs1375219
milestone56.0a1
Bug 1375219: Remove MozAfterPaint listener before calling await. r?mconley MozReview-Commit-ID: 7PIjbvtPObl
testing/talos/talos/startup_test/tspaint_test.html
--- a/testing/talos/talos/startup_test/tspaint_test.html
+++ b/testing/talos/talos/startup_test/tspaint_test.html
@@ -8,22 +8,22 @@
 <!-- In-line this to avoid compilation. -->
 <script language="Javascript" type="text/javascript" src="chrome://talos-powers-content/content/TalosPowersContent.js"></script>
 <script language="Javascript" type="text/javascript" src="../scripts/MozillaFileLogger.js"></script>
 <script language="Javascript" type="text/javascript" src="chrome://talos-powers-content/content/TalosContentProfiler.js"></script>
 <script language="JavaScript" type="text/javascript" src="../tests/quit.js"></script>
 <script language="javascript" type="text/javascript">
 
 async function painted() {
+  window.removeEventListener("MozAfterPaint", painted, true);
+
   await TalosContentProfiler.pause();
   await TalosContentProfiler.initFromURLQueryParams(location.search);
   await TalosContentProfiler.finishStartupProfiling();
 
-  window.removeEventListener("MozAfterPaint", painted, true);
-
   let startupInfo = await TalosPowersContent.getStartupInfo();
 
   let startupTime = startupInfo['firstPaint'] - startupInfo['process'];
 
   document.body.textContent = 'Startup time = ' + startupTime + ' ms';
   if (window.dump) {
     dumpLog('__start_report' + startupTime + '__end_report\n\n');
   }