Merge pull request #3530 from sveltejs/gh-3524

robustify bind:currentTime

src/compiler/compile/render_dom/wrappers/Element/index.ts

@@ -468,15 +468,25 @@ export default class ElementWrapper extends Wrapper {
 			// TODO dry this out — similar code for event handlers and component bindings
 			if (has_local_function) {
 				// need to create a block-local function that calls an instance-level function
+				if (animation_frame) {
 					block.builders.init.add_block(deindent`
 						function ${handler}() {
-						${animation_frame && deindent`
 							@_cancelAnimationFrame(${animation_frame});
-						if (!${this.var}.paused) ${animation_frame} = @raf(${handler});`}
+							if (!${this.var}.paused) {
+								${animation_frame} = @raf(${handler});
 								${needs_lock && `${lock} = true;`}
+							}
 							ctx.${handler}.call(${this.var}${contextual_dependencies.size > 0 ? ', ctx' : ''});
 						}
 					`);
+				} else {
+					block.builders.init.add_block(deindent`
+						function ${handler}() {
+							${needs_lock && `${lock} = true;`}
+							ctx.${handler}.call(${this.var}${contextual_dependencies.size > 0 ? ', ctx' : ''});
+						}
+					`);
+				}
 
 				callee = handler;
 			} else {

test/js/samples/media-bindings/expected.js

@@ -19,8 +19,10 @@ function create_fragment(ctx) {
 
 	function audio_timeupdate_handler() {
 		cancelAnimationFrame(audio_animationframe);
-		if (!audio.paused) audio_animationframe = raf(audio_timeupdate_handler);
+		if (!audio.paused) {
+			audio_animationframe = raf(audio_timeupdate_handler);
 			audio_updating = true;
+		}
 		ctx.audio_timeupdate_handler.call(audio);
 	}