Commits · 5caa8b896200b9e4e2149f71ba0488bcf550f9b0 · abdullh.alsoleman / Front-End

22 Jan, 2024 2 commits

Roll Flutter Engine from c485c574fb1d to 11c585f48b77 (1 revision) (#141952) · 5caa8b89

engine-flutter-autoroll authored Jan 22, 2024

https://github.com/flutter/engine/compare/c485c574fb1d...11c585f48b77

2024-01-22 skia-flutter-autoroll@skia.org Roll Skia from 97a00523af28 to 8774ef8a314e (1 revision) (flutter/engine#49927)

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

5caa8b89

Roll Flutter Engine from a7b207d5a1fe to c485c574fb1d (1 revision) (#141951) · fb3a1e2b

engine-flutter-autoroll authored Jan 22, 2024

https://github.com/flutter/engine/compare/a7b207d5a1fe...c485c574fb1d

2024-01-22 skia-flutter-autoroll@skia.org Roll Skia from c42ac527ff0e to 97a00523af28 (1 revision) (flutter/engine#49926)

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

fb3a1e2b

21 Jan, 2024 4 commits

Roll Flutter Engine from 2b31ad2fb819 to a7b207d5a1fe (1 revision) (#141945) · 5dea6b99

engine-flutter-autoroll authored Jan 21, 2024

https://github.com/flutter/engine/compare/2b31ad2fb819...a7b207d5a1fe

2024-01-21 skia-flutter-autoroll@skia.org Roll Fuchsia Linux SDK from kOnlSGX_aTAK7vvLc... to p0njgljtokVLYfzqK... (flutter/engine#49924)

Also rolling transitive DEPS:
fuchsia/sdk/core/linux-amd64 from kOnlSGX_aTAK to p0njgljtokVL

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

5dea6b99

[flutter_tools] update analyze_once_test.dart to be null-safe (#141790) · 92094802

Christopher Fujino authored Jan 21, 2024

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

I should have made this change in https://github.com/flutter/flutter/pull/124039, but it escaped my grep search.

92094802

Adjust the position of require File.expand_path (#141521) · f340d207

林洵锋 authored Jan 21, 2024

On `Podfile`:

```ruby
flutter_application_path = '../flutter_module'
load File.join(flutter_application_path, '.ios', 'Flutter', 'podhelper.rb')

target 'OCProject' do
  # Comment the next line if you don't want to use dynamic frameworks
  use_frameworks!

  # Pods for OCProject
  # install_all_flutter_pods(flutter_application_path)
  # install_flutter_engine_pod(flutter_application_path)
  # install_flutter_application_pod(flutter_application_path)
  install_flutter_plugin_pods(flutter_application_path)

end

post_install do |installer|
  flutter_post_install(installer)
end
```
Encountering the following error after executing `pod install`:

```shell
pod install

[!] Invalid `Podfile` file: undefined method `flutter_relative_path_from_podfile' for #<Pod::Podfile:0x000000010e74c520 @defined_in_file=#<Pathname:/Users/lxf/gitHub/flutter_hybrid_bug/OCProject/Podfile>, @internal_hash={}, @root_target_definitions=[#<Pod::Podfile::TargetDefinition label=Pods>], @current_target_definition=#<Pod::Podfile::TargetDefinition label=Pods>>

  relative = flutter_relative_path_from_podfile(export_script_directory)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^.

 #  from /Users/lxf/gitHub/flutter_hybrid_bug/OCProject/Podfile:17
 #  -------------------------------------------
 #    # install_flutter_plugin_pods(flutter_application_path)
 >    install_flutter_application_pod(flutter_application_path)
 #
 #  -------------------------------------------
```

The `flutter_relative_path_from_podfile` method is in `flutter_tools/bin/podhelper.rb`, but now `flutter_tools/bin/podhelper.rb` is only required in `install_all_flutter_pods` in `podhelper.rb.tmpl`.

Sometimes we only need to use the `install_flutter_plugin_pods` method in podhelper.rb. For example, using `Shorebird` in an iOS hybird app scenario, we need to build `Flutter.xcframework` and `App.xcframework` and embed them into the iOS native project. In order to avoid unnecessary conflicts, use `install_flutter_plugin_pods` method to install Flutter plugin pods.

[Shorebird - Code Push In Hybrid Apps](https://docs.shorebird.dev/guides/hybrid-app/ios)

So I adjust the position of `require File.expand_path(File.join('packages', 'flutter_tools', 'bin', 'podhelper'), flutter_root)`.

f340d207

Add RadioListItem use-case to a11y_assessments (#140984) · a9faaf26

Yegor authored Jan 21, 2024

Adds a use-case screen for `RadioListTile`, similar to the `CheckBoxListTile`. This screen can help test scenarios such as the one reported in https://github.com/flutter/flutter/issues/126805.

a9faaf26

20 Jan, 2024 8 commits

Roll Flutter Engine from 704ef3399012 to 2b31ad2fb819 (1 revision) (#141937) · 954d30f0

engine-flutter-autoroll authored Jan 20, 2024

https://github.com/flutter/engine/compare/704ef3399012...2b31ad2fb819

2024-01-20 skia-flutter-autoroll@skia.org Roll Skia from 3cac1efbdd7a to c42ac527ff0e (1 revision) (flutter/engine#49921)

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

954d30f0

Roll Flutter Engine from 1385f76018ab to 704ef3399012 (1 revision) (#141932) · ddf60fb0

engine-flutter-autoroll authored Jan 20, 2024

https://github.com/flutter/engine/compare/1385f76018ab...704ef3399012

2024-01-20 skia-flutter-autoroll@skia.org Roll Fuchsia Linux SDK from geXIwPzzJiSXrZEEb... to kOnlSGX_aTAK7vvLc... (flutter/engine#49919)

Also rolling transitive DEPS:
fuchsia/sdk/core/linux-amd64 from geXIwPzzJiSX to kOnlSGX_aTAK

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

ddf60fb0

Roll Flutter Engine from 6f6649bdfef2 to 1385f76018ab (1 revision) (#141926) · 3c1cb8ac

engine-flutter-autoroll authored Jan 20, 2024

https://github.com/flutter/engine/compare/6f6649bdfef2...1385f76018ab

2024-01-20 skia-flutter-autoroll@skia.org Manual roll Dart SDK from 8d697eaa395c to 7a91ef52d200 (5 revisions) (flutter/engine#49917)

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

3c1cb8ac

Roll Flutter Engine from 4d67f267b115 to 6f6649bdfef2 (1 revision) (#141920) · 3225cfeb

engine-flutter-autoroll authored Jan 20, 2024

https://github.com/flutter/engine/compare/4d67f267b115...6f6649bdfef2

2024-01-20 skia-flutter-autoroll@skia.org Roll Skia from 61a2dcff4194 to 3cac1efbdd7a (4 revisions) (flutter/engine#49916)

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

3225cfeb

Roll Flutter Engine from 53a2436cf75c to 4d67f267b115 (1 revision) (#141917) · 61c3455b

engine-flutter-autoroll authored Jan 20, 2024

https://github.com/flutter/engine/compare/53a2436cf75c...4d67f267b115

2024-01-20 jonahwilliams@google.com [Impeller] Encode directly to command buffer for Metal. (flutter/engine#49785)

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

61c3455b

Roll Flutter Engine from a9b87c6288d2 to 53a2436cf75c (1 revision) (#141916) · 0d73663d

engine-flutter-autoroll authored Jan 20, 2024

https://github.com/flutter/engine/compare/a9b87c6288d2...53a2436cf75c

2024-01-20 jonahwilliams@google.com [Impeller] use string view for labels in more places to defer allocatons. (flutter/engine#49905)

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

0d73663d

Roll Flutter Engine from d00e55f7f8ed to a9b87c6288d2 (3 revisions) (#141914) · 3842c24a

engine-flutter-autoroll authored Jan 20, 2024

https://github.com/flutter/engine/compare/d00e55f7f8ed...a9b87c6288d2

2024-01-20 jonahwilliams@google.com [Impeller] null check vertex buffer. (flutter/engine#49915)
2024-01-19 30870216+gaaclarke@users.noreply.github.com [Impeller] adds vulkan golden images (flutter/engine#49849)
2024-01-19 98614782+auto-submit[bot]@users.noreply.github.com Reverts "[Fuchsia] Redo - Use chromium test-scripts to download images and execute tests" (flutter/engine#49908)

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

3842c24a

Remove more textScaleFactor references (#141816) · 5892a003

LongCatIsLooong authored Jan 20, 2024

Remove more `textScaleFactor` references from flutter/flutter.  

- Some changes are related to label scaling: the padding EdgeInsets values of some chip subclasses scale linearly between predetermined "max" padding values and "min" padding values. Before they scale with the `textScaleFactor` scalar, now they scale with the font size and are still capped at the original "max" and "min" values.
- The rest of them are tests or size heuristics that depend on `textScaleFactor`, these are replaced by an effective text scale factor computed using a default font size (which is determined in a pretty random fashion, but it will only make a difference on Android 14+).

No API changes in this batch. There are still some references left that I intend to remove in a different batch that would introduce API changes.

5892a003

19 Jan, 2024 20 commits

Roll Flutter Engine from 517fc2bbfa51 to d00e55f7f8ed (1 revision) (#141909) · bc254ce3

engine-flutter-autoroll authored Jan 19, 2024

https://github.com/flutter/engine/compare/517fc2bbfa51...d00e55f7f8ed

2024-01-19 737941+loic-sharma@users.noreply.github.com [Windows] Refactor EGL initialization (flutter/engine#49895)

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

bc254ce3

Roll Flutter Engine from f2b441a26416 to 517fc2bbfa51 (2 revisions) (#141904) · cd6fe510

engine-flutter-autoroll authored Jan 19, 2024

https://github.com/flutter/engine/compare/f2b441a26416...517fc2bbfa51

2024-01-19 skia-flutter-autoroll@skia.org Roll Skia from fabb71dcf9b7 to 61a2dcff4194 (1 revision) (flutter/engine#49904)
2024-01-19 skia-flutter-autoroll@skia.org Roll Skia from c09908c48f14 to fabb71dcf9b7 (3 revisions) (flutter/engine#49902)

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

cd6fe510

Fix "Delete" tooltip is shown disabled on chips with `onDeleted` callback (#141770) · 788614d1

Taha Tesser authored Jan 19, 2024

fixes [Disabled chips with `onDeleted` callback shows "Delete" tooltip on hover](https://github.com/flutter/flutter/issues/141336)

### 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 const MaterialApp(
      debugShowCheckedModeBanner: false,
      home: Example(),
    );
  }
}

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

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

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

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Center(
        child: Column(
          mainAxisAlignment: MainAxisAlignment.spaceEvenly,
          children: <Widget>[
            RawChip(
              label: const Text('RawChip'),
              onPressed: () {},
              isEnabled: _isEnable,
              onDeleted: () {},
            ),
            FilterChip(
              label: const Text('FilterChip'),
              selected: false,
              onSelected: _isEnable ? (bool value) {} : null,
              onDeleted: () {},
            ),
            InputChip(
              label: const Text('InputChip'),
              isEnabled: _isEnable,
              onDeleted: () {},
            ),
          ],
        ),
      ),
      floatingActionButton: FloatingActionButton.extended(
        onPressed: () {
          setState(() {
            _isEnable = !_isEnable;
          });
        },
        label: Text(_isEnable ? 'Disable' : 'Enable'),
      ),
    );
  }
}
```

</details>

### Preview

| Before | After |
| --------------- | --------------- |
| <img src="https://github.com/flutter/flutter/assets/48603081/f80ae5f7-0a6d-4041-ade3-cbc2b5c78188" height="450" /> | <img src="https://github.com/flutter/flutter/assets/48603081/04e62854-e3f1-4b65-9753-183d288f3cfe" height="450" /> |

788614d1

Run module_test_ios on arm and x64 (#141815) · 6f7aed59
godofredoc authored Jan 19, 2024
```
This will ensure the test runs on the supported platforms.
```
6f7aed59
`SearchAnchor` search view clear button only shows up when text input is not empty (#141755) · 05854afa
Qun Cheng authored Jan 19, 2024

05854afa

Use Integer instead of int in map in flutter.groovy (#141895) · 684247a3

Reid Baker authored Jan 19, 2024

packages Roller breakage 
https://ci.chromium.org/ui/p/flutter/builders/try/Linux_android%20android_build_all_packages%20master/5504/overview
Fixes flutter/flutter/issues/141897
```
FAILURE: Build failed with an exception.

* Where:
Script '/b/s/w/ir/x/w/flutter/packages/flutter_tools/gradle/src/main/groovy/flutter.groovy' line: 168

* What went wrong:
Could not compile script '/b/s/w/ir/x/w/flutter/packages/flutter_tools/gradle/src/main/groovy/flutter.groovy'.
> startup failed:
  script '/b/s/w/ir/x/w/flutter/packages/flutter_tools/gradle/src/main/groovy/flutter.groovy': 168: primitive type parameters not allowed here;
     solution: use the corresponding wrapper type, such as Integer for int @ line 168, column 41.
     e static final Map<String, int> ABI_VERS
```

Covered by tests in packages.

684247a3

Roll Flutter Engine from c953c83112ba to f2b441a26416 (4 revisions) (#141894) · f0e9d232

engine-flutter-autoroll authored Jan 19, 2024

https://github.com/flutter/engine/compare/c953c83112ba...f2b441a26416

2024-01-19 jonahwilliams@google.com Reland: Encode directly to command buffer. (flutter/engine#49821)
2024-01-19 skia-flutter-autoroll@skia.org Roll Skia from 2536dc6fef1d to c09908c48f14 (6 revisions) (flutter/engine#49899)
2024-01-19 dnfield@google.com [Impeller] Documentation for SurfaceContextVK (flutter/engine#49882)
2024-01-19 32242716+ricardoamador@users.noreply.github.com Adding ignore paths to cache test (flutter/engine#49874)

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

f0e9d232

Native assets: package in framework on iOS and MacOS (#140907) · 2e229be2

Daco Harkes authored Jan 19, 2024

Packages the native assets for iOS and MacOS in frameworks.

Issue:

* https://github.com/flutter/flutter/issues/140544
* https://github.com/flutter/flutter/issues/129757

## Details

* [x] This packages dylibs from the native assets feature in frameworks. It packages every dylib in a separate framework.
* [x] The dylib name is updated to use `@rpath` instead of `@executable_path`.
* [x] The dylibs for flutter-tester are no longer modified to change the install name. (Previously it was wrongly updating the install name to the location the dylib would have once deployed in an app.)
* [x] Use symlinking on MacOS.

2e229be2

Revert "Make tests more resilient to Skia gold failures and refactor... · 77c3807c

Ian Hickson authored Jan 19, 2024

Revert "Make tests more resilient to Skia gold failures and refactor flutter_goldens for extensive technical debt removal (#140101)" (#141814)

Reverts https://github.com/flutter/flutter/pull/140101

That PR somehow made non-matching gold tests not fail at HEAD.

Fixes https://github.com/flutter/flutter/issues/141880
- Blocked by https://github.com/flutter/flutter/issues/140169
  - https://github.com/flutter/flutter/pull/141427

77c3807c

Roll Flutter Engine from 538975f2511b to c953c83112ba (3 revisions) (#141886) · 49447fc6

engine-flutter-autoroll authored Jan 19, 2024

https://github.com/flutter/engine/compare/538975f2511b...c953c83112ba

2024-01-19 flar@google.com [Impeller] switch Rect fields to LTRB implementation (flutter/engine#49816)
2024-01-19 68449066+zijiehe-google-com@users.noreply.github.com [Fuchsia] Redo - Use chromium test-scripts to download images and execute tests (flutter/engine#49847)
2024-01-19 skia-flutter-autoroll@skia.org Roll Skia from fb3f61d932e6 to 2536dc6fef1d (2 revisions) (flutter/engine#49892)

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

49447fc6

Add `showDragHandle` to `showBottomSheet` (#141754) · ba4a11da
Qun Cheng authored Jan 19, 2024

ba4a11da

Roll Flutter Engine from 9a6c64de8a46 to 538975f2511b (8 revisions) (#141881) · 3563372f

engine-flutter-autoroll authored Jan 19, 2024

https://github.com/flutter/engine/compare/9a6c64de8a46...538975f2511b

2024-01-19 skia-flutter-autoroll@skia.org Roll Skia from 0a78d1f0ef48 to fb3f61d932e6 (1 revision) (flutter/engine#49891)
2024-01-19 skia-flutter-autoroll@skia.org Roll Skia from 961ad2c3fcf5 to 0a78d1f0ef48 (1 revision) (flutter/engine#49890)
2024-01-19 103135467+sealesj@users.noreply.github.com Re-enable osv-scanner on pushes to main (flutter/engine#49729)
2024-01-19 skia-flutter-autoroll@skia.org Roll Skia from 273b0628a969 to 961ad2c3fcf5 (1 revision) (flutter/engine#49889)
2024-01-19 skia-flutter-autoroll@skia.org Roll Fuchsia Linux SDK from dTy8CK4YxnUsW305O... to geXIwPzzJiSXrZEEb... (flutter/engine#49888)
2024-01-19 skia-flutter-autoroll@skia.org Roll Skia from b6fa819eb54c to 273b0628a969 (1 revision) (flutter/engine#49887)
2024-01-19 skia-flutter-autoroll@skia.org Roll Skia from e073a4fad57a to b6fa819eb54c (1 revision) (flutter/engine#49886)
2024-01-19 leroux_bruno@yahoo.fr [Android] Fix KeyEmbedderResponder throws a NullPointerException (flutter/engine#49860)

Also rolling transitive DEPS:
fuchsia/sdk/core/linux-amd64 from dTy8CK4YxnUs to geXIwPzzJiSX

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

3563372f

Make pumpWidget's arguments named (#141728) · cc544169

Michael Goderbauer authored Jan 19, 2024

Much nicer calling API and simplifies evolving this API in the future.

I wish we could write a dart fix for this, but that's blocked on https://github.com/dart-lang/sdk/issues/54668.

cc544169

Tiny fix inaccurate documentations about bindings (#140282) · 9e024fdf

fzyzcjy authored Jan 19, 2024

The old doc says that, AutomatedTestWidgetsFlutterBinding for `flutter test` and LiveTestWidgetsFlutterBinding for `flutter run`. However, suppose we `flutter test integration_test/simple_test.dart` with the following code:

```
void main() {
  testWidgets('hi', (WidgetTester tester) async {
    print('hi ${TestWidgetsFlutterBinding.instance} ${Platform.operatingSystem}');
  });
}
```

We will see: `hi <IntegrationTestWidgetsFlutterBinding> ios`. Therefore, we see `IntegrationTestWidgetsFlutterBinding` is used in a `flutter test` command, which is contrary to the documentation.

9e024fdf

Roll engine to 9a6c64de8a4694cef59a338cd33ac1a9e7d23d9d (#141870) · d4707d12

Zachary Anderson authored Jan 19, 2024

Includes the Engine roll from
https://github.com/flutter/flutter/pull/141841

A new version of Dart is having trouble with the tool integration test
test `passing one file with errors are detected`:
https://ci.chromium.org/ui/p/flutter/builders/try/Mac%20tool_integration_tests_2_4/31851/overview.

However the analysis server emits the expected errors when we give it
both the file without issues and the file with issues.

My guess is that the analysis server has changed it's behavior slightly
when supplied with a single malformed file.

Since the Dart roll is >20 dev versions behind, and this is the only
failing presubmit test, and it's testing something a bit weird, I
suggest we investigate the right way to test the thing that test was
attempting to cover as a follow-up.

d4707d12

Roll Packages from 83c2c4da97c5 to 129e08c1dff1 (13 revisions) (#141865) · 0487931a

engine-flutter-autoroll authored Jan 19, 2024

https://github.com/flutter/packages/compare/83c2c4da97c5...129e08c1dff1

2024-01-19 magder@google.com [pigeon] Run swift-format on ungenerated example app Swift files (flutter/packages#5934)
2024-01-19 magder@google.com [path_provider_foundation] Run swift-format on Swift files (flutter/packages#5935)
2024-01-19 magder@google.com [shared_preferences_foundation] Run swift-format on Swift files (flutter/packages#5933)
2024-01-19 magder@google.com [various] Run swift-format on example app Swift files (flutter/packages#5931)
2024-01-19 magder@google.com [camera_avfoundation] Remove development team from Xcode example app (flutter/packages#5930)
2024-01-18 jsharp83@gmail.com [webview_flutter_wkwebview] Add javascript panel interface for wkwebview (flutter/packages#5795)
2024-01-18 amirpanahandeh@yahoo.com [two_dimensional_scrollables] Fix must_call_super (flutter/packages#5921)
2024-01-18 cedvandenbosch@gmail.com [pointer_interceptor] fix width and height unset warning on web platform (flutter/packages#5864)
2024-01-18 49699333+dependabot[bot]@users.noreply.github.com Bump github/codeql-action from 3.23.0 to 3.23.1 (flutter/packages#5922)
2024-01-18 stuartmorgan@google.com [google_maps_flutter] Clean up iOS example project (flutter/packages#5925)
2024-01-18 magder@google.com [ci] Add flags to formatter command to decide which formatters to run (flutter/packages#5905)
2024-01-18 15619084+vashworth@users.noreply.github.com Update tests to Xcode 15 and iOS 17 simulator (flutter/packages#5914)
2024-01-18 engine-flutter-autoroll@skia.org Roll Flutter from def6af0b to f77f8243 (25 revisions) (flutter/packages#5924)

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

0487931a

Add mac_x64_ios configuration. (#141828) · c95bb6fb

godofredoc authored Jan 19, 2024

This is in preparation to allow tests to run on x64 and arm64 if they rely only on the ios phone.

c95bb6fb

Roll Flutter Engine from 90be25d8aac3 to d1afda52d254 (1 revision) (#141825) · 70a4a5de

engine-flutter-autoroll authored Jan 19, 2024

https://github.com/flutter/engine/compare/90be25d8aac3...d1afda52d254

2024-01-19 skia-flutter-autoroll@skia.org Roll Skia from c71519e854d1 to dd24bd11d79e (1 revision) (flutter/engine#49878)

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

70a4a5de

Move the requestKeyboard up to the widgets layer (#141655) · 8ff0af0c

Ian Hickson authored Jan 19, 2024

Turns out all implementations of this method made this call, so it seems like it should belong in the superclass.

8ff0af0c

Roll Flutter Engine from dde3ebf6551a to 90be25d8aac3 (1 revision) (#141817) · 46bbf402

engine-flutter-autoroll authored Jan 19, 2024

https://github.com/flutter/engine/compare/dde3ebf6551a...90be25d8aac3

2024-01-18 skia-flutter-autoroll@skia.org Roll Skia from da3bfb25fb84 to c71519e854d1 (1 revision) (flutter/engine#49875)

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

46bbf402

18 Jan, 2024 6 commits

enable more tests in web mode (#141791) · 5987563e

Yegor authored Jan 18, 2024

- Unskip `text_style_test` for CanvasKit.
- Remove no longer necessary `kIsWeb` checks in a few tests.

This PR depends on https://github.com/flutter/engine/pull/49786, which rolled into the framework. If the engine PR needs to be reverted, this PR will need to be reverted too.

5987563e

Roll Flutter Engine from 9dded186bcff to dde3ebf6551a (2 revisions) (#141811) · cdf82b28

engine-flutter-autoroll authored Jan 18, 2024

https://github.com/flutter/engine/compare/9dded186bcff...dde3ebf6551a

2024-01-18 737941+loic-sharma@users.noreply.github.com [Windows] Refactor logic when window resize completes (flutter/engine#49872)
2024-01-18 skia-flutter-autoroll@skia.org Roll Skia from 40200ceca00e to da3bfb25fb84 (1 revision) (flutter/engine#49873)

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

cdf82b28

Update margin between label and icon in Tab to better reflect Material specs (#140698) · 197cd4d6

David Martos authored Jan 18, 2024

This PR improves the distance between the label and the icon in the Tab widget. 
I updated the margin to 2 pixels, taken from the Figma design page for Material 3. On Material 2 I left the default value of 10 pixels.

Related to #128696 (In particular, the distance between label and icon)

Here are some screenshots for comparison. I looked a bit into the other mentioned issue of the tab height not following the M3 spec. Flutter uses 72 and the spec uses 64. But because Tab is a PreferredSizeWidget, I don't think there is an easy way to provide a different size depending on `ThemeData.useMaterial3`, because there is no `BuildContext` available.
I provide a sample image for the 64 height as well for context on the linked issue, even though it's not part of the PR changes.

The screenshots are taken side by side with the image at: https://m3.material.io/components/tabs/guidelines

## Original

![original](https://github.com/flutter/flutter/assets/22084723/f52d46bb-eaf9-4519-976e-9ea07c021e14)

## New (tab height = 72, Flutter default for 8 years)

![new_72](https://github.com/flutter/flutter/assets/22084723/8c9d3510-eaca-4b7d-92d8-0d06a7e75136)

## New (tab height = 64, M3 spec)

![new_64](https://github.com/flutter/flutter/assets/22084723/f8811b70-766f-4a4f-b069-33673b1e3744)

197cd4d6

Roll Flutter Engine from 3106e08e1219 to 9dded186bcff (2 revisions) (#141807) · c1b01ec1

engine-flutter-autoroll authored Jan 18, 2024

https://github.com/flutter/engine/compare/3106e08e1219...9dded186bcff

2024-01-18 fmil@google.com [fuchsia] Wire through the tzdata-icu capability (flutter/engine#48907)
2024-01-18 skia-flutter-autoroll@skia.org Roll Skia from 31fae9f0cdd1 to 40200ceca00e (1 revision) (flutter/engine#49871)

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

c1b01ec1

Reverts "Enable native compilation for windows-arm64 " (#141809) · 1901d6fa

auto-submit[bot] authored Jan 18, 2024

Reverts flutter/flutter#137618
Initiated by: Jasguerrero
This change reverts the following previous change:
Original Description:
It's now possible to natively compile a flutter app for
windows-arm64. Cross-compilation is not yet implemented.

Uses arm64 artifacts now available for Dart/Flutter.
Platform detection is based on Abi class, provided by Dart. Depending if
Dart is an arm64 or x64 binary, the Abi is set accordingly.
Initial bootstrap of dart artifacts (update_dart_sdk.ps1) is checking
PROCESSOR_ARCHITECTURE environment variable, which is the way to detect
host architecture on Windows.

This is available only for master channel (on other channels, it
fallbacks to windows-x64).

On windows-x64, it produces an x64 app. On windows-arm64, it produces an
arm64 app.

1901d6fa

Run framework_tests_misc in arm64 and x64. (#141797) · 9d9bd884
godofredoc authored Jan 18, 2024
```
This is to ensure coverage in both platforms.
```
9d9bd884