refactor: remove old TTI code (#2452)

GoogleChrome · Jun 7, 2017 · 4096e68 · 4096e68
1 parent 65db419
commit 4096e68
Show file tree

Hide file tree

Showing 5 changed files with 3 additions and 304 deletions.
diff --git a/lighthouse-core/audits/time-to-interactive.js b/lighthouse-core/audits/time-to-interactive.js
diff --git a/lighthouse-core/config/default.js b/lighthouse-core/config/default.js
@@ -76,7 +76,6 @@ module.exports = {
     // 'time-to-firstbyte',
     'first-interactive',
     'consistently-interactive',
-    'time-to-interactive',
     'user-timings',
     'critical-request-chains',
     'webapp-install-banner',

diff --git a/lighthouse-core/lib/traces/pwmetrics-events.js b/lighthouse-core/lib/traces/pwmetrics-events.js
@@ -119,18 +119,6 @@ class Metrics {
           return safeGet(siExt, 'value.timings.visuallyComplete');
         }
       },
-      {
-        name: 'Time to Interactive (vAlpha)',
-        id: 'tti',
-        getTs: auditResults => {
-          const ttiExt = auditResults['time-to-interactive'].extendedInfo;
-          return safeGet(ttiExt, 'value.timestamps.timeToInteractive');
-        },
-        getTiming: auditResults => {
-          const ttiExt = auditResults['time-to-interactive'].extendedInfo;
-          return safeGet(ttiExt, 'value.timings.timeToInteractive');
-        }
-      },
       {
         name: 'First Interactive (vBeta)',
         id: 'ttfi',

diff --git a/lighthouse-core/test/audits/time-to-interactive-test.js b/lighthouse-core/test/audits/time-to-interactive-test.js
diff --git a/lighthouse-core/test/config/config-test.js b/lighthouse-core/test/config/config-test.js
@@ -304,14 +304,14 @@ describe('Config', () => {
       audits: [
         'accessibility/color-contrast',
         'first-meaningful-paint',
-        'time-to-interactive',
+        'first-interactive',
         'estimated-input-latency',
       ],
       categories: {
         'needed-category': {
           audits: [
             {id: 'first-meaningful-paint'},
-            {id: 'time-to-interactive'},
+            {id: 'first-interactive'},
           ],
         },
         'other-category': {
@@ -376,7 +376,7 @@ describe('Config', () => {
       extends: true,
       settings: {
         onlyCategories: ['performance', 'missing-category'],
-        onlyAudits: ['time-to-interactive', 'missing-audit'],
+        onlyAudits: ['first-interactive', 'missing-audit'],
       },
     });