Update TLHC PlatformViewWrapper offset without requesting layout - #194899

…tter#194899)

Currently when a TLHC platform view scrolls,
`PlatformViewsController.offset` updates the `FrameLayout.LayoutParams`
margins and calls `viewWrapper.setLayoutParams(layoutParams)`. Calling
`setLayoutParams` triggers `requestLayout()`, which forces a full
measure/layout pass and causes `PlatformViewWrapper` to re-record its
display list and re-render the embedded view into the render target on
every scroll frame, even though only the wrapper's offset changed.

1. Moves offset handling into `PlatformViewWrapper.setOffset(newLeft,
newTop)`.
2. Keeps `FrameLayout.LayoutParams` margins updated in place so that any
subsequent layout pass preserves the offset, and when already laid out,
translates the view directly with `offsetLeftAndRight` and
`offsetTopAndBottom` instead of calling `setLayoutParams`.
3. Calls `onScrollChanged` after translating so
`ViewTreeObserver.OnScrollChangedListener`s (e.g. `PopupWindow` and ad
viewability trackers) still fire when the view moves without a layout
pass.
4. Simplifies `PlatformViewWrapper.onTouchEvent` to translate touches
using `getLeft()` and `getTop()`, matching `FlutterMutatorView`.

Left is before, right is after. There's still some jitter on the right
because the recording process running on the device seems to push us in
to missing a couple frames, but it's nothing like the bad feedback loop
on the left, and also isn't visible locally when I test.


https://github.com/user-attachments/assets/8295b58f-4e3b-4399-8535-a6520c533220

Fixes flutter#183626

## Pre-launch Checklist

- [x] I read the [Contributor Guide] and followed the process outlined
there for submitting PRs.
- [x] I read the [Tree Hygiene] wiki page, which explains my
responsibilities.
- [x] I read and followed the [Flutter Style Guide], including [Features
we expect every widget to implement].
- [x] I signed the [CLA].
- [x] I listed at least one issue that this PR fixes in the description
above.
- [x] I updated/added relevant documentation (doc comments with `///`).
- [x] I added new tests to check the change I am making, or this PR is
[test-exempt].
- [x] I followed the [breaking change policy] and added [Data Driven
Fixes] where supported.
- [x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel
on [Discord].

<!-- Links -->
[Contributor Guide]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#overview
[Tree Hygiene]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md
[test-exempt]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#tests
[Flutter Style Guide]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Style-guide-for-Flutter-repo.md
[Features we expect every widget to implement]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Style-guide-for-Flutter-repo.md#features-we-expect-every-widget-to-implement
[CLA]: https://cla.developers.google.com/
[flutter/tests]: https://github.com/flutter/tests
[breaking change policy]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#handling-breaking-changes
[Discord]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Chat.md
[Data Driven Fixes]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Data-driven-Fixes.md

Co-authored-by: Gray Mackall <[email protected]>