Added capability to remove debug functions when building minified ver…

…sion. Corrected feature detect to avoid latest Chrome. Removing cruft

Jakefile

@@ -112,13 +112,27 @@ file("build",[],function() {
 
 desc("Minifies an already built Captionator.js.");
 task("minify",["build"],function() {
+
+	// Remove debug code
+	function removeDebugCode(codeIn) {
+		var codeParts = codeIn.split(/\/\/\s*DEBUG\->/g);
+
+		return codeParts.map(function(part) {
+				if (part.match(/\/\/\s*<\-DEBUG/g)) {
+					return part.split(/\s*\/\/\s*<\-DEBUG/g).pop();
+				}
+				// console.log(part);
+				return part;
+			}).join("");
+	}
+
 	console.log("Minifying...");
 	fs.readFile("./js/captionator.js",function(err,data) {
 		if (err) {
 			fail("Unable to read unminified captionator.js infile.",1);
 		} else {
 			var unminifiedLength = data.length;
-			var captionatorData = data.toString("utf8");
+			var captionatorData = removeDebugCode(data.toString("utf8"));
 			var ast = uglifyParser.parse(captionatorData);
 			ast = uglifyProcessor.ast_mangle(ast);
 			ast = uglifyProcessor.ast_squeeze(ast);

index.html

@@ -33,7 +33,7 @@ <h1>HTML5 Video Closed Captioning Example</h1>
 		<h2 id="mediacontrols">Media Controls</h2>
 
 		<!-- Include Captionator -->
-		<script type="text/javascript" src="js/captionator.js"></script>
+		<script type="text/javascript" src="js/captionator-min.js"></script>
 
 		<!-- Example Usage -->
 		<script type="text/javascript" src="js/captionator-example-api.js"></script>

js/captionator.js

@@ -649,11 +649,12 @@
 		if (!HTMLVideoElement && !(element instanceof VirtualMediaContainer) ) {
 			// Browser doesn't support HTML5 video - die here.
 			return false;
-		} else {
-			// Browser supports native track API
-			if (typeof(document.createElement("video").addTextTrack) === "function") {
-				return false;
-			}
+		}
+
+		// Browser supports native track API
+		// This should catch Chrome latest and IE10.
+		if (typeof(document.createElement("video").addTextTrack) === "function" || typeof(document.createElement("video").addTrack) === "function") {
+			return false;
 		}
 
 		// if requested by options, export the object types
@@ -1785,6 +1786,8 @@
 				videoElement._captionator_availableCueArea.top;
 		}
 
+		// DEBUG->
+
 		// DEBUG FUNCTIONS
 		// This function can be used for debugging WebVTT captions. It will not be
 		// included in production versions of Captionator.
@@ -1886,6 +1889,7 @@
 			drawLines();
 		}
 		// END DEBUG FUNCTIONS
+		// <-DEBUG
 	};
 
 	/*

source/captionator.general.init.js

@@ -54,11 +54,12 @@ captionator.captionify = function(element,defaultLanguage,options) {
 	if (!HTMLVideoElement && !(element instanceof VirtualMediaContainer) ) {
 		// Browser doesn't support HTML5 video - die here.
 		return false;
-	} else {
-		// Browser supports native track API
-		if (typeof(document.createElement("video").addTextTrack) === "function") {
-			return false;
-		}
+	}
+
+	// Browser supports native track API
+	// This should catch Chrome latest and IE10.
+	if (typeof(document.createElement("video").addTextTrack) === "function" || typeof(document.createElement("video").addTrack) === "function") {
+		return false;
 	}
 
 	// if requested by options, export the object types

source/captionator.general.style.js

@@ -466,6 +466,8 @@ captionator.styleCue = function(DOMNode, cueObject, videoElement) {
 			videoElement._captionator_availableCueArea.top;
 	}
 
+	// DEBUG->
+
 	// DEBUG FUNCTIONS
 	// This function can be used for debugging WebVTT captions. It will not be
 	// included in production versions of Captionator.
@@ -567,6 +569,7 @@ captionator.styleCue = function(DOMNode, cueObject, videoElement) {
 		drawLines();
 	}
 	// END DEBUG FUNCTIONS
+	// <-DEBUG
 };
 
 /*