open-telemetry · mayurkale22 · Oct 23, 2019 · Oct 14, 2019 · Oct 15, 2019 · Oct 15, 2019
diff --git a/examples/tracer-web/index.html b/examples/tracer-web/index.html
@@ -3,14 +3,14 @@
 
 <head>
   <meta charset="utf-8">
-  <title>JS Example</title>
+  <title>Web Tracer Example</title>
   <base href="/">
 
   <meta name="viewport" content="width=device-width, initial-scale=1">
 </head>
 
 <body>
-  Testing, debugging in development
+  Example of using Web Tracer with document load plugin and console exporter
   <script type="text/javascript" src="/bundle.js"></script>
 </body>
 

diff --git a/examples/tracer-web/index.js b/examples/tracer-web/index.js
@@ -1,38 +1,11 @@
+import { ConsoleSpanExporter, SimpleSpanProcessor } from '@opentelemetry/tracing';
 import { WebTracer } from '@opentelemetry/web';
+import { DocumentLoad } from '@opentelemetry/plugin-document-load';
 
-import * as shimmer from 'shimmer';
-
-class Tester {
-  constructor() {
-  }
-  add(name) {
-    console.log('calling add', name);
-  }
-}
-
-const tester = new Tester();
-
-const webTracer = new WebTracer();
-const span = webTracer.startSpan('span1');
-
-shimmer.wrap(Tester.prototype, 'add', (originalFunction) => {
-  return function patchedFunction() {
-    try {
-      span.addEvent('start');
-    } catch (e) {
-      console.log('error', e);
-    } finally {
-      const result = originalFunction.apply(this, arguments);
-      span.addEvent('after call');
-      span.end();
-      return result;
-    }
-  };
-});
-
-webTracer.withSpan(span, function () {
-  console.log(this === span);
+const webTracer = new WebTracer({
+  plugins: [
+    new DocumentLoad()
+  ]
 });
 
-tester.add('foo');
-console.log(span);
+webTracer.addSpanProcessor(new SimpleSpanProcessor(new ConsoleSpanExporter()));
diff --git a/examples/tracer-web/package.json b/examples/tracer-web/package.json
@@ -26,16 +26,17 @@
   },
   "devDependencies": {
     "@babel/core": "^7.6.0",
-    "@types/shimmer": "^1.0.1",
     "babel-loader": "^8.0.6",
-    "shimmer": "^1.2.0",
+    "ts-loader": "^6.0.4",
     "webpack": "^4.35.2",
     "webpack-cli": "^3.3.9",
     "webpack-dev-server": "^3.8.1",
     "webpack-merge": "^4.2.2"
   },
   "dependencies": {
+    "@opentelemetry/plugin-document-load": "^0.1.1",
+    "@opentelemetry/tracing": "^0.1.1",
     "@opentelemetry/web": "^0.1.1"
   },
   "homepage": "https://github.com/open-telemetry/opentelemetry-js#readme"
-}
+}
diff --git a/packages/opentelemetry-core/src/common/time.ts b/packages/opentelemetry-core/src/common/time.ts
@@ -16,6 +16,7 @@
 
 import * as types from '@opentelemetry/types';
 import { otperformance as performance } from '../platform';
+import { TimeOriginLegacy } from './types';
 
 const NANOSECOND_DIGITS = 9;
 const SECOND_TO_NANOSECONDS = Math.pow(10, NANOSECOND_DIGITS);
@@ -32,10 +33,20 @@ function numberToHrtime(epochMillis: number): types.HrTime {
   return [seconds, nanos];
 }
 
