perf(core): no need for batchUpdated if no selector (NativeScript#9121)

farfromrefug · web-flow · commit 3a14a0a7609e · 2021-01-29T12:19:15.000-08:00
diff --git a/packages/core/ui/styling/style-scope.ts b/packages/core/ui/styling/style-scope.ts
@@ -472,6 +472,9 @@ export class CssState {
 		}
 
 		const matchingSelectors = this._match.selectors.filter((sel) => (sel.dynamic ? sel.match(view) : true));
+		if (!matchingSelectors || matchingSelectors.length === 0) {
+			return;
+		}
 		view._batchUpdate(() => {
 			this.stopKeyframeAnimations();
 			this.setPropertyValues(matchingSelectors);

Original file line number	Diff line number	Diff line change
`@@ -472,6 +472,9 @@ export class CssState {`
`472`	`472`	`}`
`473`	`473`
`474`	`474`	`const matchingSelectors = this._match.selectors.filter((sel) => (sel.dynamic ? sel.match(view) : true));`
	`475`	`+ if (!matchingSelectors \|\| matchingSelectors.length === 0) {`
	`476`	`+ return;`
	`477`	`+ }`
`475`	`478`	`view._batchUpdate(() => {`
`476`	`479`	`this.stopKeyframeAnimations();`
`477`	`480`	`this.setPropertyValues(matchingSelectors);`