Dan Field
Dan Field
For my own reference: With this change, on an Android GO ARM 32 device, this experiment does 1000 loops in ``` Current: 0:00:05.476430 New: 0:00:03.629906 ```
(this is in `--profile` mode)
Like this: ``` class BadPath extends parse.PathProxy { Path path = Path(); @override void close() { path.close() } @override void moveTo(double x, double y) { path.moveTo(x, y); } @override void...
I took the example from https://api.flutter.dev/flutter/material/TextField-class.html, but really any example using a text field works (you don't need to have any properties set on it).
Bisected this to https://github.com/flutter/flutter/pull/105011 @jpnurmi @wmww both had linux based changes in there.
Looks like https://github.com/flutter/engine/pull/33111 broke it @jpnurmi
``` name of display: :20 display: :20 screen: 0 direct rendering: Yes Extended renderer info (GLX_MESA_query_renderer): Vendor: Mesa/X.org (0xffffffff) Device: llvmpipe (LLVM 13.0.1, 256 bits) (0xffffffff) Version: 21.3.8 Accelerated: no...
> Reverting [flutter/engine#33111](https://github.com/flutter/engine/pull/33111) is not straightforward because there are several conflicting changes on top. It might be more reasonable to add a window delegate interface to be able to move...
> Could you give it a try with https://github.com/jpnurmi/engine/commits/text-input-view-delegate please? This patch makes the bug go away.
@iskakaushik do you know why OpenGL would be used here? I was under the impression Metal is the default for macOS. This is a bug that likely would not exist...