+function getTimeOrigin(): number {
+  let timeOrigin = performance.timeOrigin;
+  if (typeof timeOrigin !== 'number') {
+    const perf: TimeOriginLegacy = (performance as unknown) as TimeOriginLegacy;
+    timeOrigin = perf.timing && perf.timing.fetchStart;
+  }
+  return timeOrigin;
+}
 // Returns an hrtime calculated via performance component.
 export function hrTime(performanceNow?: number): types.HrTime {
-  const timeOrigin = numberToHrtime(performance.timeOrigin);
-  const now = numberToHrtime(performanceNow || performance.now());
+  const timeOrigin = numberToHrtime(getTimeOrigin());
+  const now = numberToHrtime(
+    typeof performanceNow === 'number' ? performanceNow : performance.now()
+  );
 
   let seconds = timeOrigin[0] + now[0];
   let nanos = timeOrigin[1] + now[1];
@@ -56,11 +67,10 @@ export function timeInputToHrTime(time: types.TimeInput): types.HrTime {
     return time;
   } else if (typeof time === 'number') {
     // Must be a performance.now() if it's smaller than process start time.
-    if (time < performance.timeOrigin) {
+    if (time < getTimeOrigin()) {
       return hrTime(time);
-    }
-    // epoch milliseconds or performance.timeOrigin
-    else {
+    } else {
+      // epoch milliseconds or performance.timeOrigin
       return numberToHrtime(time);
     }
   } else if (time instanceof Date) {

diff --git a/packages/opentelemetry-core/src/common/types.ts b/packages/opentelemetry-core/src/common/types.ts
@@ -21,3 +21,14 @@ export enum LogLevel {
   INFO,
   DEBUG,
 }
+
+/**
+ * This interface defines a fallback to read a timeOrigin when it is not available on performance.timeOrigin,
+ * this happens for example on Safari Mac
+ * then the timeOrigin is taken from fetchStart - which is the closest to timeOrigin
+ */
+export interface TimeOriginLegacy {
+  timing: {
+    fetchStart: number;
+  };
+}
diff --git a/packages/opentelemetry-core/test/common/time.test.ts b/packages/opentelemetry-core/test/common/time.test.ts
@@ -62,6 +62,47 @@ describe('time', () => {
       const output = hrTime(performanceNow);
       assert.deepStrictEqual(output, [0, 23100000]);
     });
+
+    it('should allow passed "performanceNow" equal to 0', () => {
+      sandbox.stub(performance, 'timeOrigin').value(11.5);
+      sandbox.stub(performance, 'now').callsFake(() => 11.3);
+
+      const output = hrTime(0);
+      assert.deepStrictEqual(output, [0, 11500000]);
+    });
+
+    it('should use performance.now() when "performanceNow" is equal to undefined', () => {
+      sandbox.stub(performance, 'timeOrigin').value(11.5);
+      sandbox.stub(performance, 'now').callsFake(() => 11.3);
+
+      const output = hrTime(undefined);
+      assert.deepStrictEqual(output, [0, 22800000]);
+    });
+
+    it('should use performance.now() when "performanceNow" is equal to null', () => {
+      sandbox.stub(performance, 'timeOrigin').value(11.5);
+      sandbox.stub(performance, 'now').callsFake(() => 11.3);
+
+      const output = hrTime(null as any);
+      assert.deepStrictEqual(output, [0, 22800000]);
+    });
+
+    describe('when timeOrigin is not available', () => {
+      it('should use the performance.timing.fetchStart as a fallback', () => {
+        Object.defineProperty(performance, 'timing', {
+          writable: true,
+          value: {
+            fetchStart: 11.5,
+          },
+        });
+
+        sandbox.stub(performance, 'timeOrigin').value(undefined);
+        sandbox.stub(performance, 'now').callsFake(() => 11.3);
+
+        const output = hrTime();
+        assert.deepStrictEqual(output, [0, 22800000]);
+      });
+    });
   });
 
   describe('#timeInputToHrTime', () => {

diff --git a/packages/opentelemetry-plugin-document-load/karma.conf.js b/packages/opentelemetry-plugin-document-load/karma.conf.js
@@ -0,0 +1,24 @@
+/*!
+ * Copyright 2019, OpenTelemetry Authors
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+const webpackConfig = require('./webpack/test.config.js');
+const karmaBaseConfig = require('../../karma.base');
+
+module.exports = (config) => {
+  config.set(Object.assign({}, karmaBaseConfig, {
+    webpack: webpackConfig
+  }))
+};
diff --git a/packages/opentelemetry-plugin-document-load/package.json b/packages/opentelemetry-plugin-document-load/package.json
@@ -0,0 +1,76 @@
+{
+  "name": "@opentelemetry/plugin-document-load",
+  "version": "0.1.1",
+  "description": "OpenTelemetry document-load automatic instrumentation package.",
+  "main": "build/src/index.js",
+  "types": "build/src/index.d.ts",
+  "repository": "open-telemetry/opentelemetry-js",
+  "scripts": {
+    "clean": "rimraf build/*",
+    "check": "gts check",
+    "codecov:browser": "nyc report --reporter=json && codecov -f coverage/*.json -p ../../",
+    "compile": "tsc -p .",
+    "fix": "gts fix",
+    "prepare": "npm run compile",
+    "tdd": "karma start",
+    "test:browser": "nyc karma start --single-run",
+    "watch": "tsc -w"
+  },
+  "keywords": [
+    "opentelemetry",
+    "document-load",
+    "web",
+    "tracing",
+    "profiling",
+    "plugin"
+  ],
+  "author": "OpenTelemetry Authors",
+  "license": "Apache-2.0",
+  "engines": {
+    "node": ">=8.0.0"
+  },
+  "files": [
+    "build/src/**/*.js",
+    "build/src/**/*.d.ts",
+    "doc",
+    "LICENSE",
+    "README.md"
+  ],
+  "publishConfig": {
+    "access": "public"
+  },
+  "devDependencies": {
+    "@types/mocha": "^5.2.5",
+    "@types/node": "^12.6.8",
+    "@types/webpack-env": "1.13.9",
+    "@types/sinon": "^7.0.13",
+    "@babel/core": "^7.6.0",
+    "babel-loader": "^8.0.6",
+    "codecov": "^3.1.0",
+    "gts": "^1.0.0",
+    "karma": "^4.1.0",
+    "karma-chrome-launcher": "^2.2.0",
+    "karma-mocha": "^1.3.0",
+    "karma-spec-reporter": "^0.0.32",
+    "karma-webpack": "^4.0.2",
+    "mocha": "^6.1.0",
+    "nyc": "^14.1.1",
+    "rimraf": "^3.0.0",
+    "sinon": "^7.5.0",
+    "tslint-consistent-codestyle" : "^1.16.0",
+    "tslint-microsoft-contrib": "^6.2.0",
+    "ts-loader": "^6.0.4",
+    "ts-mocha": "^6.0.0",
+    "ts-node": "^8.0.0",
+    "typescript": "^3.6.3",
+    "webpack": "^4.35.2",
+    "webpack-cli": "^3.3.9",
+    "webpack-merge": "^4.2.2"
+  },
+  "dependencies": {
+    "@opentelemetry/core": "^0.1.1",
+    "@opentelemetry/types": "^0.1.1",
+    "@opentelemetry/tracing": "^0.1.1",
+    "@opentelemetry/web": "^0.1.1"
+  }
+}