do not require Protection and MetricsReporting as dependencies as the…

…se are optional and intended to be loaded separately

src/streaming/MediaPlayer.js

@@ -59,8 +59,6 @@ import EventBus from './../core/EventBus.js';
 import Events from './../core/events/Events.js';
 import MediaPlayerEvents from './MediaPlayerEvents.js';
 import FactoryMaker from '../core/FactoryMaker.js';
-import Protection from './protection/Protection.js';
-import MetricsReporting from './metrics/MetricsReporting.js';
 //Dash
 import DashAdapter from '../dash/DashAdapter.js';
 import DashParser from '../dash/DashParser.js';
@@ -1606,7 +1604,8 @@ function MediaPlayer() {
         if (protectionController) {
             return protectionController;
         }
-
+        // do not require Protection as dependencies as this is optional and intended to be loaded separately
+        let Protection = dashjs.Protection; /* jshint ignore:line */
         if (typeof Protection == 'function') {//TODO need a better way to register/detect plugin components
             let protection = Protection(context).create();
             Events.extend(Protection.events);
@@ -1628,7 +1627,8 @@ function MediaPlayer() {
         if (metricsReportingController) {
             return metricsReportingController;
         }
-
+        // do not require MetricsReporting as dependencies as this is optional and intended to be loaded separately
+        let MetricsReporting = dashjs.MetricsReporting; /* jshint ignore:line */
         if (typeof MetricsReporting === 'function') {//TODO need a better way to register/detect plugin components
             let metricsReporting = MetricsReporting(context).create();