@heff added back the default cdn url for the swf

closes #2533
fixes #2488

CHANGELOG.md

@@ -117,6 +117,7 @@ CHANGELOG
 * @gkatsev made the sass files available via npm in src/css ([view](https://github.com/videojs/video.js/pull/2546))
 * @heff removed playerOptions from plugin options because it created an inconsistency in plugin inits ([view](https://github.com/videojs/video.js/pull/2532))
 * @heff added a default data attribute to fix the progress handle display in IE8 ([view](https://github.com/videojs/video.js/pull/2547))
+* @heff added back the default cdn url for the swf ([view](https://github.com/videojs/video.js/pull/2533))
 
 --------------------
 

grunt.js

@@ -259,6 +259,10 @@ module.exports = function(grunt) {
           ['browserify-versionify', {
             placeholder: '__VERSION_NO_PATCH__',
             version: version.majorMinor
+          }],
+          ['browserify-versionify', {
+            placeholder: '__SWF_VERSION__',
+            version: pkg.dependencies['videojs-swf']
           }]
         ]
       },

src/js/tech/flash.js

@@ -69,6 +69,14 @@ class Flash extends Tech {
   createEl() {
     let options = this.options_;
 
+    // If video.js is hosted locally you should also set the location
+    // for the hosted swf, which should be relative to the page (not video.js)
+    // Otherwise this adds a CDN url.
+    // The CDN also auto-adds a swf URL for that specific version.
+    if (!options.swf) {
+      options.swf = '//vjs.zencdn.net/swf/__SWF_VERSION__/video-js.swf';
+    }
+
     // Generate ID for swf object
     let objId = options.techId;