conservative updates for await blocks

src/compiler/compile/render_dom/wrappers/AwaitBlock.ts

@@ -188,29 +188,35 @@ export default class AwaitBlockWrapper extends Wrapper {
 			conditions.push(
 				`(${dependencies.map(dep => `'${dep}' in changed`).join(' || ')})`
 			);
-		}
 
-		conditions.push(
+			conditions.push(
-			`${promise} !== (${promise} = ${snippet})`,
+				`${promise} !== (${promise} = ${snippet})`,
-			`@handle_promise(${promise}, ${info})`
+				`@handle_promise(${promise}, ${info})`
-		);
+			);
 
-		block.builders.update.add_line(
+			block.builders.update.add_line(
-			`${info}.ctx = ctx;`
+				`${info}.ctx = ctx;`
-		);
+			);
 
-		if (this.pending.block.has_update_method) {
+			if (this.pending.block.has_update_method) {
-			block.builders.update.add_block(deindent`
+				block.builders.update.add_block(deindent`
-				if (${conditions.join(' && ')}) {
+					if (${conditions.join(' && ')}) {
-					// nothing
+						// nothing
-				} else {
+					} else {
-					${info}.block.p(changed, @assign(@assign({}, ctx), ${info}.resolved));
+						${info}.block.p(changed, @assign(@assign({}, ctx), ${info}.resolved));
-				}
+					}
-			`);
+				`);
+			} else {
+				block.builders.update.add_block(deindent`
+					${conditions.join(' && ')}
+				`);
+			}
 		} else {
-			block.builders.update.add_block(deindent`
+			if (this.pending.block.has_update_method) {
-				${conditions.join(' && ')}
+				block.builders.update.add_block(deindent`
-			`);
+					${info}.block.p(changed, @assign(@assign({}, ctx), ${info}.resolved));
+				`);
+			}
 		}
 
 		if (this.pending.block.has_outro_method) {

test/runtime/samples/await-conservative-update/_config.js

@@ -0,0 +1,16 @@
+import { sleep } from './sleep.js';
+
+export default {
+	html: `
+		<p>loading...</p>
+	`,
+
+	test({ assert, component, target }) {
+		return sleep(50).then(() => {
+			assert.htmlEqual(target.innerHTML, `
+				<p>the answer is 42</p>
+				<p>count: 1</p>
+			`);
+		});
+	}
+};

test/runtime/samples/await-conservative-update/main.svelte

@@ -0,0 +1,19 @@
+<script>
+	import { sleep } from './sleep.js';
+
+	let count = 0;
+
+	const get_promise = () => {
+		return sleep(10).then(() => {
+			count += 1;
+			return 42;
+		});
+	};
+</script>
+
+{#await get_promise()}
+	<p>loading...</p>
+{:then value}
+	<p>the answer is {value}</p>
+	<p>count: {count}</p>
+{/await}

test/runtime/samples/await-conservative-update/sleep.js

@@ -0,0 +1,11 @@
+export let stopped = false;
+
+export const stop = () => stopped = true;
+
+export const sleep = ms => new Promise(f => {
+	if (stopped) return;
+	setTimeout(() => {
+		if (stopped) return;
+		f();
+	}, ms);
+});