Commits · 50e7f2b84b4cbf119a819bc6e7a3f955eb70a006 · abdullh.alsoleman / Front-End

18 Mar, 2024 6 commits

InputDecorator M3 tests migration - Step6 - constraints (#145213) · 50e7f2b8

Bruno Leroux authored Mar 18, 2024

## Description

This PR migrates `InputDecorator.constraints` related tests to M3 and also various regression tests.

It is the fifth step for the M3 test migration for `InputDecorator`.
Step 1: https://github.com/flutter/flutter/pull/142981
Step 2: https://github.com/flutter/flutter/pull/143369
Step 3: https://github.com/flutter/flutter/pull/143520
Step 4: https://github.com/flutter/flutter/pull/144169
Step 5: https://github.com/flutter/flutter/pull/144932

## Related Issue

Related to https://github.com/flutter/flutter/issues/139076

@justinmc A somewhat small PR, I wanted to migrate the small group related to 'InputDecoration.constraints' (starting at line 2801), it adds a test checking min interactive height and two others that shows one way to bypass the standard min interactive height.

50e7f2b8

Update `inherited_theme_test.dart`, `ink_paint_test.dart`,... · 06ed849c

Taha Tesser authored Mar 18, 2024

Update `inherited_theme_test.dart`, `ink_paint_test.dart`, `ink_splash_test.dart`, `opacity_test.dart` for Material 3 (#144013)

Updated unit tests for `Tooltip` to have M2 and M3 versions.

More info in #139076

06ed849c

Roll Flutter Engine from eac9db75f902 to f11015e14d24 (2 revisions) (#145321) · 28fea594

engine-flutter-autoroll authored Mar 18, 2024

https://github.com/flutter/engine/compare/eac9db75f902...f11015e14d24

2024-03-18 skia-flutter-autoroll@skia.org Roll Skia from d32e7eda9307 to 9b08a3f3881c (1 revision) (flutter/engine#51487)
2024-03-18 skia-flutter-autoroll@skia.org Roll Skia from 666607a99603 to d32e7eda9307 (1 revision) (flutter/engine#51485)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC bdero@google.com,rmistry@google.com,zra@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

28fea594

Roll Flutter Engine from aebc0021cd18 to eac9db75f902 (1 revision) (#145308) · c6f2b53e

engine-flutter-autoroll authored Mar 18, 2024

https://github.com/flutter/engine/compare/aebc0021cd18...eac9db75f902

2024-03-18 skia-flutter-autoroll@skia.org Roll Skia from 38f06cbd2f37 to 666607a99603 (2 revisions) (flutter/engine#51484)

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

c6f2b53e

Roll Flutter Engine from 89c775963926 to aebc0021cd18 (1 revision) (#145305) · 03b5ea51

engine-flutter-autoroll authored Mar 18, 2024

https://github.com/flutter/engine/compare/89c775963926...aebc0021cd18

2024-03-18 skia-flutter-autoroll@skia.org Roll Fuchsia Linux SDK from Ow67Jf4AHxeapVlCa... to I5fL_9DJQBS1Bydbz... (flutter/engine#51482)

Also rolling transitive DEPS:
fuchsia/sdk/core/linux-amd64 from Ow67Jf4AHxea to I5fL_9DJQBS1

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

03b5ea51

Roll Flutter Engine from 5c31faa15ab7 to 89c775963926 (1 revision) (#145303) · dd865ce4

engine-flutter-autoroll authored Mar 18, 2024

https://github.com/flutter/engine/compare/5c31faa15ab7...89c775963926

2024-03-17 ian@hixie.ch Minor markup tweaks in render_frame_capture.md (flutter/engine#51480)

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

dd865ce4

17 Mar, 2024 3 commits

Roll Flutter Engine from 4c420dd2dfd9 to 5c31faa15ab7 (2 revisions) (#145294) · 69a3f540

engine-flutter-autoroll authored Mar 17, 2024

https://github.com/flutter/engine/compare/4c420dd2dfd9...5c31faa15ab7

2024-03-17 skia-flutter-autoroll@skia.org Roll Skia from b91914d51880 to 38f06cbd2f37 (1 revision) (flutter/engine#51481)
2024-03-17 zanderso@users.noreply.github.com Pass some missing --disable-dart-dev and --suppress-analytics to dart (flutter/engine#51473)

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

69a3f540

Roll Flutter Engine from cfbf11d757d1 to 4c420dd2dfd9 (2 revisions) (#145280) · 7c860ddf

engine-flutter-autoroll authored Mar 17, 2024

https://github.com/flutter/engine/compare/cfbf11d757d1...4c420dd2dfd9

2024-03-17 skia-flutter-autoroll@skia.org Roll Dart SDK from 6eb2bc37b91c to d5cf2d3849ca (1 revision) (flutter/engine#51478)
2024-03-16 skia-flutter-autoroll@skia.org Roll Fuchsia Linux SDK from nTth7xxzAfENAJjMH... to Ow67Jf4AHxeapVlCa... (flutter/engine#51477)

Also rolling transitive DEPS:
fuchsia/sdk/core/linux-amd64 from nTth7xxzAfEN to Ow67Jf4AHxea

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

7c860ddf

Factor out use of "print" in flutter_goldens (#144846) · c7123c96

Ian Hickson authored Mar 17, 2024

This is part 5 of a broken down version of the #140101 refactor.

This PR removes direct use of `print` and replaces it with a callback.

c7123c96

16 Mar, 2024 16 commits

Roll Flutter Engine from dd4050334ff3 to cfbf11d757d1 (1 revision) (#145275) · b24b6b1a

engine-flutter-autoroll authored Mar 16, 2024

https://github.com/flutter/engine/compare/dd4050334ff3...cfbf11d757d1

2024-03-16 skia-flutter-autoroll@skia.org Roll Dart SDK from cf4f757cb51e to 6eb2bc37b91c (1 revision) (flutter/engine#51476)

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

b24b6b1a

Roll Flutter Engine from 618ee6c8ba34 to dd4050334ff3 (1 revision) (#145270) · ed845c7a

engine-flutter-autoroll authored Mar 16, 2024

https://github.com/flutter/engine/compare/618ee6c8ba34...dd4050334ff3

2024-03-16 skia-flutter-autoroll@skia.org Roll Skia from 70bc8aad2a0a to b91914d51880 (1 revision) (flutter/engine#51472)

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

ed845c7a

Roll Flutter Engine from e4602aa149d8 to 618ee6c8ba34 (1 revision) (#145266) · 16d07313

engine-flutter-autoroll authored Mar 16, 2024

https://github.com/flutter/engine/compare/e4602aa149d8...618ee6c8ba34

2024-03-16 skia-flutter-autoroll@skia.org Roll Skia from 28701a2820eb to 70bc8aad2a0a (1 revision) (flutter/engine#51470)

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

16d07313

Roll Flutter Engine from 91b9cf24d328 to e4602aa149d8 (1 revision) (#145259) · e7baebdc

engine-flutter-autoroll authored Mar 16, 2024

https://github.com/flutter/engine/compare/91b9cf24d328...e4602aa149d8

2024-03-16 skia-flutter-autoroll@skia.org Roll Dart SDK from 934c41c1c18e to cf4f757cb51e (1 revision) (flutter/engine#51469)

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

e7baebdc

Reverts "Reland #128236 "Improve build output for all platforms" (#143166)" (#145261) · 48c1c231

auto-submit[bot] authored Mar 16, 2024

Reverts: flutter/flutter#143166
Initiated by: guidezpl
Reason for reverting: breaks devicelab windows tests
Original PR Author: guidezpl

Reviewed By: {loic-sharma}

This change reverts the following previous change:
Reland #128236, reverted in https://github.com/flutter/flutter/pull/143125.

This PR contains [one additional commit](https://github.com/flutter/flutter/pull/143166/commits/199baea9a99bc905e72d2b069259498dd8f7defd), fixing the 2 failed tests.

## Original description

Improves the build output:

1. Gives confirmation that the build succeeded, in green
1. Gives the path to the built executable, without a trailing period to make it slightly easier to cmd/ctrl+open
1. Gives the size of the built executable (when the built executable is self contained) 

### `apk`, `appbundle` 

<img width="607" alt="image" src="https://github.com/flutter/flutter/assets/6655696/ecc52abe-cd2e-4116-b22a-8385ae3e980d">

<img width="634" alt="image" src="https://github.com/flutter/flutter/assets/6655696/8af8bd33-c0bd-4215-9a06-9652ee019436">

### `macos`, `ios`, `ipa`
Build executables are self-contained and use a newly introduced `OperatingSystemUtils.getDirectorySize`.

<img width="514" alt="image" src="https://github.com/flutter/flutter/assets/6655696/b5918a69-3959-4417-9205-4f501d185257">

<img width="581" alt="image" src="https://github.com/flutter/flutter/assets/6655696/d72fd420-18cf-4470-9e4b-b6ac10fbcd50">

<img width="616" alt="image" src="https://github.com/flutter/flutter/assets/6655696/5f235ce1-252a-4c13-898f-139f6c7bc698">

### `windows`, `linux`, and `web`
Build executables aren't self-contained, and folder size can sometimes overestimate distribution size, therefore their size isn't mentioned (see discussion below).

<img width="647" alt="image" src="https://github.com/flutter/flutter/assets/6655696/7179e771-1eb7-48f6-b770-975bc073437b">

<img width="658" alt="image" src="https://github.com/flutter/flutter/assets/6655696/a6801cab-7b5a-4975-a406-f4c9fa44d7a2">

<img width="608" alt="image" src="https://github.com/flutter/flutter/assets/6655696/ee7c4125-a273-4a65-95d7-ab441edf8ac5">

### Size reporting
When applicable, the printed size matches the OS reported size.

- macOS
    <img width="391" alt="image" src="https://github.com/flutter/flutter/assets/6655696/881cbfb1-d355-444b-ab44-c1a6343190ce">
- Windows
    <img width="338" alt="image" src="https://github.com/flutter/flutter/assets/6655696/3b806def-3d15-48a9-8a25-df200d6feef7">
- Linux   
    <img width="320" alt="image" src="https://github.com/flutter/flutter/assets/6655696/89a4aa3d-2148-4f3b-b231-f93a057fee2b">

## Related issues
Part of #120127
Fixes https://github.com/flutter/flutter/issues/121401

48c1c231

Reland #128236 "Improve build output for all platforms" (#143166) · 2fc76c78

Pierre-Louis authored Mar 16, 2024

Reland #128236, reverted in https://github.com/flutter/flutter/pull/143125.

This PR contains [one additional commit](https://github.com/flutter/flutter/pull/143166/commits/199baea9a99bc905e72d2b069259498dd8f7defd), fixing the 2 failed tests.

## Original description

Improves the build output:

1. Gives confirmation that the build succeeded, in green
1. Gives the path to the built executable, without a trailing period to make it slightly easier to cmd/ctrl+open
1. Gives the size of the built executable (when the built executable is self contained) 

### `apk`, `appbundle` 

<img width="607" alt="image" src="https://github.com/flutter/flutter/assets/6655696/ecc52abe-cd2e-4116-b22a-8385ae3e980d">

<img width="634" alt="image" src="https://github.com/flutter/flutter/assets/6655696/8af8bd33-c0bd-4215-9a06-9652ee019436">

### `macos`, `ios`, `ipa`
Build executables are self-contained and use a newly introduced `OperatingSystemUtils.getDirectorySize`.

<img width="514" alt="image" src="https://github.com/flutter/flutter/assets/6655696/b5918a69-3959-4417-9205-4f501d185257">

<img width="581" alt="image" src="https://github.com/flutter/flutter/assets/6655696/d72fd420-18cf-4470-9e4b-b6ac10fbcd50">

<img width="616" alt="image" src="https://github.com/flutter/flutter/assets/6655696/5f235ce1-252a-4c13-898f-139f6c7bc698">

### `windows`, `linux`, and `web`
Build executables aren't self-contained, and folder size can sometimes overestimate distribution size, therefore their size isn't mentioned (see discussion below).

<img width="647" alt="image" src="https://github.com/flutter/flutter/assets/6655696/7179e771-1eb7-48f6-b770-975bc073437b">

<img width="658" alt="image" src="https://github.com/flutter/flutter/assets/6655696/a6801cab-7b5a-4975-a406-f4c9fa44d7a2">

<img width="608" alt="image" src="https://github.com/flutter/flutter/assets/6655696/ee7c4125-a273-4a65-95d7-ab441edf8ac5">

### Size reporting
When applicable, the printed size matches the OS reported size.

- macOS
    <img width="391" alt="image" src="https://github.com/flutter/flutter/assets/6655696/881cbfb1-d355-444b-ab44-c1a6343190ce">
- Windows
    <img width="338" alt="image" src="https://github.com/flutter/flutter/assets/6655696/3b806def-3d15-48a9-8a25-df200d6feef7">
- Linux   
    <img width="320" alt="image" src="https://github.com/flutter/flutter/assets/6655696/89a4aa3d-2148-4f3b-b231-f93a057fee2b">

## Related issues
Part of #120127
Fixes https://github.com/flutter/flutter/issues/121401

2fc76c78

Introduce `Split` curve (#143130) · 3109b111

Pierre-Louis authored Mar 16, 2024

`Split` is a curve that progresses according to `beginCurve` until
`split`, then according to `endCurve`.

This curve is used with bottom sheets to allow linear finger dragging
and non-linear enter/exit animations. This PR cleans up a previously
private class I introduced and replaces it with the more customisable
`Split`.

Fixes https://github.com/flutter/flutter/issues/51627

Diagram to be added with
https://github.com/flutter/assets-for-api-docs/pull/239

## 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] 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/wiki/Tree-hygiene#overview
[Tree Hygiene]: https://github.com/flutter/flutter/wiki/Tree-hygiene
[test-exempt]:
https://github.com/flutter/flutter/wiki/Tree-hygiene#tests
[Flutter Style Guide]:
https://github.com/flutter/flutter/wiki/Style-guide-for-Flutter-repo
[Features we expect every widget to implement]:
https://github.com/flutter/flutter/wiki/Style-guide-for-Flutter-repo#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/wiki/Tree-hygiene#handling-breaking-changes
[Discord]: https://github.com/flutter/flutter/wiki/Chat

3109b111

Roll Flutter Engine from 120e52b8a1c2 to 91b9cf24d328 (1 revision) (#145256) · 988bee80

engine-flutter-autoroll authored Mar 16, 2024

https://github.com/flutter/engine/compare/120e52b8a1c2...91b9cf24d328

2024-03-16 skia-flutter-autoroll@skia.org Roll Skia from 05e1b6675da4 to 28701a2820eb (1 revision) (flutter/engine#51468)

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

988bee80

Roll Flutter Engine from 3abc25139959 to 120e52b8a1c2 (1 revision) (#145254) · 360d46fa

engine-flutter-autoroll authored Mar 16, 2024

https://github.com/flutter/engine/compare/3abc25139959...120e52b8a1c2

2024-03-16 skia-flutter-autoroll@skia.org Roll Skia from 2d46a81facd8 to 05e1b6675da4 (1 revision) (flutter/engine#51467)

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

360d46fa

Roll Flutter Engine from 6f180615e6ac to 3abc25139959 (1 revision) (#145253) · 483aa35c

engine-flutter-autoroll authored Mar 16, 2024

https://github.com/flutter/engine/compare/6f180615e6ac...3abc25139959

2024-03-16 skia-flutter-autoroll@skia.org Roll Dart SDK from 2241333f69f3 to 934c41c1c18e (1 revision) (flutter/engine#51465)

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

483aa35c

Platform view devicelab ad banner scroll list real ads (#145239) · a2678be7

Jenn Magder authored Mar 16, 2024

Reland https://github.com/flutter/flutter/pull/144745, which was reverted due to a the Android app ads not being set up correctly, crashing on launch: https://github.com/flutter/flutter/pull/145228

Add the missing [`com.google.android.gms.ads.APPLICATION_ID` `meta-data` tag](https://developers.google.com/admob/android/quick-start#import_the_mobile_ads_sdk) to the manifest.

Validated both `platform_views_scroll_perf__timeline_summary` and `platform_views_scroll_perf_impeller__timeline_summary` ran locally on an Android emulator.

Successful presubmit runs:
https://ci.chromium.org/ui/p/flutter/builders/try/Linux_pixel_7pro%20platform_views_scroll_perf__timeline_summary/4/overview
https://ci.chromium.org/ui/p/flutter/builders/try/Linux_pixel_7pro%20platform_views_scroll_perf_impeller__timeline_summary/4/overview

Original commit message:
_________
This benchmark is to measure the platform view performance improvement. 

It is similar to https://github.com/lucalooz/flutter_ads_list_perf

There's still a pending issue https://github.com/flutter/flutter/issues/144339

Fixes https://github.com/flutter/flutter/issues/143534
Fixes https://github.com/flutter/flutter/issues/143257

a2678be7

Roll Flutter Engine from 6ddaa40bf1a6 to 6f180615e6ac (1 revision) (#145251) · 75996665

engine-flutter-autoroll authored Mar 16, 2024

https://github.com/flutter/engine/compare/6ddaa40bf1a6...6f180615e6ac

2024-03-16 flar@google.com Silence gn warning about unused Skia flag (flutter/engine#51464)

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

75996665

Roll Flutter Engine from d46c1fa17dda to 6ddaa40bf1a6 (1 revision) (#145250) · 33bf4757

engine-flutter-autoroll authored Mar 16, 2024

https://github.com/flutter/engine/compare/d46c1fa17dda...6ddaa40bf1a6

2024-03-16 flar@google.com Reland "Add DisplayList Region and Transform benchmarks to CI" (flutter/engine#51458)

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

33bf4757

Roll Flutter Engine from 7f15dc1200bc to d46c1fa17dda (2 revisions) (#145248) · 2ac7b95a

engine-flutter-autoroll authored Mar 16, 2024

https://github.com/flutter/engine/compare/7f15dc1200bc...d46c1fa17dda

2024-03-16 skia-flutter-autoroll@skia.org Roll Dart SDK from 03cc4d817056 to 2241333f69f3 (2 revisions) (flutter/engine#51461)
2024-03-16 chinmaygarde@google.com Re-land "[Impeller] A toolkit for managed handles to Android NDK vended objects." (flutter/engine#51460)

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

2ac7b95a

Use a separate `TextPainter` for intrinsics calculation in `RenderEditable`... · cc017017

LongCatIsLooong authored Mar 16, 2024

Use a separate `TextPainter` for intrinsics calculation in `RenderEditable` and `RenderParagraph` (#144577)

Use a dedicated `TextPainter` for intrinsic size calculation in `RenderEditable` and `RenderParagraph`.

This is an implementation detail so the change should be covered by existing tests. Performance wise this shouldn't be significantly slower since SkParagraph [caches the result of slower operations across different paragraphs](https://github.com/google/skia/blob/9c62e7b382cf387195ef82895530c97ccceda690/modules/skparagraph/src/ParagraphCache.cpp#L254-L272). Existing benchmarks should be able to catch potential regressions (??).

The reason for making this change is to make sure that intrinsic size computations don't destroy text layout artifacts, so I can expose the text layout as a stream of immutable `TextLayout` objects, to signify other render objects that text-layout-dependent-cache (such as caches for `getBoxesForRange` which can be relatively slow to compute) should be invalidated and `markNeedsPaint` needs to be called if the painting logic depended on text layout.
Without this change, the intrinsics/dry layout calculations will add additional events to the text layout stream, which violates the "dry"/non-destructive contract.

cc017017

Roll Flutter Engine from 2db915c64e6e to 7f15dc1200bc (1 revision) (#145245) · 15e90ad2

engine-flutter-autoroll authored Mar 16, 2024

https://github.com/flutter/engine/compare/2db915c64e6e...7f15dc1200bc

2024-03-15 skia-flutter-autoroll@skia.org Roll Skia from 11b1a958c2ac to 2d46a81facd8 (11 revisions) (flutter/engine#51459)

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

15e90ad2

15 Mar, 2024 15 commits

Roll Flutter Engine from ba1115c30381 to 2db915c64e6e (2 revisions) (#145240) · 20105db1

engine-flutter-autoroll authored Mar 15, 2024

https://github.com/flutter/engine/compare/ba1115c30381...2db915c64e6e

2024-03-15 737941+loic-sharma@users.noreply.github.com [Windows] Move to new present callback (flutter/engine#51293)
2024-03-15 matanlurey@users.noreply.github.com Log messages when an image successfully uploads to Skia Gold (flutter/engine#51455)

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

20105db1

Roll Flutter Engine from c2fd5333a5d2 to ba1115c30381 (3 revisions) (#145237) · 55d8b20d

engine-flutter-autoroll authored Mar 15, 2024

https://github.com/flutter/engine/compare/c2fd5333a5d2...ba1115c30381

2024-03-15 matanlurey@users.noreply.github.com Add more explicit logging (just to `stderr`) if a try-job detects an untriaged image (flutter/engine#51454)
2024-03-15 98614782+auto-submit[bot]@users.noreply.github.com Reverts "[Impeller] A toolkit for managed handles to Android NDK vended objects. (#51334)" (flutter/engine#51457)
2024-03-15 chinmaygarde@google.com [Impeller] A toolkit for managed handles to Android NDK vended objects. (flutter/engine#51334)

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

55d8b20d

Renaming the virtual device Nexus5 name to Nexus5.gce_x86 (#145225) · 1c153c38

yusuf-goog authored Mar 15, 2024

This is required to match a rename being made for the devices in firebase lab.

Also removed `version=24` entry, since thats not supported on Nexus5x, per the firebase team.

Bug:b/322830672

*Replace this paragraph with a description of what this PR is changing or adding, and why. Consider including before/after screenshots.*

*List which issues are fixed by this PR. You must list at least one issue. An issue is not required if the PR fixes something trivial like a typo.*

*If you had to change anything in the [flutter/tests] repo, include a link to the migration guide as per the [breaking change policy].*

1c153c38

Roll Flutter Engine from d107f65a8ee6 to c2fd5333a5d2 (4 revisions) (#145226) · 71606af8

engine-flutter-autoroll authored Mar 15, 2024

https://github.com/flutter/engine/compare/d107f65a8ee6...c2fd5333a5d2

2024-03-15 1961493+harryterkelsen@users.noreply.github.com [canvaskit] Fix backdrop filter with platform views as children (flutter/engine#51442)
2024-03-15 skia-flutter-autoroll@skia.org Roll Skia from 54ecc23acf31 to 11b1a958c2ac (1 revision) (flutter/engine#51440)
2024-03-15 flar@google.com [DisplayList] Pre-Compute saveLayer bounds in DLBuilder (flutter/engine#50935)
2024-03-15 skia-flutter-autoroll@skia.org Roll Fuchsia Linux SDK from da-siZ7wPDA0Z0wXJ... to nTth7xxzAfENAJjMH... (flutter/engine#51446)

Also rolling transitive DEPS:
fuchsia/sdk/core/linux-amd64 from da-siZ7wPDA0 to nTth7xxzAfEN

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

71606af8

Reverts "Reland: "Platform view devicelab ad banner scroll list real ads" (#145224)" (#145228) · ca2d4865

auto-submit[bot] authored Mar 15, 2024

Reverts: flutter/flutter#145224
Initiated by: hellohuanlin
Reason for reverting: breaks the tree
Original PR Author: hellohuanlin

Reviewed By: {gmackall, jmagman}

This change reverts the following previous change:
Reland https://github.com/flutter/flutter/pull/144745, which got reverted due to Android lockfile. Fixed by `dart dev/tools/bin/generate_gradle_lockfiles.dart`

*List which issues are fixed by this PR. You must list at least one issue. An issue is not required if the PR fixes something trivial like a typo.*

Fixes https://github.com/flutter/flutter/issues/143534
Fixes https://github.com/flutter/flutter/issues/143257

*If you had to change anything in the [flutter/tests] repo, include a link to the migration guide as per the [breaking change policy].*

ca2d4865

Roll Flutter Engine from 76f33eb0eced to d107f65a8ee6 (3 revisions) (#145223) · c770ca7e

engine-flutter-autoroll authored Mar 15, 2024

https://github.com/flutter/engine/compare/76f33eb0eced...d107f65a8ee6

2024-03-15 skia-flutter-autoroll@skia.org Roll Dart SDK from 406955fe6950 to 03cc4d817056 (1 revision) (flutter/engine#51441)
2024-03-15 dnfield@google.com Add a pixel test to cover multiple clips with a BDF (flutter/engine#51431)
2024-03-15 72562119+tgucio@users.noreply.github.com Add missing baseline check in text layout tests (flutter/engine#51444)

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

c770ca7e

Reland: "Platform view devicelab ad banner scroll list real ads" (#145224) · da4d0e4f

hellohuanlin authored Mar 15, 2024

Reland https://github.com/flutter/flutter/pull/144745, which got reverted due to Android lockfile. Fixed by `dart dev/tools/bin/generate_gradle_lockfiles.dart`

*List which issues are fixed by this PR. You must list at least one issue. An issue is not required if the PR fixes something trivial like a typo.*

Fixes https://github.com/flutter/flutter/issues/143534
Fixes https://github.com/flutter/flutter/issues/143257

*If you had to change anything in the [flutter/tests] repo, include a link to the migration guide as per the [breaking change policy].*

da4d0e4f

Roll Flutter Engine from 1e199ce6239d to 76f33eb0eced (3 revisions) (#145221) · d113ee4c

engine-flutter-autoroll authored Mar 15, 2024

https://github.com/flutter/engine/compare/1e199ce6239d...76f33eb0eced

2024-03-15 skia-flutter-autoroll@skia.org Roll Skia from 52bb1539c5cf to 54ecc23acf31 (11 revisions) (flutter/engine#51438)
2024-03-15 matanlurey@users.noreply.github.com Add `android_debug_unopt_arm64` to `local_engine.json`. (flutter/engine#51427)
2024-03-15 matanlurey@users.noreply.github.com Remove the Android-drawn surface rendering from scenario_app. (flutter/engine#51433)

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

d113ee4c

Run gradle_plugin_*_apk_test on presubmit on flutter_tool changes. (#142090) · a97af209

Jackson Gardner authored Mar 15, 2024

These tests were failing in postsubmit after a change to `packages/flutter_tools`. We should run these in presubmit when `packages/flutter_tools` changes.

a97af209

Update a11y for SliverAppBar (#144437) · 58eb0e8a

hangyu authored Mar 15, 2024

1. Set cacheExtent for sliverAppBar so it's not dropped from the
semantics tree.
2. Update its toolbarOpacity in a11y mode to 1.0. When scrolling in a11y
mode and the focus is back to the sliverAppBar, the content should be
visible.
fixes: https://github.com/flutter/flutter/issues/143437


## Pre-launch Checklist

- [ ] I read the [Contributor Guide] and followed the process outlined
there for submitting PRs.
- [ ] I read the [Tree Hygiene] wiki page, which explains my
responsibilities.
- [ ] I read and followed the [Flutter Style Guide], including [Features
we expect every widget to implement].
- [ ] I signed the [CLA].
- [ ] I listed at least one issue that this PR fixes in the description
above.
- [ ] I updated/added relevant documentation (doc comments with `///`).
- [ ] I added new tests to check the change I am making, or this PR is
[test-exempt].
- [ ] I followed the [breaking change policy] and added [Data Driven
Fixes] where supported.
- [ ] 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/wiki/Tree-hygiene#overview
[Tree Hygiene]: https://github.com/flutter/flutter/wiki/Tree-hygiene
[test-exempt]:
https://github.com/flutter/flutter/wiki/Tree-hygiene#tests
[Flutter Style Guide]:
https://github.com/flutter/flutter/wiki/Style-guide-for-Flutter-repo
[Features we expect every widget to implement]:
https://github.com/flutter/flutter/wiki/Style-guide-for-Flutter-repo#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/wiki/Tree-hygiene#handling-breaking-changes
[Discord]: https://github.com/flutter/flutter/wiki/Chat
[Data Driven Fixes]:
https://github.com/flutter/flutter/wiki/Data-driven-Fixes

58eb0e8a

Roll Packages from b21c5424783a to 756dcc11180b (6 revisions) (#145214) · eeff96d4

engine-flutter-autoroll authored Mar 15, 2024

https://github.com/flutter/packages/compare/b21c5424783a...756dcc11180b

2024-03-15 32538273+ValentinVignal@users.noreply.github.com [go_router] Use `leak_tracker_flutter_testing` (flutter/packages#6210)
2024-03-15 10687576+bparrishMines@users.noreply.github.com [camera_web][google_maps_flutter] Fix tests throwing errors after test completion with manual roll (flutter/packages#6318)
2024-03-14 10687576+bparrishMines@users.noreply.github.com [pigeon] Fixes double prefixes added to enum names for Objc HostApis and FlutterApis (flutter/packages#6263)
2024-03-14 10687576+bparrishMines@users.noreply.github.com [webview_flutter_android][webview_flutter_wkwebview] Adds platform implementations for onHttpError (flutter/packages#6149)
2024-03-14 34871572+gmackall@users.noreply.github.com [image_picker_android] Fix deprecation warnings by branching based on build version, and suppressing only when needed (flutter/packages#6233)
2024-03-14 30870216+gaaclarke@users.noreply.github.com [google_maps_flutter] Started dispatching platform messages from platform thread (flutter/packages#6069)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages-flutter-autoroll
Please CC flutter-ecosystem@google.com,rmistry@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

eeff96d4

Roll pub packages manually (#145170) · 8d54abfc

Loïc Sharma authored Mar 15, 2024

The automated roll failed as a test needs to be updated: https://github.com/flutter/flutter/pull/145167

Fixes: https://github.com/flutter/flutter/issues/139861

8d54abfc

Roll Flutter Engine from 995d8903e074 to 1e199ce6239d (1 revision) (#145196) · 985576db

engine-flutter-autoroll authored Mar 15, 2024

https://github.com/flutter/engine/compare/995d8903e074...1e199ce6239d

2024-03-15 68449066+zijiehe-google-com@users.noreply.github.com [Fuchsia] Remove the use of //build/ in fuchsia (flutter/engine#51072)

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

985576db

Roll Flutter Engine from 622b37282f20 to 995d8903e074 (8 revisions) (#145191) · 9b276be7

engine-flutter-autoroll authored Mar 15, 2024

https://github.com/flutter/engine/compare/622b37282f20...995d8903e074

2024-03-14 matanlurey@users.noreply.github.com Add some header-goodies for et. (flutter/engine#51434)
2024-03-14 jonahwilliams@google.com [Impeller] revert glyph atlas texture recycling. (flutter/engine#51428)
2024-03-14 98614782+auto-submit[bot]@users.noreply.github.com Reverts "Add DisplayList Region and Transform benchmarks to CI (#51429)" (flutter/engine#51432)
2024-03-14 flar@google.com Add DisplayList Region and Transform benchmarks to CI (flutter/engine#51429)
2024-03-14 skia-flutter-autoroll@skia.org Roll Dart SDK from 2bc8b222d01f to 70ca2323a702 (1 revision) (flutter/engine#51430)
2024-03-14 matanlurey@users.noreply.github.com Reduce code-duplication a bit and add more error context across `SkiaGoldClient`. (flutter/engine#51426)
2024-03-14 zanderso@users.noreply.github.com [et] build and run commands disable RBE with a flag or when not available (flutter/engine#51404)
2024-03-14 matej.knopp@gmail.com Fix flakiness in FlutterVSyncWaiterTest.VSyncWorks and FlutterDisplayLinkTest.WorkaroundForFB13482573 (flutter/engine#51405)

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

9b276be7

Roll Flutter Engine from 7b49dc9dd073 to 622b37282f20 (2 revisions) (#145179) · 60674fa3

engine-flutter-autoroll authored Mar 15, 2024

https://github.com/flutter/engine/compare/7b49dc9dd073...622b37282f20

2024-03-14 jonahwilliams@google.com [Impeller] revert round out. (flutter/engine#51395)
2024-03-14 kjlubick@users.noreply.github.com Remove always-true skia_enable_flutter_defines (flutter/engine#51423)

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

60674fa3