Bug 1531520 - Make sessionrestore talos tests measure the delta to process creation time, and not to sessionRestoreInit. r=mconley
authorFelipe Gomes <felipc@gmail.com>
Mon, 04 Mar 2019 16:57:45 +0000
changeset 520118 0ecb667b6b046767759ca6fda321d5ee2385d3e6
parent 520117 5caf48a420eb337a296c8b337332f2933884f467
child 520119 2e89b1f1e01c37b604bc37c2cdc41b30698f73fb
push id10862
push userffxbld-merge
push dateMon, 11 Mar 2019 13:01:11 +0000
treeherdermozilla-beta@a2e7f5c935da [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmconley
bugs1531520
milestone67.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1531520 - Make sessionrestore talos tests measure the delta to process creation time, and not to sessionRestoreInit. r=mconley Differential Revision: https://phabricator.services.mozilla.com/D21788
testing/talos/talos/startup_test/sessionrestore/addon/api.js
--- a/testing/talos/talos/startup_test/sessionrestore/addon/api.js
+++ b/testing/talos/talos/startup_test/sessionrestore/addon/api.js
@@ -76,39 +76,39 @@ this.sessionrestore = class extends Exte
         await SessionStartup.onceInitialized;
 
         if (SessionStartup.sessionType == SessionStartup.NO_SESSION ||
             SessionStartup.sessionType == SessionStartup.DEFER_SESSION) {
           // We should only hit this patch in sessionrestore_no_auto_restore
           if (!Services.prefs.getBoolPref("talos.sessionrestore.norestore", false)) {
             throw new Error("Session was not restored!");
           }
-          await this.finishProfiling("This test measures the time between sessionRestoreInit " +
-                                     "and sessionRestored, ignore everything around that");
+          await this.finishProfiling("This test measures the time between process " +
+                                     "creation and sessionRestored.");
 
           didRestore = false;
         } else {
           await new Promise(resolve => {
             let observe = async () => {
               Services.obs.removeObserver(observe, StartupPerformance.RESTORED_TOPIC);
-              await this.finishProfiling("This test measures the time between sessionRestoreInit " +
-                                         "and the last restored tab, ignore everything around that");
+              await this.finishProfiling("This test measures the time between process " +
+                                         "creation and the last restored tab.");
 
               resolve();
             };
             Services.obs.addObserver(observe, StartupPerformance.RESTORED_TOPIC);
           });
         }
       }
 
       let startup_info = Services.startup.getStartupInfo();
       let restoreTime = didRestore
                       ? StartupPerformance.latestRestoredTimeStamp
                       : startup_info.sessionRestored;
-      let duration = restoreTime - startup_info.sessionRestoreInit;
+      let duration = restoreTime - startup_info.process;
 
       // Report data to Talos, if possible.
       dump("__start_report" + duration + "__end_report\n\n");
 
       // Next one is required by the test harness but not used.
       // eslint-disable-next-line mozilla/avoid-Date-timing
       dump("__startTimestamp" + Date.now() + "__endTimestamp\n\n");
     } catch (ex) {