engine icon indicating copy to clipboard operation
engine copied to clipboard

Avoid unnecessary transform resets for pixel snapping

Open flar opened this issue 1 year ago • 1 comments

During some new development work that might make transform resets more expensive we realized that the resets were mostly coming from the calls to snap the transform to a pixel translate value and many of those were NOPs since the transform was already on a pixel translate value. This PR will avoid those trivially unnecessary reset operations.

flar avatar Mar 11 '24 22:03 flar

Discovered while working on https://github.com/flutter/engine/pull/50935

flar avatar Mar 11 '24 22:03 flar

There were added to fix problems we had with not maintaining full 4x4 transforms during the layer recursion so they are tested someplace indirectly. But, I could add some LSS/DlMCT tests to test them more directly.

flar avatar Mar 13 '24 18:03 flar