[Optimize] Use platform-owned native render resource ids#7523
Open
ShouruiSong wants to merge 1 commit into
Open
[Optimize] Use platform-owned native render resource ids#7523ShouruiSong wants to merge 1 commit into
ShouruiSong wants to merge 1 commit into
Conversation
Native UI Render text and image display-list operations now reference platform-owned paint resource ids instead of fragment ids. Fragment behaviors keep the last image request key or text bundle identity and request a new platform resource only when that identity changes. Android and iOS now keep resource registries in their renderer contexts, preserve owner ids for event and legacy lookup paths, and let display-list appliers release text and image resources after active display lists stop referencing them. The Fragment Layer Render architecture note and platform API snapshots were updated to match the new contract. TEST: tools/env.sh clang-format -i for changed C++, Objective-C++, and Java files TEST: tools/env.sh git lynx check TEST: ./out/Default/fragment_test_exec --gtest_filter=DisplayListBuilderTest.DrawImageOperation:DisplayListBuilderTest.DrawImageWithBoxIndex TEST: ./out/Default/dom_unittest_exec --gtest_filter=FragmentTest.TestUpdateLayoutAndDefineBoxAndDrawImage:TextFragmentBehaviorTest.* TEST: tools/env.sh ninja -C out/Default dom_unittest_exec TEST: ANDROID_SERIAL=11161FDD4001F3 ./gradlew :LynxAndroid:connectedNoasanDebugAndroidTest -Pandroid.testInstrumentationRunnerArguments.class=com.lynx.tasm.behavior.render.DisplayListApplierTest built Java and native targets, then device install was blocked by INSTALL_FAILED_DEPRECATED_SDK_VERSION because the package targetSdk is 22 on Android 14 TEST: xcodebuild build-for-testing for LynxRendererContextUnitTest and LynxDisplayListApplierUnitTest was attempted, but the workspace failed while copying missing source header lynx/core/runtime/js/bindings/api_invoke_ctrl.h referenced by Pods/Lynx.xcodeproj AutoSubmit: true
jianliang00
force-pushed
the
shadow/p/ShouruiSong/native_render_resource_id
branch
from
e999b8d to
e091b54
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.