-
Notifications
You must be signed in to change notification settings - Fork 29.8k
Closed
Labels
a: existing-appsIntegration with existing apps via the add-to-app flowIntegration with existing apps via the add-to-app flowc: performanceRelates to speed or footprint issues (see "perf:" labels)Relates to speed or footprint issues (see "perf:" labels)engineflutter/engine related. See also e: labels.flutter/engine related. See also e: labels.platform-iosiOS applications specificallyiOS applications specifically
Milestone
Description
Traces
https://drive.google.com/file/d/1gFj_fxjHZ_rmvhC7UNGklHScb7ilTBbE/view?usp=sharing
https://drive.google.com/drive/folders/1ZQpyw8bZw1pdCiKduZRil8_Y3Bcx-myt?usp=sharing
Repro
Clone https://github.com/xster/flutter-test/tree/master/t132_performance_test_ios and https://github.com/xster/flutter-test/tree/master/t132_performance_test_flutter.
flutter build ios --debug --no-codesign or wtv in t132_performance_test_flutter
pod install in t132_performance_test_ios run in instruments.
Press 'load engine', 'show vc' and 'pop'.
PlatformView.NotifyDestroyed is called through FlutterViewController.viewDidDisappear. But looking at the VM: IOSurface allocation, they're still not freed.
@chinmaygarde can you take a look?
Metadata
Metadata
Labels
a: existing-appsIntegration with existing apps via the add-to-app flowIntegration with existing apps via the add-to-app flowc: performanceRelates to speed or footprint issues (see "perf:" labels)Relates to speed or footprint issues (see "perf:" labels)engineflutter/engine related. See also e: labels.flutter/engine related. See also e: labels.platform-iosiOS applications specificallyiOS applications specifically


