Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.
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/engine
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: be7db94196fe
Choose a base ref
...
head repository: flutter/engine
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: afa7ce19bca8
Choose a head ref
  • 18 commits
  • 19 files changed
  • 8 contributors

Commits on Jun 24, 2024

  1. Roll Skia from f6b4344d73cc to ea84df425483 (1 revision) (#53531)

    https://skia.googlesource.com/skia.git/+log/f6b4344d73cc..ea84df425483
    
    2024-06-24 kjlubick@google.com Remove shims for GrVk* types
    
    If this roll has caused a breakage, revert this CL and stop the roller
    using the controls here:
    https://autoroll.skia.org/r/skia-flutter-autoroll
    Please CC brianosman@google.com,jsimmons@google.com,rmistry@google.com,scroggo@google.com on the revert to ensure that a human
    is aware of the problem.
    
    To file a bug in Skia: https://bugs.chromium.org/p/skia/issues/entry
    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
    skia-flutter-autoroll authored Jun 24, 2024
    Configuration menu
    Copy the full SHA
    666e488 View commit details
    Browse the repository at this point in the history
  2. Roll Dart SDK from 95470b2cac1f to 5df89347bddf (1 revision) (#53534)

    https://dart.googlesource.com/sdk.git/+log/95470b2cac1f..5df89347bddf
    
    2024-06-24 dart-internal-merge@dart-ci-internal.iam.gserviceaccount.com Version 3.5.0-294.0.dev
    
    If this roll has caused a breakage, revert this CL and stop the roller
    using the controls here:
    https://autoroll.skia.org/r/dart-sdk-flutter-engine
    Please CC dart-vm-team@google.com,jsimmons@google.com on the revert to ensure that a human
    is aware of the problem.
    
    To file a bug in Flutter Engine: 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
    skia-flutter-autoroll authored Jun 24, 2024
    Configuration menu
    Copy the full SHA
    4495907 View commit details
    Browse the repository at this point in the history
  3. [fuchsia] Update Fuchsia API level to 19 (#53494)

    Update Fuchsia API level to 19
    
    b/347290402
    
    Tests: cl/645133515
    
    ## Pre-launch Checklist
    
    - [x] I read the [Contributor Guide] and followed the process outlined
    there for submitting PRs.
    - [ ] I read the [Tree Hygiene] wiki page, which explains my
    responsibilities.
    - [x] I read and followed the [Flutter Style Guide] and the [C++,
    Objective-C, Java style guides].
    - [x] I listed at least one issue that this PR fixes in the description
    above.
    - [x] I added new tests to check the change I am making or feature I am
    adding, or the PR is [test-exempt]. See [testing the engine] for
    instructions on writing and running engine tests.
    - [x] I updated/added relevant documentation (doc comments with `///`).
    - [x] I signed the [CLA].
    - [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
    [C++, Objective-C, Java style guides]:
    https://github.com/flutter/engine/blob/main/CONTRIBUTING.md#style
    [testing the engine]:
    https://github.com/flutter/flutter/wiki/Testing-the-engine
    [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
    jrwang authored Jun 24, 2024
    Configuration menu
    Copy the full SHA
    226e02c View commit details
    Browse the repository at this point in the history
  4. [Impeller] added a fallback that will make sure the blur fragment sha…

    …der doesn't overflow (#53466)
    
    issue: flutter/flutter#150462
    
    This makes that usage never crash.  I would like to reevaluate for a better fix later though.  Something like:
    1) Rework the downsample logic to make sure this gap doesn't exist.
    1) Simply increase the kernel size.  We already did that before and technically we should be clamped to the 500 sigma so we won't have to do it forever.
    
    [C++, Objective-C, Java style guides]: https://github.com/flutter/engine/blob/main/CONTRIBUTING.md#style
    gaaclarke authored Jun 24, 2024
    Configuration menu
    Copy the full SHA
    2badafd View commit details
    Browse the repository at this point in the history
  5. Roll Skia from ea84df425483 to 0fa58b6ddba0 (3 revisions) (#53535)

    https://skia.googlesource.com/skia.git/+log/ea84df425483..0fa58b6ddba0
    
    2024-06-24 michaelludwig@google.com [graphite] Add validation for PaintParamsKeyBuilder::addData
    2024-06-24 kjlubick@google.com Build skslc and sksl-minify from the modular Bazel build.
    2024-06-24 skia-autoroll@skia-public.iam.gserviceaccount.com Roll vulkan-deps from 3a8b46766c4e to ff63e67e4739 (1 revision)
    
    If this roll has caused a breakage, revert this CL and stop the roller
    using the controls here:
    https://autoroll.skia.org/r/skia-flutter-autoroll
    Please CC brianosman@google.com,jsimmons@google.com,rmistry@google.com,scroggo@google.com on the revert to ensure that a human
    is aware of the problem.
    
    To file a bug in Skia: https://bugs.chromium.org/p/skia/issues/entry
    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
    skia-flutter-autoroll authored Jun 24, 2024
    Configuration menu
    Copy the full SHA
    d775cd3 View commit details
    Browse the repository at this point in the history
  6. Roll Skia from 0fa58b6ddba0 to 1948fd53e280 (2 revisions) (#53536)

    https://skia.googlesource.com/skia.git/+log/0fa58b6ddba0..1948fd53e280
    
    2024-06-24 brianosman@google.com Remove asserts about "valid" premultiplied colors in SkPackARGB32
    2024-06-24 skia-autoroll@skia-public.iam.gserviceaccount.com Roll ANGLE from e53b3ad72322 to 3c472b45024a (9 revisions)
    
    If this roll has caused a breakage, revert this CL and stop the roller
    using the controls here:
    https://autoroll.skia.org/r/skia-flutter-autoroll
    Please CC brianosman@google.com,jsimmons@google.com,rmistry@google.com,scroggo@google.com on the revert to ensure that a human
    is aware of the problem.
    
    To file a bug in Skia: https://bugs.chromium.org/p/skia/issues/entry
    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
    skia-flutter-autoroll authored Jun 24, 2024
    Configuration menu
    Copy the full SHA
    7b5b82e View commit details
    Browse the repository at this point in the history
  7. Roll Skia from 1948fd53e280 to e20c8b0bac0c (1 revision) (#53540)

    https://skia.googlesource.com/skia.git/+log/1948fd53e280..e20c8b0bac0c
    
    2024-06-24 mike@reedtribe.org API to return a scaled version of an image
    
    If this roll has caused a breakage, revert this CL and stop the roller
    using the controls here:
    https://autoroll.skia.org/r/skia-flutter-autoroll
    Please CC brianosman@google.com,jsimmons@google.com,rmistry@google.com,scroggo@google.com on the revert to ensure that a human
    is aware of the problem.
    
    To file a bug in Skia: https://bugs.chromium.org/p/skia/issues/entry
    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
    skia-flutter-autoroll authored Jun 24, 2024
    Configuration menu
    Copy the full SHA
    daa525e View commit details
    Browse the repository at this point in the history
  8. Re-re-land "Upgrade all[most] androidx dependencies to latest" (#53532)

    Re-re-land #53001
    
    (Reland attempt was at #53462)
    
    Unblocked by flutter/flutter#150585, but will still need a manual roll because I will need to re-generate a bunch of lockfiles.
    
    Also, https://github.com/flutter/flutter/blob/master/docs/engine/Testing-presubmit-Engine-PRs-with-the-Flutter-framework.md doesn't work (it's blocked on flutter/flutter#149780) so I mostly just have to pray that no new issues occur in the roll. But I believe all issues that came up in the last attempt should be addressed by the above pr and by manual lockfile generation.
    
    [C++, Objective-C, Java style guides]: https://github.com/flutter/engine/blob/main/CONTRIBUTING.md#style
    gmackall authored Jun 24, 2024
    Configuration menu
    Copy the full SHA
    6884e83 View commit details
    Browse the repository at this point in the history
  9. [macOS] Generate universal gen_snapshots (#53524)

    This reverts commit ac4c31a (#52913).
    This relands commit 4e33c10 (#52885).
    
    Previously, the `gen_snapshot_arm64` and `gen_snapshot_x64` binaries used by the tool were all built for x64 architecture. As such, developers building apps with Flutter rely on Rosetta translation with every build.
    
    This refactors the gen_snapshot build rules on macOS hosts to consistently produce `gen_snapshot_arm64` and `gen_snapshot_x64` binaries with the target architecture of the build but with as universal binaries with both host architectures.
    
    ## arm64 host build
    
    Prior to this patch we emitted:
    * gen_snapshot_arm64 (arch: x64, target_arch: simarm64)
    
    After this patch, we emit:
    * artifacts_x64/gen_snapshot_arm64 (arch: x64, target_arch: simarm64)
    * artifacts_arm64/gen_snapshot_arm64 (arch: arm64, target_arch: arm64)
    * gen_snapshot_arm64 (universal binary composed of both of the above)
    
    ## x64 host build
    
    Prior to this patch we emitted:
    * gen_snapshot_x64 (arch: x64, target_arch: x64)
    
    After this patch, we emit:
    * artifacts_x64/gen_snapshot_x64 (arch: x64, target_arch: x64)
    * artifacts_arm64/gen_snapshot_x64 (arch: arm64, target_arch: simx64)
    * gen_snapshot_x64 (universal binary composed of both of the above)
    
    Note that host builds on macOS currently default to a host architecture of x64 (can be overridden via `--force-mac-arm64`) regardless of host architecture and thus, the build itself relies on Rosetta translation when invoked on Apple Silicon arm64 hardware. This is to ensure a consistent build in CI regardless of bot architecture. See: https://github.com/flutter/engine/blob/6fa734d686888a39add026a2a98d6ec311c23efb/tools/gn#L502-L505
    
    Issue: flutter/flutter#101138
    Issue: flutter/flutter#69157
    
    Related issue: flutter/flutter#103386
    
    [C++, Objective-C, Java style guides]: https://github.com/flutter/engine/blob/main/CONTRIBUTING.md#style
    cbracken authored Jun 24, 2024
    Configuration menu
    Copy the full SHA
    552ca5b View commit details
    Browse the repository at this point in the history
  10. Roll Dart SDK from 5df89347bddf to c187d4b3ec88 (1 revision) (#53542)

    https://dart.googlesource.com/sdk.git/+log/5df89347bddf..c187d4b3ec88
    
    2024-06-24 dart-internal-merge@dart-ci-internal.iam.gserviceaccount.com Version 3.5.0-295.0.dev
    
    If this roll has caused a breakage, revert this CL and stop the roller
    using the controls here:
    https://autoroll.skia.org/r/dart-sdk-flutter-engine
    Please CC dart-vm-team@google.com,jsimmons@google.com on the revert to ensure that a human
    is aware of the problem.
    
    To file a bug in Flutter Engine: 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
    skia-flutter-autoroll authored Jun 24, 2024
    Configuration menu
    Copy the full SHA
    1fb20a2 View commit details
    Browse the repository at this point in the history
  11. [web] Reland "Fix focus management for text fields (#51009)" (#53537)

    Relands: [**Fix focus management for text fields** (#51009)](#51009) by:
    
    * Reverting commit: cf3ac2d (#53502).
    * Keeping the new `ViewFocusBinding` disabled, as [suggested](#51009 (comment)).
    
    [C++, Objective-C, Java style guides]: https://github.com/flutter/engine/blob/main/CONTRIBUTING.md#style
    ditman authored Jun 24, 2024
    Configuration menu
    Copy the full SHA
    63c2403 View commit details
    Browse the repository at this point in the history
  12. Roll Skia from e20c8b0bac0c to 5feca3095719 (1 revision) (#53544)

    https://skia.googlesource.com/skia.git/+log/e20c8b0bac0c..5feca3095719
    
    2024-06-24 zhusida@bytedance.com Add 16bit BT2020, YCGCO color space support
    
    If this roll has caused a breakage, revert this CL and stop the roller
    using the controls here:
    https://autoroll.skia.org/r/skia-flutter-autoroll
    Please CC brianosman@google.com,jsimmons@google.com,rmistry@google.com,scroggo@google.com on the revert to ensure that a human
    is aware of the problem.
    
    To file a bug in Skia: https://bugs.chromium.org/p/skia/issues/entry
    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
    skia-flutter-autoroll authored Jun 24, 2024
    Configuration menu
    Copy the full SHA
    40a81fc View commit details
    Browse the repository at this point in the history

Commits on Jun 25, 2024

  1. Bump impeller-cmake-example (#53538)

    - impeller-cmake: bdero/impeller-cmake#25
    - impeller-cmake-example: bdero/impeller-cmake-example#23
    bdero authored Jun 25, 2024
    Configuration menu
    Copy the full SHA
    6991653 View commit details
    Browse the repository at this point in the history
  2. Reverts "Re-re-land "Upgrade all[most] androidx dependencies to lates…

    …t" (#53532)" (#53546)
    
    Reverts: #53532
    Initiated by: gmackall
    Reason for reverting: This breaks apps that use plugins that use compileSdk 31, some of which we use in our postsubmit (so it blocks the tree).
    Original PR Author: gmackall
    
    Reviewed By: {reidbaker, matanlurey}
    
    This change reverts the following previous change:
    Re-re-land #53001
    
    (Reland attempt was at #53462)
    
    Unblocked by flutter/flutter#150585, but will still need a manual roll because I will need to re-generate a bunch of lockfiles.
    
    Also, https://github.com/flutter/flutter/blob/master/docs/engine/Testing-presubmit-Engine-PRs-with-the-Flutter-framework.md doesn't work (it's blocked on flutter/flutter#149780) so I mostly just have to pray that no new issues occur in the roll. But I believe all issues that came up in the last attempt should be addressed by the above pr and by manual lockfile generation.
    
    [C++, Objective-C, Java style guides]: https://github.com/flutter/engine/blob/main/CONTRIBUTING.md#style
    auto-submit[bot] authored Jun 25, 2024
    Configuration menu
    Copy the full SHA
    a07ba90 View commit details
    Browse the repository at this point in the history
  3. Roll Fuchsia Linux SDK from _6HNhJ6G59VMceKoN... to WUN7NQK04NjF9fRmf…

    …... (#53545)
    
    If this roll has caused a breakage, revert this CL and stop the roller
    using the controls here:
    https://autoroll.skia.org/r/fuchsia-linux-sdk-flutter-engine
    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
    skia-flutter-autoroll authored Jun 25, 2024
    Configuration menu
    Copy the full SHA
    77462b5 View commit details
    Browse the repository at this point in the history
  4. Roll Dart SDK from c187d4b3ec88 to bb18127b2a8e (1 revision) (#53547)

    https://dart.googlesource.com/sdk.git/+log/c187d4b3ec88..bb18127b2a8e
    
    2024-06-25 dart-internal-merge@dart-ci-internal.iam.gserviceaccount.com Version 3.5.0-296.0.dev
    
    If this roll has caused a breakage, revert this CL and stop the roller
    using the controls here:
    https://autoroll.skia.org/r/dart-sdk-flutter-engine
    Please CC dart-vm-team@google.com,jsimmons@google.com on the revert to ensure that a human
    is aware of the problem.
    
    To file a bug in Flutter Engine: 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
    skia-flutter-autoroll authored Jun 25, 2024
    Configuration menu
    Copy the full SHA
    f39b5ab View commit details
    Browse the repository at this point in the history
  5. Roll Skia from 5feca3095719 to 335200e57c26 (1 revision) (#53549)

    https://skia.googlesource.com/skia.git/+log/5feca3095719..335200e57c26
    
    2024-06-25 michaelludwig@google.com [graphite] Fix conic_sk_conical_grad_buf_shader signature
    
    If this roll has caused a breakage, revert this CL and stop the roller
    using the controls here:
    https://autoroll.skia.org/r/skia-flutter-autoroll
    Please CC brianosman@google.com,jsimmons@google.com,rmistry@google.com,scroggo@google.com on the revert to ensure that a human
    is aware of the problem.
    
    To file a bug in Skia: https://bugs.chromium.org/p/skia/issues/entry
    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
    skia-flutter-autoroll authored Jun 25, 2024
    Configuration menu
    Copy the full SHA
    4bb4c1c View commit details
    Browse the repository at this point in the history
  6. Roll Dart SDK from bb18127b2a8e to b5fc85cfcf1b (1 revision) (#53552)

    https://dart.googlesource.com/sdk.git/+log/bb18127b2a8e..b5fc85cfcf1b
    
    2024-06-25 dart-internal-merge@dart-ci-internal.iam.gserviceaccount.com Version 3.5.0-297.0.dev
    
    If this roll has caused a breakage, revert this CL and stop the roller
    using the controls here:
    https://autoroll.skia.org/r/dart-sdk-flutter-engine
    Please CC dart-vm-team@google.com,jsimmons@google.com on the revert to ensure that a human
    is aware of the problem.
    
    To file a bug in Flutter Engine: 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
    skia-flutter-autoroll authored Jun 25, 2024
    Configuration menu
    Copy the full SHA
    afa7ce1 View commit details
    Browse the repository at this point in the history
Loading