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: be7db94
Choose a base ref
...
head repository: flutter/engine
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: afa7ce1
Choose a head ref
  • 18 commits
  • 19 files changed
  • 8 contributors

Commits on Jun 24, 2024

  1. Configuration menu
    Copy the full SHA
    666e488 View commit details
    Browse the repository at this point in the history
  2. 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)

    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. Configuration menu
    Copy the full SHA
    7b5b82e View commit details
    Browse the repository at this point in the history
  7. 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. 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. 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. Configuration menu
    Copy the full SHA
    77462b5 View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    f39b5ab View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    4bb4c1c View commit details
    Browse the repository at this point in the history
  6. Configuration menu
    Copy the full SHA
    afa7ce1 View commit details
    Browse the repository at this point in the history
Loading