Commits · cff36c9922ab17dece454caadf9a972085e15d85 · abdullh.alsoleman / Front-End

13 Oct, 2023 21 commits

Roll Flutter Engine from 93f02f7237db to 799e8de11c12 (6 revisions) (#136553) · cff36c99

engine-flutter-autoroll authored Oct 13, 2023

https://github.com/flutter/engine/compare/93f02f7237db...799e8de11c12

2023-10-13 skia-flutter-autoroll@skia.org Roll Skia from 643d74cf4e0b to 4783e1146146 (2 revisions) (flutter/engine#46890)
2023-10-13 skia-flutter-autoroll@skia.org Roll Dart SDK from 95e2ecee106a to c8143a7c026f (1 revision) (flutter/engine#46889)
2023-10-13 skia-flutter-autoroll@skia.org Roll Skia from 77e73bb2744b to 643d74cf4e0b (2 revisions) (flutter/engine#46888)
2023-10-13 derekx@google.com Use --timeline_recorder=systrace instead of --systrace_timeline (flutter/engine#46884)
2023-10-13 erikgerman917@gmail.com fixed AutofillHints.middleName tanslation for web. (flutter/engine#46429)
2023-10-13 skia-flutter-autoroll@skia.org Roll Skia from 3acf82dcc479 to 77e73bb2744b (2 revisions) (flutter/engine#46886)

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 jsimmons@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

cff36c99

Mark leak in NativeCodec.getNextFrame. (#136514) · 9f512d70
Polina Cherkasova authored Oct 13, 2023

9f512d70
Stop skipping leaks in the test. (#136512) · 2ea541ac
Polina Cherkasova authored Oct 13, 2023

2ea541ac

run tests under dev/tools as part of framework-misc and get them passing (#136501) · c353e641

Christopher Fujino authored Oct 13, 2023

part of https://github.com/flutter/flutter/issues/136500

this should not affect any production code, it merely runs the tests on CI and makes the changes needed to get them passing.

c353e641

Roll Flutter Engine from 60968c892649 to 93f02f7237db (5 revisions) (#136546) · 7d3b7d14

engine-flutter-autoroll authored Oct 13, 2023

https://github.com/flutter/engine/compare/60968c892649...93f02f7237db

2023-10-13 30870216+gaaclarke@users.noreply.github.com [Impeller] Started throwing errors if dart:ui/Image.toByteData fails (flutter/engine#46738)
2023-10-13 skia-flutter-autoroll@skia.org Roll Skia from e566db061ce8 to 3acf82dcc479 (2 revisions) (flutter/engine#46883)
2023-10-13 kjlubick@users.noreply.github.com Fix forward declare and some deprecated enums (flutter/engine#46882)
2023-10-13 skia-flutter-autoroll@skia.org Roll Skia from c640eeed2695 to e566db061ce8 (1 revision) (flutter/engine#46881)
2023-10-13 kjlubick@users.noreply.github.com Update to use GrDirectContexts::MakeVulkan (flutter/engine#46878)

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

7d3b7d14

Remove bringup from tests and move some back to presubmit. Reorganize TESTOWNERS. (#136498) · 9c791f46

Victoria Ashworth authored Oct 13, 2023

We renamed a few tests and therefore had to run them in bringup first
(https://github.com/flutter/flutter/pull/136463). This removes bringup
and moves some of them back to presubmit. Also, reorganize TESTOWNERS
since they're running on hostonly now.

## 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].
- [ ] 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

9c791f46

[leak-tracking] Add leak tracking in test/rendering -2 (#136310) · 86c9b8ea
droidbg authored Oct 13, 2023

86c9b8ea
[leak-tracking] Add leak tracking in test/rendering -3 (#136308) · 8f14104c
droidbg authored Oct 13, 2023

8f14104c

Roll Packages from 93c3f69fa6c6 to fd84e6545a45 (3 revisions) (#136538) · b808e5c2

engine-flutter-autoroll authored Oct 13, 2023

https://github.com/flutter/packages/compare/93c3f69fa6c6...fd84e6545a45

2023-10-12 48316647+yamhoresh@users.noreply.github.com [go_router] Fix missing build parameter in type-safe routes topic documentation (flutter/packages#5040)
2023-10-12 engine-flutter-autoroll@skia.org Roll Flutter (stable) from ead455963c12 to 2f708eb8396e (1 revision) (flutter/packages#5134)
2023-10-12 goderbauer@google.com [web_benchmarks] Bump process version constraints (flutter/packages#5122)

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

b808e5c2

Roll Flutter Engine from f9aed0267352 to 60968c892649 (1 revision) (#136539) · e88bbfbc

engine-flutter-autoroll authored Oct 13, 2023

https://github.com/flutter/engine/compare/f9aed0267352...60968c892649

2023-10-13 zanderso@users.noreply.github.com Don't use objcopy to embed Android icudtl (flutter/engine#46862)

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

e88bbfbc

Roll Flutter Engine from e94f191d0ba4 to f9aed0267352 (2 revisions) (#136537) · 3865e49a

engine-flutter-autoroll authored Oct 13, 2023

https://github.com/flutter/engine/compare/e94f191d0ba4...f9aed0267352

2023-10-13 skia-flutter-autoroll@skia.org Roll Dart SDK from 54ef02480719 to 95e2ecee106a (1 revision) (flutter/engine#46877)
2023-10-13 skia-flutter-autoroll@skia.org Roll Skia from f27c0f222320 to c640eeed2695 (1 revision) (flutter/engine#46874)

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

3865e49a

Roll Flutter Engine from 40ddc30b9d6c to e94f191d0ba4 (1 revision) (#136532) · b4a67ce6

engine-flutter-autoroll authored Oct 13, 2023

https://github.com/flutter/engine/compare/40ddc30b9d6c...e94f191d0ba4

2023-10-13 skia-flutter-autoroll@skia.org Roll ICU from 995db880cff2 to a622de35ac31 (1 revision) (flutter/engine#46873)

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

b4a67ce6

Roll Flutter Engine from 5acdac549034 to 40ddc30b9d6c (1 revision) (#136526) · 0b15c3e0

engine-flutter-autoroll authored Oct 13, 2023

https://github.com/flutter/engine/compare/5acdac549034...40ddc30b9d6c

2023-10-13 skia-flutter-autoroll@skia.org Roll Dart SDK from 849253fd69b6 to 54ef02480719 (2 revisions) (flutter/engine#46871)

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

0b15c3e0

Roll Flutter Engine from b59d779d4f7f to 5acdac549034 (2 revisions) (#136523) · 4603314c

engine-flutter-autoroll authored Oct 13, 2023

https://github.com/flutter/engine/compare/b59d779d4f7f...5acdac549034

2023-10-13 skia-flutter-autoroll@skia.org Roll Fuchsia Mac SDK from XnH2CtqEH7byTHna7... to SDPa6SfaKvRRSUCij... (flutter/engine#46868)
2023-10-13 skia-flutter-autoroll@skia.org Roll Skia from faad330e7120 to f27c0f222320 (1 revision) (flutter/engine#46869)

Also rolling transitive DEPS:
fuchsia/sdk/core/mac-amd64 from XnH2CtqEH7by to SDPa6SfaKvRR

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

4603314c

Roll Flutter Engine from dc30b4cd0239 to b59d779d4f7f (2 revisions) (#136521) · d479b5e2

engine-flutter-autoroll authored Oct 13, 2023

https://github.com/flutter/engine/compare/dc30b4cd0239...b59d779d4f7f

2023-10-13 skia-flutter-autoroll@skia.org Roll Fuchsia Linux SDK from QLiomHC9D-GWh_gyK... to pI1BVH08V0eG0M7sw... (flutter/engine#46867)
2023-10-13 skia-flutter-autoroll@skia.org Roll Skia from aee708412bec to faad330e7120 (1 revision) (flutter/engine#46866)

Also rolling transitive DEPS:
fuchsia/sdk/core/linux-amd64 from QLiomHC9D-GW to pI1BVH08V0eG

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

d479b5e2

Roll Flutter Engine from 31ec5e31a914 to dc30b4cd0239 (1 revision) (#136518) · 962fcd62

engine-flutter-autoroll authored Oct 13, 2023

https://github.com/flutter/engine/compare/31ec5e31a914...dc30b4cd0239

2023-10-13 skia-flutter-autoroll@skia.org Roll Skia from bf557aeaaef8 to aee708412bec (1 revision) (flutter/engine#46865)

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

962fcd62

Roll Flutter Engine from 7a6172e9d34c to 31ec5e31a914 (1 revision) (#136516) · bae67d96

engine-flutter-autoroll authored Oct 13, 2023

https://github.com/flutter/engine/compare/7a6172e9d34c...31ec5e31a914

2023-10-13 skia-flutter-autoroll@skia.org Roll Dart SDK from b0d65ad14354 to 849253fd69b6 (1 revision) (flutter/engine#46864)

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

bae67d96

Roll Flutter Engine from ffb3b5b67f61 to 7a6172e9d34c (1 revision) (#136515) · 43d8a133

engine-flutter-autoroll authored Oct 13, 2023

https://github.com/flutter/engine/compare/ffb3b5b67f61...7a6172e9d34c

2023-10-13 zanderso@users.noreply.github.com Revert "Remove the frontend server wrapper" (flutter/engine#46863)

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

43d8a133

SearchAnchor should dispose created FocusNode and SearchController. (#136120) · 7d794725
Kostia Sokolovskyi authored Oct 13, 2023

7d794725

Roll Flutter Engine from dee90f16aacd to ffb3b5b67f61 (2 revisions) (#136506) · e0b8ec07

engine-flutter-autoroll authored Oct 13, 2023

https://github.com/flutter/engine/compare/dee90f16aacd...ffb3b5b67f61

2023-10-13 skia-flutter-autoroll@skia.org Roll Dart SDK from 34c06f0bcb29 to b0d65ad14354 (3 revisions) (flutter/engine#46859)
2023-10-12 jason-simmons@users.noreply.github.com Temporarily remove the runs of the engine Dart tests with Impeller enabled (flutter/engine#46854)

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

e0b8ec07

Make constraints a covariant argument in RenderBox.computeDryLayout() (#136432) · bc688cf0

Todd Volkert authored Oct 13, 2023

Some render box subclasses have a specific layout contract that is tightly coupled with other render box subclasses (e.g. two private classes in a local project file). In these cases, it is also possible that they use a constraints object that is a subclass of `BoxConstraints`. To allow for this, this change makes the `constraints` argument to `RenderBox.computeDryLayout()` a covariant argument.

For completeness' sake, this updates the other render objects in the rendering package to also use the covariant keyword for this argument.

bc688cf0

12 Oct, 2023 19 commits

[web] remove loading indicator in -d web-server builds (#136482) · 0f082889
Yegor authored Oct 12, 2023
```
Fixes https://github.com/flutter/flutter/issues/135226
```
0f082889

Roll Flutter Engine from cb37ebc81939 to dee90f16aacd (3 revisions) (#136502) · 5f796b29

engine-flutter-autoroll authored Oct 12, 2023

https://github.com/flutter/engine/compare/cb37ebc81939...dee90f16aacd

2023-10-12 skia-flutter-autoroll@skia.org Roll Skia from 8b110fd65de9 to bf557aeaaef8 (1 revision) (flutter/engine#46853)
2023-10-12 uysalere@gmail.com [fuchsia] Add fatal error for Vulkan failure (flutter/engine#46831)
2023-10-12 mdebbar@google.com [web] Stop using `flutterViewEmbedder` for platform views (flutter/engine#46046)

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

5f796b29

Roll Flutter Engine from 7eb20a09073e to cb37ebc81939 (2 revisions) (#136492) · 856475c0

engine-flutter-autoroll authored Oct 12, 2023

https://github.com/flutter/engine/compare/7eb20a09073e...cb37ebc81939

2023-10-12 skia-flutter-autoroll@skia.org Roll Skia from 222fe6b9b0eb to 8b110fd65de9 (1 revision) (flutter/engine#46852)
2023-10-12 skia-flutter-autoroll@skia.org Roll Skia from fdcbbc4d5c25 to 222fe6b9b0eb (2 revisions) (flutter/engine#46849)

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

856475c0

[SingleChildScrollView] Correct the offset pixels if it is out of range during layout (#136239) · 3f722c14

xubaolin authored Oct 12, 2023

Fixes https://github.com/flutter/flutter/issues/105733
FIxes https://github.com/flutter/flutter/issues/135865

The above two issues will not occur on the `ListView` widget, so after analyzing their differences, it was found that the `SingleChildScrollView` pixels were not corrected according to the content dimensions during layout.

### ListView correct the pixels codes
https://github.com/flutter/flutter/blob/5dfd78c2e30483a9a8668ba6b754d089b1e132c8/packages/flutter/lib/src/rendering/viewport.dart#L1462-L1465

We should correct the pixels to prevent the physics ballistic simulations.

This change makes them have the same behavior.

3f722c14

Allow `TapRegion` to consume tap events (#136305) · 22b0a62a

Greg Spencer authored Oct 12, 2023

## Description

In order for `MenuAnchor` menus to be able to not pass on the taps that close their menus, `TapRegion` needed a way to consume them.  This change adds a flag to the `TapRegion`, `consumeOutsideTap` that will consume taps that occur outside of the region if the flag is set (it is false by default). The same flag is added to `MenuAnchor` to allow selecting the behavior for menus.

`TapRegion` consumes the tap event by registering with the gesture arena and immediately resolving the tap as accepted if any regions in a group have `consumeOutsideTap` set to true.

This PR also deprecates `MenuAnchor.anchorTapClosesMenu`, since it is a much more limited version of the same feature that only applied to the anchor itself, and even then only applied to closing the menu, not passing along the tap.  The same functionality can now be implemented by handling a tap on the anchor widget and checking to see if the menu is open before closing it.

## Related Issues
 - https://github.com/flutter/flutter/issues/135327

## Tests
 - Added tests for `TapRegion` to make sure taps are consumed properly.

22b0a62a

Fix doc TODO (#136485) · 7ea5536f

Michael Goderbauer authored Oct 12, 2023

Underlying bug (https://github.com/dart-lang/dartdoc/issues/2276) has been fixed a while ago.

7ea5536f

Roll Flutter Engine from f02006736390 to 7eb20a09073e (4 revisions) (#136478) · 1b797baf

engine-flutter-autoroll authored Oct 12, 2023

https://github.com/flutter/engine/compare/f02006736390...7eb20a09073e

2023-10-12 skia-flutter-autoroll@skia.org Roll Skia from 1d134a8f3ae2 to fdcbbc4d5c25 (7 revisions) (flutter/engine#46844)
2023-10-12 skia-flutter-autoroll@skia.org Roll Fuchsia Mac SDK from 8AXhoo-MMME8NMKt0... to XnH2CtqEH7byTHna7... (flutter/engine#46843)
2023-10-12 chinmaygarde@google.com [Impeller] Specify default border color for texture wrap mode that clamps. (flutter/engine#46806)
2023-10-12 skia-flutter-autoroll@skia.org Roll Fuchsia Linux SDK from giQtXxdSj6NdIViCw... to QLiomHC9D-GWh_gyK... (flutter/engine#46838)

Also rolling transitive DEPS:
fuchsia/sdk/core/linux-amd64 from giQtXxdSj6Nd to QLiomHC9D-GW
fuchsia/sdk/core/mac-amd64 from 8AXhoo-MMME8 to XnH2CtqEH7by

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

1b797baf

Bump file,process,process_runner (#136418) · 43736132
Michael Goderbauer authored Oct 12, 2023

43736132

Roll Flutter Engine from 664657d32992 to f02006736390 (3 revisions) (#136467) · 90b24fb5

engine-flutter-autoroll authored Oct 12, 2023

https://github.com/flutter/engine/compare/664657d32992...f02006736390

2023-10-12 zanderso@users.noreply.github.com Remove the frontend server wrapper (flutter/engine#46440)
2023-10-12 jason-simmons@users.noreply.github.com Roll SwiftShader to include a fix required by the latest Clang toolchain (flutter/engine#46803)
2023-10-12 skia-flutter-autoroll@skia.org Roll Skia from 2dd1579c6921 to 1d134a8f3ae2 (5 revisions) (flutter/engine#46824)

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

90b24fb5

Fix PageView API doc sample fails on Desktop and Web (#135910) · e8a0cf3a

huycozy authored Oct 12, 2023

### Description

This PR aims to improve/fix the PageView API doc sample for Web and Desktop platforms. On these platforms, mouse dragging gestures do not provide natural behavior similar to other desktop applications. This PR will add navigation buttons (indicators) so users can interact with the demo.

### Related issue

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

### Demo video

https://github.com/flutter/flutter/assets/104349824/0f9c60bd-8b18-404e-b5b6-1d594604de31

e8a0cf3a

Add `--trace-to-file` option to `flutter run` (#135713) · 406c4dbf
Derek Xu authored Oct 12, 2023

406c4dbf
[flutter_tools] handle ERROR_INVALID_FUNCTION when trying to symlink across drives (#136424) · 9751fe64
Christopher Fujino authored Oct 12, 2023
```
~~Fixes https://github.com/flutter/flutter/issues/136321~~

edit by @andrewkolos: Fixes https://github.com/flutter/flutter/issues/66224
```
9751fe64

Updates references to `finders.dart` in `controller.dart` to use a namespace. (#136423) · fa827eb2

pdblasi-google authored Oct 12, 2023

Updates references to `finders.dart` in `controller.dart` to use a namespace. This enables us to make use of `finders.dart#find` in `SemanticsController` without anything being shadowed.

Fixes #136421

fa827eb2

Change some tests to run on macs without iOS devices attached (#136463) · d6ec0cf5

Victoria Ashworth authored Oct 12, 2023

Some tests have been running on devicelab bots with iOS devices attached, however, they don't require an iOS device. So moving them to run on bots without iOS devices.

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

d6ec0cf5

Roll Packages from 4b483f242936 to 93c3f69fa6c6 (9 revisions) (#136461) · ee7e41fe

engine-flutter-autoroll authored Oct 12, 2023

https://github.com/flutter/packages/compare/4b483f242936...93c3f69fa6c6

2023-10-12 goderbauer@google.com [shared_preferences] update file version constraints (flutter/packages#5121)
2023-10-12 supercionci@hotmail.it [cross_file] Improved documentation about ignored parameters in IO module. (flutter/packages#4416)
2023-10-11 41873024+droidbg@users.noreply.github.com [in_app_purchase] [#135759] Fix. doc reference finishPurchase to completePurchase. (flutter/packages#5081)
2023-10-10 zeucxb@gmail.com Update repo README issues link (flutter/packages#5106)
2023-10-10 43643339+nohli@users.noreply.github.com [flutter_markdown] Fix typo in readme (flutter/packages#5096)
2023-10-10 43054281+camsim99@users.noreply.github.com [camerax] Implements torch mode (flutter/packages#4903)
2023-10-10 25266387+Leptopoda@users.noreply.github.com [shared_preferences] fix documentation wording (flutter/packages#4986)
2023-10-10 katelovett@google.com [two_dimensional_scrollables] Fix paint bug when rows/columns are pinned and axes are reversed (flutter/packages#5038)
2023-10-10 engine-flutter-autoroll@skia.org Roll Flutter from f52fe4f7 to 83134ac7 (25 revisions) (flutter/packages#5104)

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

ee7e41fe

Fix typo in function name (#136273) · ddc9b40a
Victoria Ashworth authored Oct 12, 2023
```
Rename `removeIOSimulator` to `removeIOSSimulator`.
```
ddc9b40a

Fix chip widgets don't the apply provided `iconTheme` (#135751) · f65dd3ba

Taha Tesser authored Oct 12, 2023

fixes [`Chip.iconTheme` does not apply the icon theme](https://github.com/flutter/flutter/issues/111828)

### Description
- Fix chip widgets that don't utilize the provided `iconTheme`.
- Prevent `iconTheme` with just color from overriding the default icon size.
- Add some missing M3 tests for the chip and chip theme properties.

### Code sample

<details>
<summary>expand to view the code sample</summary> 

```dart
import 'package:flutter/material.dart';

void main() => runApp(const MyApp());

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      debugShowCheckedModeBanner: false,
      theme: ThemeData(useMaterial3: true),
      home: const Example(),
    );
  }
}

class Example extends StatefulWidget {
  const Example({super.key});

  @override
  State<Example> createState() => _ExampleState();
}

class _ExampleState extends State<Example> {
  final bool _isEnable = true;

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Center(
        child: Column(
          mainAxisAlignment: MainAxisAlignment.spaceEvenly,
          children: <Widget>[
            RawChip(
              iconTheme: const IconThemeData(color: Colors.amber),
              avatar: const Icon(Icons.favorite_rounded),
              label: const Text('RawChip'),
              onPressed: () {},
              isEnabled: _isEnable,
            ),
            const Chip(
              iconTheme: IconThemeData(color: Colors.amber),
              avatar: Icon(Icons.favorite_rounded),
              label: Text('Chip'),
              // onDeleted: () {},
            ),
            FilterChip(
              iconTheme: const IconThemeData(color: Colors.amber),
              avatar: const Icon(Icons.favorite_rounded),
              label: const Text('FilterChip'),
              selected: false,
              onSelected: _isEnable ? (bool value) {} : null,
            ),
            InputChip(
              iconTheme: const IconThemeData(color: Colors.amber),
              avatar: const Icon(Icons.favorite_rounded),
              label: const Text('InputChip'),
              isEnabled: _isEnable,
              onPressed: () {},
            ),
            ActionChip(
              iconTheme: const IconThemeData(color: Colors.amber),
              avatar: const Icon(Icons.favorite_rounded),
              label: const Text('ActionChip'),
              onPressed: _isEnable ? () {} : null,
            ),
            ChoiceChip(
              iconTheme: const IconThemeData(color: Colors.amber),
              avatar: const Icon(Icons.favorite_rounded),
              label: const Text('ChoiceChip'),
              selected: false,
              onSelected: _isEnable ? (bool value) {} : null,
            ),
          ],
        ),
      ),
      floatingActionButton: FloatingActionButton(
        onPressed: () {},
        child: const Icon(Icons.add),
      ),
    );
  }
}

```

</details>

### Before
![Screenshot 2023-09-29 at 16 59 39](https://github.com/flutter/flutter/assets/48603081/4bc32032-cff3-4237-812f-86f17ed95337)

### After

![Screenshot 2023-09-29 at 16 55 24](https://github.com/flutter/flutter/assets/48603081/05a1fc52-fb31-4790-a840-18f2e9718241)

f65dd3ba

Roll Flutter Engine from 33a6d21b3364 to 664657d32992 (2 revisions) (#136450) · fad8bda6

engine-flutter-autoroll authored Oct 12, 2023

https://github.com/flutter/engine/compare/33a6d21b3364...664657d32992

2023-10-12 skia-flutter-autoroll@skia.org Roll Dart SDK from 9afac395dfb1 to 34c06f0bcb29 (1 revision) (flutter/engine#46820)
2023-10-12 skia-flutter-autoroll@skia.org Roll ICU from 985b9a6f70e1 to 995db880cff2 (1 revision) (flutter/engine#46818)

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

fad8bda6

Roll Flutter Engine from d00fabf0b919 to 33a6d21b3364 (5 revisions) (#136442) · b89e1df7

engine-flutter-autoroll authored Oct 12, 2023

https://github.com/flutter/engine/compare/d00fabf0b919...33a6d21b3364

2023-10-12 skia-flutter-autoroll@skia.org Roll Fuchsia Mac SDK from Qjz4zE4Oe0AO_7T3f... to 8AXhoo-MMME8NMKt0... (flutter/engine#46814)
2023-10-12 skia-flutter-autoroll@skia.org Roll Skia from 9f5080c58aac to 2dd1579c6921 (1 revision) (flutter/engine#46813)
2023-10-12 skia-flutter-autoroll@skia.org Roll Dart SDK from db2465f106ad to 9afac395dfb1 (1 revision) (flutter/engine#46812)
2023-10-12 skia-flutter-autoroll@skia.org Roll Fuchsia Linux SDK from hMA99PoacaRZVXHgr... to giQtXxdSj6NdIViCw... (flutter/engine#46811)
2023-10-12 skia-flutter-autoroll@skia.org Roll Skia from ff3c2e3d45bd to 9f5080c58aac (1 revision) (flutter/engine#46809)

Also rolling transitive DEPS:
fuchsia/sdk/core/linux-amd64 from hMA99PoacaRZ to giQtXxdSj6Nd
fuchsia/sdk/core/mac-amd64 from Qjz4zE4Oe0AO to 8AXhoo-MMME8

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

b89e1df7