-
Notifications
You must be signed in to change notification settings - Fork 28.7k
Comparing changes
Open a pull request
base repository: flutter/flutter
base: 077d644bbc5f
head repository: flutter/flutter
compare: ab573048e74d
- 18 commits
- 72 files changed
- 12 contributors
Commits on May 19, 2023
-
[iOS] Dispose of log readers and port forwarders if launch fails (#12…
…7140) When tests run in our CI using `flutter drive`, if there is a failure it will loop and try again. https://github.com/flutter/flutter/blob/434b81f1a58b8528299a63ae7c3ded2aa519e3c9/packages/flutter_tools/lib/src/drive/drive_service.dart#L177-L186 However, it's using the same `device` instance for each iteration. So what was happening was when it would fail to launch, it would tell its listeners that it was cancelled. https://github.com/flutter/flutter/blob/434b81f1a58b8528299a63ae7c3ded2aa519e3c9/packages/flutter_tools/lib/src/ios/ios_deploy.dart#L486-L489 Then when the next iteration started, the `vmServiceDiscovery` would immediately return with null because the `deviceLogReader` would be cached from the previous iteration and would already be cancelled. Therefore, bypassing and cancelling the timer. https://github.com/flutter/flutter/blob/434b81f1a58b8528299a63ae7c3ded2aa519e3c9/packages/flutter_tools/lib/src/ios/devices.dart#L585-L591 https://github.com/flutter/flutter/blob/434b81f1a58b8528299a63ae7c3ded2aa519e3c9/packages/flutter_tools/lib/src/ios/devices.dart#L627 In addition, it seems like sometimes the stop would fail and therefore the the drain would never get the signal that it was done and therefore would hang forever. There was no indication that the stop had failed though because the logs were going to the stream that had no listeners since `deviceLogReader` was already cancelled. https://github.com/flutter/flutter/blob/434b81f1a58b8528299a63ae7c3ded2aa519e3c9/packages/flutter_tools/lib/src/ios/ios_deploy.dart#L563-L576 Fixes #127141
Configuration menu - View commit details
-
Copy full SHA for b9e8b0a - Browse repository at this point
Copy the full SHA b9e8b0aView commit details -
[tool] delete xcresult bundle file before each xcode retry. (#127144)
xcodebuild command generates a xcresult bundle file on each run, however, it doesn't delete the file generated from previous run and will throw an error if the exact file already exists. In tool, we manually delete the file after each `flutter build` or `flutter run` command. However, there are some internal logic where xcodebuild retries multiple times. This PR deletes the xcresult bundle file at the start of each retry if it exists. Fixes #127119
Chris Yang authoredMay 19, 2023 Configuration menu - View commit details
-
Copy full SHA for 972b447 - Browse repository at this point
Copy the full SHA 972b447View commit details -
Revert "Migrate benchmarks to package:web" (#127207)
Reverts #126848 Triggered some kind of measuring discrepancy / performance regression.
Configuration menu - View commit details
-
Copy full SHA for 3001d6a - Browse repository at this point
Copy the full SHA 3001d6aView commit details -
Roll Flutter Engine from a0ea4d2d9ea5 to f471b37a2146 (1 revision) (#…
…127212) flutter/engine@a0ea4d2...f471b37 2023-05-19 [email protected] [floating_cursor_selection]add some comments on the tricky part (flutter/engine#42136) 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 [email protected],[email protected] 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://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
Configuration menu - View commit details
-
Copy full SHA for 4f6f188 - Browse repository at this point
Copy the full SHA 4f6f188View commit details -
Roll Packages from b31a128 to 1e214d7 (3 revisions) (#127217)
flutter/packages@b31a128...1e214d7 2023-05-18 [email protected] Roll Flutter from d0d1feb to 5ae6438 (42 revisions) (flutter/packages#4038) 2023-05-18 [email protected] [ci] Lower iOS LUCI timeouts (flutter/packages#4035) 2023-05-18 [email protected] [ci] Increase Android sharding (flutter/packages#4029) 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 [email protected],[email protected] 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://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
Configuration menu - View commit details
-
Copy full SHA for c7b898b - Browse repository at this point
Copy the full SHA c7b898bView commit details -
Roll goldctl to f808dcff91b221ae313e540c09d79696cd08b8de (#127218)
goldctl hasn't been updated in about a year, and an update may be needed to reland #127110.
Configuration menu - View commit details
-
Copy full SHA for 31a16f3 - Browse repository at this point
Copy the full SHA 31a16f3View commit details -
Configuration menu - View commit details
-
Copy full SHA for e310475 - Browse repository at this point
Copy the full SHA e310475View commit details -
Roll Flutter Engine from f471b37a2146 to 2b14f8a1f21c (1 revision) (#…
…127221) flutter/engine@f471b37...2b14f8a 2023-05-19 [email protected] [Impeller] use host image upload path on simulator (flutter/engine#42161) 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 [email protected],[email protected] 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://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
Configuration menu - View commit details
-
Copy full SHA for d211dc1 - Browse repository at this point
Copy the full SHA d211dc1View commit details -
Remove deprecated OverscrollIndicatorNotification.disallowGlow (#127050)
The deprecated OverscrollIndicatorNotification.disallowGlow has expired and is removed in the PR. The replacement is OverscrollIndicatorNotification.disallowIndicator. This deprecation was introduced in #87839 when the StretchingOverscrollIndicator was added. The name change made it clearer since there is now more than one overscroll indicator. This change is supported by dart fix. � Part of #127042
Configuration menu - View commit details
-
Copy full SHA for e8b7889 - Browse repository at this point
Copy the full SHA e8b7889View commit details -
fixes to anticipate next Dart linter release (#127211)
The upcoming linter release notices null literals as unnecessary argument values and flags more `type_literal_in_constant_pattern` cases. See breakages: https://logs.chromium.org/logs/dart/buildbucket/cr-buildbucket/8780744067138629361/+/u/analyze_flutter_flutter/stdout
Configuration menu - View commit details
-
Copy full SHA for d0c0439 - Browse repository at this point
Copy the full SHA d0c0439View commit details -
Roll Flutter Engine from 2b14f8a1f21c to 3267fa29491a (4 revisions) (#…
…127224) flutter/engine@2b14f8a...3267fa2 2023-05-19 [email protected] [Impeller] Limit blur kernel to 1000x1000 pixels (flutter/engine#42154) 2023-05-19 [email protected] Roll Fuchsia Mac SDK from IGeTvUK2wkajmQQ2k... to pwdDQgM88sqLmZczj... (flutter/engine#42163) 2023-05-19 [email protected] Roll Skia from 0a63dbe8cd15 to 7202b405f061 (1 revision) (flutter/engine#42162) 2023-05-19 [email protected] [Impeller] Fix 1d threadgroup dispatch size, Implement drawPoint geometry computation with compute, fallback to specialized CPU. (flutter/engine#42060) Also rolling transitive DEPS: fuchsia/sdk/core/mac-amd64 from IGeTvUK2wkaj to pwdDQgM88sqL 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 [email protected],[email protected] 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://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
Configuration menu - View commit details
-
Copy full SHA for 7bb6044 - Browse repository at this point
Copy the full SHA 7bb6044View commit details -
Roll Flutter Engine from 3267fa29491a to f0c02aee69db (4 revisions) (#…
…127233) flutter/engine@3267fa2...f0c02ae 2023-05-19 [email protected] Roll Fuchsia Linux SDK from eal6a4HeaQon_Y4ml... to TWjmvLCOnYAUgAzvT... (flutter/engine#42168) 2023-05-19 [email protected] [macOS] Extract PlatformView mutator clip updating (flutter/engine#42164) 2023-05-19 [email protected] [web] Fix event offset for transformed widgets (and text input nodes). (flutter/engine#41870) 2023-05-19 [email protected] Roll goldctl to f808dcff91b221ae313e540c09d79696cd08b8de (flutter/engine#42166) Also rolling transitive DEPS: fuchsia/sdk/core/linux-amd64 from eal6a4HeaQon to TWjmvLCOnYAU 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 [email protected],[email protected] 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://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
Configuration menu - View commit details
-
Copy full SHA for 41e4c58 - Browse repository at this point
Copy the full SHA 41e4c58View commit details -
[Windows] Ensure window is shown (#127046)
## Background The Windows runner has a race at startup: 1. **Platform thread**: creates a hidden window 2. **Platform thread**: launches the Flutter engine 3. **UI/Raster threads**: renders the first frame 4. **Platform thread**: Registers a callback to show the window once the next frame has been rendered. Steps 3 and 4 happen in parallel and it is possible for step 3 to complete before step 4 starts. In this scenario, the next frame callback is never called and the window is never shown. As a result the `windows_startup_test`'s test, which [verifies that the "show window" callback is called](https://github.com/flutter/flutter/blob/1f09a8662dad3bb1959b24e9124e05e2b9dbff1d/dev/integration_tests/windows_startup_test/windows/runner/flutter_window.cpp#L60-L64), can flake if the first frame is rendered before the show window callback has been registered. ## Solution This change makes the runner schedule a frame after it registers the next frame callback. If step 3 hasn't completed yet, this no-ops as a frame is already scheduled. If step 3 has already completed, a new frame will be rendered, which will call the next frame callback and show the window. Part of #119415 See this thread for alternatives that were considered: flutter/engine#42061 (comment)
Configuration menu - View commit details
-
Copy full SHA for ce61eda - Browse repository at this point
Copy the full SHA ce61edaView commit details -
Run Mac intel only targets on both intel and arm (#127230)
The slowness issue was resolved with latest xcode: #119750, and all these targets should resume on both platforms as before. This will help mitigate high queue time on intel bots. This PR updates these targets: - Mac plugin_test_ios: #119764 - Mac build_tests: #120620 - Mac plugin_test: #120714 - Mac plugin_test_macos: #122212 - Mac framework_tests_misc: #122618
Configuration menu - View commit details
-
Copy full SHA for 784ebdf - Browse repository at this point
Copy the full SHA 784ebdfView commit details -
Configuration menu - View commit details
-
Copy full SHA for 7b16cbc - Browse repository at this point
Copy the full SHA 7b16cbcView commit details
Commits on May 20, 2023
-
Roll Flutter Engine from f0c02aee69db to 482c99af9c69 (1 revision) (#…
…127240) flutter/engine@f0c02ae...482c99a 2023-05-19 [email protected] Roll Dart SDK from f5716102efe8 to b3e1eeda4918 (1 revision) (flutter/engine#42172) 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 [email protected],[email protected] 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://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
Configuration menu - View commit details
-
Copy full SHA for 0a6e137 - Browse repository at this point
Copy the full SHA 0a6e137View commit details -
Configuration menu - View commit details
-
Copy full SHA for 80a4f9b - Browse repository at this point
Copy the full SHA 80a4f9bView commit details -
Roll Flutter Engine from 482c99af9c69 to aac09195688d (1 revision) (#…
…127241) flutter/engine@482c99a...aac0919 2023-05-19 [email protected] Overrides accessibilityScrollToVisible (flutter/engine#42047) 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 [email protected],[email protected] 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://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
Configuration menu - View commit details
-
Copy full SHA for ab57304 - Browse repository at this point
Copy the full SHA ab57304View commit details
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine:
git diff 077d644bbc5f...ab573048e74d