From 2305ea77eeb1b5cc7cd6208a8ae3f52de10819b2 Mon Sep 17 00:00:00 2001
From: Konstantin <konstantin.tskhovrebov@jetbrains.com>
Date: Thu, 30 May 2024 14:42:01 +0200
Subject: [PATCH] [resources] Use cached value synchronously on web. (#4893)

The change speeds resources web rendering up by the reading a cached
value instantly by request (it was being dispatched to the end of the UI
queue in `LaunchedEffect`)

## Release Notes
### Features - Resources
- Speed resources web rendering up by the reading a cached value
instantly
---
 .../compose/resources/ResourceState.web.kt    | 37 ++++++++++++-------
 1 file changed, 24 insertions(+), 13 deletions(-)

diff --git a/components/resources/library/src/webMain/kotlin/org/jetbrains/compose/resources/ResourceState.web.kt b/components/resources/library/src/webMain/kotlin/org/jetbrains/compose/resources/ResourceState.web.kt
index 8317c96d2a..3ef99c9a73 100644
--- a/components/resources/library/src/webMain/kotlin/org/jetbrains/compose/resources/ResourceState.web.kt
+++ b/components/resources/library/src/webMain/kotlin/org/jetbrains/compose/resources/ResourceState.web.kt
@@ -1,10 +1,12 @@
 package org.jetbrains.compose.resources
 
 import androidx.compose.runtime.Composable
-import androidx.compose.runtime.LaunchedEffect
 import androidx.compose.runtime.State
 import androidx.compose.runtime.mutableStateOf
 import androidx.compose.runtime.remember
+import androidx.compose.runtime.rememberCoroutineScope
+import kotlinx.coroutines.CoroutineStart
+import kotlinx.coroutines.launch
 
 @Composable
 internal actual fun <T> rememberResourceState(
@@ -13,11 +15,14 @@ internal actual fun <T> rememberResourceState(
     block: suspend (ResourceEnvironment) -> T
 ): State<T> {
     val environment = LocalComposeEnvironment.current.rememberEnvironment()
-    val state = remember(key1) { mutableStateOf(getDefault()) }
-    LaunchedEffect(key1) {
-        state.value = block(environment)
+    val scope = rememberCoroutineScope()
+    return remember(key1) {
+        val mutableState = mutableStateOf(getDefault())
+        scope.launch(start = CoroutineStart.UNDISPATCHED) {
+            mutableState.value = block(environment)
+        }
+        mutableState
     }
-    return state
 }
 
 @Composable
@@ -28,11 +33,14 @@ internal actual fun <T> rememberResourceState(
     block: suspend (ResourceEnvironment) -> T
 ): State<T> {
     val environment = LocalComposeEnvironment.current.rememberEnvironment()
-    val state = remember(key1, key2) { mutableStateOf(getDefault()) }
-    LaunchedEffect(key1, key2) {
-        state.value = block(environment)
+    val scope = rememberCoroutineScope()
+    return remember(key1, key2) {
+        val mutableState = mutableStateOf(getDefault())
+        scope.launch(start = CoroutineStart.UNDISPATCHED) {
+            mutableState.value = block(environment)
+        }
+        mutableState
     }
-    return state
 }
 
 @Composable
@@ -44,9 +52,12 @@ internal actual fun <T> rememberResourceState(
     block: suspend (ResourceEnvironment) -> T
 ): State<T> {
     val environment = LocalComposeEnvironment.current.rememberEnvironment()
-    val state = remember(key1, key2, key3) { mutableStateOf(getDefault()) }
-    LaunchedEffect(key1, key2, key3) {
-        state.value = block(environment)
+    val scope = rememberCoroutineScope()
+    return remember(key1, key2, key3) {
+        val mutableState = mutableStateOf(getDefault())
+        scope.launch(start = CoroutineStart.UNDISPATCHED) {
+            mutableState.value = block(environment)
+        }
+        mutableState
     }
-    return state
 }
\ No newline at end of file