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

Conversation

yjbanov
Copy link
Contributor

@yjbanov yjbanov commented Dec 9, 2020

Description

Reuse canvases when window resizes. This significantly improves resize performance, and prevents accumulation of WebGL contexts.

Related Issues

Fixes flutter/flutter#50721

Tests

I added the following tests: test/canvaskit/surface_test.dart.

Copy link
Contributor

@harryterkelsen harryterkelsen left a comment

Choose a reason for hiding this comment

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

LGTM

@cbenhagen
Copy link
Contributor

@yjbanov this will not fix the issue described in flutter/flutter#50721. I don't think it should be closed by this PR.

@chinmaygarde chinmaygarde added the platform-web Code specifically for the web engine label Dec 10, 2020
@yjbanov yjbanov force-pushed the ck-gl-context-cleanup branch from 8984fc5 to 5cc62e9 Compare December 11, 2020 19:52
@yjbanov yjbanov merged commit 3cdb6de into flutter:master Dec 11, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Dec 14, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Dec 14, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Dec 14, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Dec 15, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes platform-web Code specifically for the web engine
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Web] [CanvasKit] WebGL context is not cleaned up / lost
5 participants