Merge pull request #40 from Financial-Times/add-ready-state-dependency

Add ready-state dependency
Financial-Times · Feb 4, 2020 · 5aa505b · 5aa505b
2 parents bfb23d4 + 0ebea16
commit 5aa505b
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 17 deletions.
diff --git a/package.json b/package.json
@@ -38,6 +38,7 @@
     "@financial-times/o-tracking": "^2.0.3",
     "@financial-times/o-viewport": "^4.0.0",
     "perfume.js": "^4.6.0",
+    "ready-state": "^2.0.5",
     "seed-random": "^2.2.0"
   },
   "peerDependencies": {

diff --git a/src/client/trackers/real-user-performance/documentReady.js b/src/client/trackers/real-user-performance/documentReady.js
diff --git a/src/client/trackers/realUserMonitoringForPerformance.js b/src/client/trackers/realUserMonitoringForPerformance.js
@@ -1,8 +1,8 @@
 import Perfume from 'perfume.js';
+import readyState from 'ready-state';
 import { broadcast } from '../broadcast';
 import { seedIsInSample } from '../utils/seedIsInSample';
 import { getSpoorId } from '../utils/getSpoorId';
-import { documentReady } from './real-user-performance/documentReady';
 
 // @see "Important metrics to measure" https://web.dev/metrics
 const requiredMetrics = [
@@ -39,7 +39,7 @@ export const realUserMonitoringForPerformance = () => {
 
 	const context = {};
 
-	documentReady.then(() => {
+	readyState.complete.then(() => {
 		// <https://developer.mozilla.org/en-US/docs/Web/API/PerformanceNavigationTiming/domInteractive>
 		context.domInteractive = Math.round(navigation.domInteractive);
 		// <https://developer.mozilla.org/en-US/docs/Web/API/PerformanceNavigationTiming/domComplete>