Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

[web] cache sample and stencil params #38829

Merged
merged 4 commits into from
Jan 13, 2023

Conversation

jonahwilliams
Copy link
Member

Fixes flutter/flutter#117934

Now the only expensive part is some style recalc/relayout whenever the canvas is eventually resized.

Before

Screenshot 2023-01-12 at 8 09 30 PM

#After

Screenshot 2023-01-12 at 8 08 23 PM

@flutter-dashboard flutter-dashboard bot added the platform-web Code specifically for the web engine label Jan 13, 2023
@flutter-dashboard

This comment was marked as off-topic.

@jonahwilliams
Copy link
Member Author

On flutter/gallery, this reduces the amount of scripting time reporting when resizing by about 25%:

Before:
Screenshot 2023-01-13 at 9 55 47 AM

After:
Screenshot 2023-01-13 at 9 53 14 AM

Copy link
Contributor

@yjbanov yjbanov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@jonahwilliams jonahwilliams added the autosubmit Merge PR when tree becomes green via auto submit App label Jan 13, 2023
@auto-submit auto-submit bot merged commit 7dc5e7e into flutter:main Jan 13, 2023
@jonahwilliams jonahwilliams deleted the cache_params branch January 13, 2023 18:32
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jan 14, 2023
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Jan 14, 2023
…118505)

* 9204745e5 [web] remove runtime effect check (flutter/engine#38832)

* 7dc5e7efa [web] cache sample and stencil params (flutter/engine#38829)
gspencergoog pushed a commit to gspencergoog/flutter that referenced this pull request Jan 19, 2023
…lutter#118505)

* 9204745e5 [web] remove runtime effect check (flutter/engine#38832)

* 7dc5e7efa [web] cache sample and stencil params (flutter/engine#38829)
ricardoamador pushed a commit to ricardoamador/engine that referenced this pull request Jan 25, 2023
* [web] cache sample and stencil params

* test and style

* Update canvaskit_api_test.dart

* Update canvaskit_api_test.dart
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
autosubmit Merge PR when tree becomes green via auto submit App platform-web Code specifically for the web engine
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[web] lookup of gl parameters slows down surface creation
2 participants