Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: flutter/flutter
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: 077d644bbc5f
Choose a base ref
...
head repository: flutter/flutter
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: ab573048e74d
Choose a head ref
  • 18 commits
  • 72 files changed
  • 12 contributors

Commits on May 19, 2023

  1. [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
    vashworth authored May 19, 2023
    Configuration menu
    Copy the full SHA
    b9e8b0a View commit details
    Browse the repository at this point in the history
  2. [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 authored May 19, 2023
    Configuration menu
    Copy the full SHA
    972b447 View commit details
    Browse the repository at this point in the history
  3. Revert "Migrate benchmarks to package:web" (#127207)

    Reverts #126848
    
    Triggered some kind of measuring discrepancy / performance regression.
    joshualitt authored May 19, 2023
    Configuration menu
    Copy the full SHA
    3001d6a View commit details
    Browse the repository at this point in the history
  4. 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
    engine-flutter-autoroll authored May 19, 2023
    Configuration menu
    Copy the full SHA
    4f6f188 View commit details
    Browse the repository at this point in the history
  5. 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
    engine-flutter-autoroll authored May 19, 2023
    Configuration menu
    Copy the full SHA
    c7b898b View commit details
    Browse the repository at this point in the history
  6. Roll goldctl to f808dcff91b221ae313e540c09d79696cd08b8de (#127218)

    goldctl hasn't been updated in about a year, and an update may be needed to reland #127110.
    zanderso authored May 19, 2023
    Configuration menu
    Copy the full SHA
    31a16f3 View commit details
    Browse the repository at this point in the history
  7. [flutter_tools] only try to take a screenshot from flutter drive if t…

    …he --screenshot flag is passed (#127150)
    
    Fixes #123621
    christopherfujino authored May 19, 2023
    Configuration menu
    Copy the full SHA
    e310475 View commit details
    Browse the repository at this point in the history
  8. 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
    engine-flutter-autoroll authored May 19, 2023
    Configuration menu
    Copy the full SHA
    d211dc1 View commit details
    Browse the repository at this point in the history
  9. 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
    Piinks authored May 19, 2023
    Configuration menu
    Copy the full SHA
    e8b7889 View commit details
    Browse the repository at this point in the history
  10. 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
    pq authored May 19, 2023
    Configuration menu
    Copy the full SHA
    d0c0439 View commit details
    Browse the repository at this point in the history
  11. 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
    engine-flutter-autoroll authored May 19, 2023
    Configuration menu
    Copy the full SHA
    7bb6044 View commit details
    Browse the repository at this point in the history
  12. 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
    engine-flutter-autoroll authored May 19, 2023
    Configuration menu
    Copy the full SHA
    41e4c58 View commit details
    Browse the repository at this point in the history
  13. [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)
    loic-sharma authored May 19, 2023
    Configuration menu
    Copy the full SHA
    ce61eda View commit details
    Browse the repository at this point in the history
  14. 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
    keyonghan authored May 19, 2023
    Configuration menu
    Copy the full SHA
    784ebdf View commit details
    Browse the repository at this point in the history
  15. Configuration menu
    Copy the full SHA
    7b16cbc View commit details
    Browse the repository at this point in the history

Commits on May 20, 2023

  1. 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
    engine-flutter-autoroll authored May 20, 2023
    Configuration menu
    Copy the full SHA
    0a6e137 View commit details
    Browse the repository at this point in the history
  2. Reland "[tool] Move Java functions to their own file" (#126577)

    Relands #126086, which was reverted by #126569.
    andrewkolos authored May 20, 2023
    Configuration menu
    Copy the full SHA
    80a4f9b View commit details
    Browse the repository at this point in the history
  3. 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
    engine-flutter-autoroll authored May 20, 2023
    Configuration menu
    Copy the full SHA
    ab57304 View commit details
    Browse the repository at this point in the history
Loading