fix: mark global as global via externalGlobals (#40)

Using the built-in globals feature will try and do interop resolution
for requiring `window` which will break if a user has an element with an
id of `default`. This plugin does not do so and will end up removing the
lines from the browser build altogether.

index.js

@@ -8,6 +8,7 @@ const multiEntry = require('@rollup/plugin-multi-entry');
 const resolve = require('@rollup/plugin-node-resolve').default;
 const {terser} = require('rollup-plugin-terser');
 const istanbul = require('rollup-plugin-istanbul');
+const externalGlobals = require('rollup-plugin-external-globals');
 const path = require('path');
 
 const babelConfig = require('@videojs/babel-config/es.js');
@@ -62,10 +63,7 @@ const MINJS_REGEX = /^.+\.min\.js$/;
 const ORDERED_DEFAULTS = {
   globals: () => ({
     browser: {
-      'video.js': 'videojs',
-      'global': 'window',
-      'global/window': 'window',
-      'global/document': 'document'
+      'video.js': 'videojs'
     },
     module: {
       'video.js': 'videojs'
@@ -92,6 +90,7 @@ const ORDERED_DEFAULTS = {
       'resolve',
       'json',
       'commonjs',
+      'externalGlobals',
       'babel'
     ],
     module: [
@@ -135,6 +134,11 @@ const ORDERED_DEFAULTS = {
     commonjs: commonjs({sourceMap: false}),
     jsonResolve: resolve({extensions: ['.json']}),
     json: json(),
+    externalGlobals: externalGlobals({
+      'global': 'window',
+      'global/window': 'window',
+      'global/document': 'document'
+    }),
     multiEntry: multiEntry({exports: false}),
     resolve: resolve({
       mainFields: ['browser', 'module', 'jsnext:main', 'main'],

package.json

@@ -41,6 +41,7 @@
     "@rollup/plugin-multi-entry": "^4.0.0",
     "@rollup/plugin-node-resolve": "^9.0.0",
     "@videojs/babel-config": "^0.2.0",
+    "rollup-plugin-external-globals": "^0.6.1",
     "rollup-plugin-istanbul": "^2.0.1",
     "rollup-plugin-terser": "^7.0.2"
   },