timeRangesToArray: handle ranges argument being undefined

This occurs when the media element initially has no src set.
sveltejs · Sep 10, 2017 · 2782a6f · 2782a6f
1 parent 687acf7
commit 2782a6f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/src/shared/dom.js b/src/shared/dom.js
@@ -103,6 +103,7 @@ export function toNumber(value) {
 }
 
 export function timeRangesToArray(ranges) {
+	if (!ranges) return [];
 	var array = [];
 	for (let i = 0; i < ranges.length; i += 1) {
 		array.push({ start: ranges.start(i), end: ranges.end(i) });
@@ -149,4 +150,4 @@ export function setInputType(input, type) {
 
 export function setStyle(node, key, value) {
 	node.style.setProperty(key, value);
-}
+}
diff --git a/test/js/samples/media-bindings/expected-bundle.js b/test/js/samples/media-bindings/expected-bundle.js
@@ -34,6 +34,7 @@ function removeListener(node, event, handler) {
 }
 
 function timeRangesToArray(ranges) {
+	if (!ranges) return [];
 	var array = [];
 	for (let i = 0; i < ranges.length; i += 1) {
 		array.push({ start: ranges.start(i), end: ranges.end(i) });