web: make ComposeDomNode inline (#1808)

Co-authored-by: Oleksandr Karpovich <[email protected]> (cherry picked from commit 21cda3f)
JetBrains · Feb 10, 2022 · e779768 · e779768
1 parent b3deed4
commit e779768
Showing 1 changed file with 10 additions and 6 deletions.
diff --git a/web/core/src/jsMain/kotlin/org/jetbrains/compose/web/elements/Base.kt b/web/core/src/jsMain/kotlin/org/jetbrains/compose/web/elements/Base.kt
@@ -15,23 +15,25 @@ import org.w3c.dom.svg.SVGElement
 
 @Composable
 @ExplicitGroupsComposable
-private fun <TScope, T> ComposeDomNode(
-    factory: () -> T,
+private inline fun <TScope, T> ComposeDomNode(
+    crossinline factory: () -> T,
     elementScope: TScope,
     attrsSkippableUpdate: @Composable SkippableUpdater<T>.() -> Unit,
-    content: (@Composable TScope.() -> Unit)?
+    content: (@Composable TScope.() -> Unit)
 ) {
     currentComposer.startNode()
     if (currentComposer.inserting) {
-        currentComposer.createNode(factory)
+        currentComposer.createNode {
+            factory()
+        }
     } else {
         currentComposer.useNode()
     }
 
     attrsSkippableUpdate.invoke(SkippableUpdater(currentComposer))
 
     currentComposer.startReplaceableGroup(0x7ab4aae9)
-    content?.invoke(elementScope)
+    content.invoke(elementScope)
     currentComposer.endReplaceableGroup()
     currentComposer.endNode()
 }
@@ -134,7 +136,9 @@ fun <TElement : Element> TagElement(
             }
         },
         elementScope = scope,
-        content = content
+        content = {
+            content?.invoke(this)
+        }
     )
 
     refEffect?.let { effect ->