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

Commits on Jan 23, 2025

  1. Reverts "[Impeller] Migrate unit tests off of Skia geometry classes (#…

    …161855)" (#162046)
    
    <!-- start_original_pr_link -->
    Reverts: #161855
    <!-- end_original_pr_link -->
    <!-- start_initiating_author -->
    Initiated by: harryterkelsen
    <!-- end_initiating_author -->
    <!-- start_revert_reason -->
    Reason for reverting: causing test failures on `linux_unopt` shard
    
    
    https://ci.chromium.org/ui/p/flutter/builders/prod/Linux%20Production%20Engine%20Drone/1044163/infra
    <!-- end_revert_reason -->
    <!-- start_original_pr_author -->
    Original PR Author: flar
    <!-- end_original_pr_author -->
    
    <!-- start_reviewers -->
    Reviewed By: {jonahwilliams}
    <!-- end_reviewers -->
    
    <!-- start_revert_body -->
    This change reverts the following previous change:
    Handles the impeller unittests line in
    #161456
    
    This PR is focused on the unit tests in Impeller. There are still some
    uses in other non-test areas which will be handled in a separate PR.
    <!-- end_revert_body -->
    
    Co-authored-by: auto-submit[bot] <[email protected]>
    auto-submit[bot] and auto-submit[bot] authored Jan 23, 2025
    Configuration menu
    Copy the full SHA
    c75d799 View commit details
    Browse the repository at this point in the history
  2. Add a README with instructions for editing and running tests for the …

    …FGP (#161830)
    
    Add a readme with some info.
    
    ## Pre-launch Checklist
    
    - [x] I read the [Contributor Guide] and followed the process outlined
    there for submitting PRs.
    - [x] I read the [Tree Hygiene] wiki page, which explains my
    responsibilities.
    - [x] I read and followed the [Flutter Style Guide], including [Features
    we expect every widget to implement].
    - [x] I signed the [CLA].
    - [ ] I listed at least one issue that this PR fixes in the description
    above.
    - [x] I updated/added relevant documentation (doc comments with `///`).
    - [x] I added new tests to check the change I am making, or this PR is
    [test-exempt].
    - [x] I followed the [breaking change policy] and added [Data Driven
    Fixes] where supported.
    - [ ] 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/blob/main/docs/contributing/Tree-hygiene.md#overview
    [Tree Hygiene]:
    https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md
    [test-exempt]:
    https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#tests
    [Flutter Style Guide]:
    https://github.com/flutter/flutter/blob/main/docs/contributing/Style-guide-for-Flutter-repo.md
    [Features we expect every widget to implement]:
    https://github.com/flutter/flutter/blob/main/docs/contributing/Style-guide-for-Flutter-repo.md#features-we-expect-every-widget-to-implement
    [CLA]: https://cla.developers.google.com/
    [flutter/tests]: https://github.com/flutter/tests
    [breaking change policy]:
    https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#handling-breaking-changes
    [Discord]:
    https://github.com/flutter/flutter/blob/main/docs/contributing/Chat.md
    [Data Driven Fixes]:
    https://github.com/flutter/flutter/blob/main/docs/contributing/Data-driven-Fixes.md
    
    ---------
    
    Co-authored-by: Gray Mackall <[email protected]>
    Co-authored-by: Reid Baker <[email protected]>
    3 people authored Jan 23, 2025
    Configuration menu
    Copy the full SHA
    64c808b View commit details
    Browse the repository at this point in the history
  3. [Impeller] adjust coverage origin when rounding out SaveLayer bounds. (

    …#161838)
    
    Fixes #161374
    
    When we round out the texture size for a saveLayer, also round out the
    coverage so that the drawing origin is correct. Otherwise we may
    actually cut off the top left edge of rendering.
    
    ---------
    
    Co-authored-by: gaaclarke <[email protected]>
    Jonah Williams and gaaclarke authored Jan 23, 2025
    Configuration menu
    Copy the full SHA
    948e31d View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    1083356 View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    9c960ff View commit details
    Browse the repository at this point in the history
  6. Fix DropdownButtonFormField clips text when large text scale is used (#…

    …159975)
    
    ## Description
    
    This PR fixes `DropdownButtonFormField` text being clipped when using a
    large text scale.
    
    Before:
    
    
    ![image](https://github.com/user-attachments/assets/6c4deed2-eb61-4c0a-912e-dba364013038)
    
    After:
    
    
    ![image](https://github.com/user-attachments/assets/1dee5cda-9885-47c1-92a6-afbbc2312266)
    
    
    This extend the fix from #107201
    which does not work properly with Material 3 (because of
    TextStyle.height being set for M3 default text styles).
    
    ## Related Issue
    
    Fixes [DropdownButtonFormField clips text when large text scale is used
    and useMaterial3 is
    true](#159971)
    
    ## Tests
    
    Adds 1 test.
    Updates 1 test.
    bleroux authored Jan 23, 2025
    Configuration menu
    Copy the full SHA
    97ca57c View commit details
    Browse the repository at this point in the history
  7. [native assets] Roll dependencies (#162017)

    This PR rolls in a number of breaking changes for native assets:
    
    * Pub workspaces are now supported, this requires a refactoring to how
    the `NativeAssetsBuildRunner` is used. Most notably it requires being
    explicit about the `runPackageName`. Flutter does not seem to have that
    value, but it does have the `projectUri` which can be used to find the
    package name via the package config.
    * The API for build hooks has been redesigned. This PR updates the
    project template, test project files, and test project strings.
    
    ## Pre-launch Checklist
    
    - [x] I read the [Contributor Guide] and followed the process outlined
    there for submitting PRs.
    - [x] I read the [Tree Hygiene] wiki page, which explains my
    responsibilities.
    - [x] I read and followed the [Flutter Style Guide], including [Features
    we expect every widget to implement].
    - [x] I signed the [CLA].
    - [x] I listed at least one issue that this PR fixes in the description
    above.
    - [x] I updated/added relevant documentation (doc comments with `///`).
    - [ ] I added new tests to check the change I am making, or this PR is
    [test-exempt].
    - [ ] I followed the [breaking change policy] and added [Data Driven
    Fixes] where supported.
    - [x] All existing and new tests are passing.
    
    <!-- Links -->
    [Contributor Guide]:
    https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#overview
    [Tree Hygiene]:
    https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md
    [test-exempt]:
    https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#tests
    [Flutter Style Guide]:
    https://github.com/flutter/flutter/blob/main/docs/contributing/Style-guide-for-Flutter-repo.md
    [Features we expect every widget to implement]:
    https://github.com/flutter/flutter/blob/main/docs/contributing/Style-guide-for-Flutter-repo.md#features-we-expect-every-widget-to-implement
    [CLA]: https://cla.developers.google.com/
    [flutter/tests]: https://github.com/flutter/tests
    [breaking change policy]:
    https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#handling-breaking-changes
    [Discord]:
    https://github.com/flutter/flutter/blob/main/docs/contributing/Chat.md
    [Data Driven Fixes]:
    https://github.com/flutter/flutter/blob/main/docs/contributing/Data-driven-Fixes.md
    dcharkes authored Jan 23, 2025
    Configuration menu
    Copy the full SHA
    9ded795 View commit details
    Browse the repository at this point in the history
  8. Configuration menu
    Copy the full SHA
    663d020 View commit details
    Browse the repository at this point in the history
  9. Configuration menu
    Copy the full SHA
    6e9b863 View commit details
    Browse the repository at this point in the history
  10. fix: Call codec.dispose in dev/ (#161112)

    <!--
    Thanks for filing a pull request!
    Reviewers are typically assigned within a week of filing a request.
    To learn more about code review, see our documentation on Tree Hygiene:
    https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md
    -->
    
    PR derived from #159945.
    
    Added `codec.dispose()` for directories under `dev/`.
    
    ## Pre-launch Checklist
    
    - [x] I read the [Contributor Guide] and followed the process outlined
    there for submitting PRs.
    - [x] I read the [Tree Hygiene] wiki page, which explains my
    responsibilities.
    - [x] I read and followed the [Flutter Style Guide], including [Features
    we expect every widget to implement].
    - [x] I signed the [CLA].
    - [ ] I listed at least one issue that this PR fixes in the description
    above.
    - [ ] I updated/added relevant documentation (doc comments with `///`).
    - [x] I added new tests to check the change I am making, or this PR is
    [test-exempt].
    - [x] I followed the [breaking change policy] and added [Data Driven
    Fixes] where supported.
    - [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/blob/main/docs/contributing/Tree-hygiene.md#overview
    [Tree Hygiene]:
    https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md
    [test-exempt]:
    https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#tests
    [Flutter Style Guide]:
    https://github.com/flutter/flutter/blob/main/docs/contributing/Style-guide-for-Flutter-repo.md
    [Features we expect every widget to implement]:
    https://github.com/flutter/flutter/blob/main/docs/contributing/Style-guide-for-Flutter-repo.md#features-we-expect-every-widget-to-implement
    [CLA]: https://cla.developers.google.com/
    [flutter/tests]: https://github.com/flutter/tests
    [breaking change policy]:
    https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#handling-breaking-changes
    [Discord]:
    https://github.com/flutter/flutter/blob/main/docs/contributing/Chat.md
    [Data Driven Fixes]:
    https://github.com/flutter/flutter/blob/main/docs/contributing/Data-driven-Fixes.md
    koji-1009 authored Jan 23, 2025
    Configuration menu
    Copy the full SHA
    29ffb88 View commit details
    Browse the repository at this point in the history
  11. fix: Call codec.dispose in flutter_test (#161131)

    PR derived from #159945.
    
    Added `codec.dispose()` for directories under `flutter_test`.
    
    ## Pre-launch Checklist
    
    - [x] I read the [Contributor Guide] and followed the process outlined
    there for submitting PRs.
    - [x] I read the [Tree Hygiene] wiki page, which explains my
    responsibilities.
    - [x] I read and followed the [Flutter Style Guide], including [Features
    we expect every widget to implement].
    - [x] I signed the [CLA].
    - [ ] I listed at least one issue that this PR fixes in the description
    above.
    - [ ] I updated/added relevant documentation (doc comments with `///`).
    - [ ] I added new tests to check the change I am making, or this PR is
    [test-exempt].
    - [x] I followed the [breaking change policy] and added [Data Driven
    Fixes] where supported.
    - [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/blob/main/docs/contributing/Tree-hygiene.md#overview
    [Tree Hygiene]:
    https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md
    [test-exempt]:
    https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#tests
    [Flutter Style Guide]:
    https://github.com/flutter/flutter/blob/main/docs/contributing/Style-guide-for-Flutter-repo.md
    [Features we expect every widget to implement]:
    https://github.com/flutter/flutter/blob/main/docs/contributing/Style-guide-for-Flutter-repo.md#features-we-expect-every-widget-to-implement
    [CLA]: https://cla.developers.google.com/
    [flutter/tests]: https://github.com/flutter/tests
    [breaking change policy]:
    https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#handling-breaking-changes
    [Discord]:
    https://github.com/flutter/flutter/blob/main/docs/contributing/Chat.md
    [Data Driven Fixes]:
    https://github.com/flutter/flutter/blob/main/docs/contributing/Data-driven-Fixes.md
    koji-1009 authored Jan 23, 2025
    Configuration menu
    Copy the full SHA
    11a15cb View commit details
    Browse the repository at this point in the history
  12. fix: Call codec.dispose in flutter/test (#161127)

    PR derived from #159945.
    
    Added `codec.dispose()` for directories under `flutter/test`.
    
    ## Pre-launch Checklist
    
    - [x] I read the [Contributor Guide] and followed the process outlined
    there for submitting PRs.
    - [x] I read the [Tree Hygiene] wiki page, which explains my
    responsibilities.
    - [x] I read and followed the [Flutter Style Guide], including [Features
    we expect every widget to implement].
    - [x] I signed the [CLA].
    - [ ] I listed at least one issue that this PR fixes in the description
    above.
    - [ ] I updated/added relevant documentation (doc comments with `///`).
    - [x] I added new tests to check the change I am making, or this PR is
    [test-exempt].
    - [x] I followed the [breaking change policy] and added [Data Driven
    Fixes] where supported.
    - [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/blob/main/docs/contributing/Tree-hygiene.md#overview
    [Tree Hygiene]:
    https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md
    [test-exempt]:
    https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#tests
    [Flutter Style Guide]:
    https://github.com/flutter/flutter/blob/main/docs/contributing/Style-guide-for-Flutter-repo.md
    [Features we expect every widget to implement]:
    https://github.com/flutter/flutter/blob/main/docs/contributing/Style-guide-for-Flutter-repo.md#features-we-expect-every-widget-to-implement
    [CLA]: https://cla.developers.google.com/
    [flutter/tests]: https://github.com/flutter/tests
    [breaking change policy]:
    https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#handling-breaking-changes
    [Discord]:
    https://github.com/flutter/flutter/blob/main/docs/contributing/Chat.md
    [Data Driven Fixes]:
    https://github.com/flutter/flutter/blob/main/docs/contributing/Data-driven-Fixes.md
    koji-1009 authored Jan 23, 2025
    Configuration menu
    Copy the full SHA
    d59995b View commit details
    Browse the repository at this point in the history
  13. fix: Call codec.dispose in tests of engine/src/flutter (#161115)

    PR derived from #159945.
    
    Added `codec.dispose()` for directories under `engine/src/flutter`.
    
    ## Pre-launch Checklist
    
    - [x] I read the [Contributor Guide] and followed the process outlined
    there for submitting PRs.
    - [x] I read the [Tree Hygiene] wiki page, which explains my
    responsibilities.
    - [x] I read and followed the [Flutter Style Guide], including [Features
    we expect every widget to implement].
    - [x] I signed the [CLA].
    - [ ] I listed at least one issue that this PR fixes in the description
    above.
    - [ ] I updated/added relevant documentation (doc comments with `///`).
    - [x] I added new tests to check the change I am making, or this PR is
    [test-exempt].
    - [x] I followed the [breaking change policy] and added [Data Driven
    Fixes] where supported.
    - [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/blob/main/docs/contributing/Tree-hygiene.md#overview
    [Tree Hygiene]:
    https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md
    [test-exempt]:
    https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#tests
    [Flutter Style Guide]:
    https://github.com/flutter/flutter/blob/main/docs/contributing/Style-guide-for-Flutter-repo.md
    [Features we expect every widget to implement]:
    https://github.com/flutter/flutter/blob/main/docs/contributing/Style-guide-for-Flutter-repo.md#features-we-expect-every-widget-to-implement
    [CLA]: https://cla.developers.google.com/
    [flutter/tests]: https://github.com/flutter/tests
    [breaking change policy]:
    https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#handling-breaking-changes
    [Discord]:
    https://github.com/flutter/flutter/blob/main/docs/contributing/Chat.md
    [Data Driven Fixes]:
    https://github.com/flutter/flutter/blob/main/docs/contributing/Data-driven-Fixes.md
    koji-1009 authored Jan 23, 2025
    Configuration menu
    Copy the full SHA
    2919003 View commit details
    Browse the repository at this point in the history
  14. Keyboard tidy ups (#162054)

    Make async calls use standard GLib async API.
    Remove test specific methods.
    Smaller cleanups.
    robert-ancell authored Jan 23, 2025
    Configuration menu
    Copy the full SHA
    b56b5b6 View commit details
    Browse the repository at this point in the history
  15. [Impeller] Implement inherited opacity for ColorFilterContents (#161834)

    Inherited opacity should be combined with the alpha of the
    ColorFilterContents when drawing objects with filters inside an opacity
    layer.
    jason-simmons authored Jan 23, 2025
    Configuration menu
    Copy the full SHA
    6a8d77d View commit details
    Browse the repository at this point in the history
  16. [Impeller] check both linear sampling props for AHBs. (#162043)

    According to
    KhronosGroup/Vulkan-ValidationLayers#5806 ,
    both eSampledImageYcbcrConversionLinearFilter and
    eSampledImageFilterLinear properties are required to use a linear
    chromaFilter.
    Jonah Williams authored Jan 23, 2025
    Configuration menu
    Copy the full SHA
    b46479c View commit details
    Browse the repository at this point in the history
  17. Shift mac_clang_tidy to a Linux orchestrator. (#162042)

    Towards #160701.
    
    This adds a `bringup: true` node that duplicates what `Mac
    mac_clang_tidy` is doing, to be turned down/up in a follow-up.
    matanlurey authored Jan 23, 2025
    Configuration menu
    Copy the full SHA
    211b46f View commit details
    Browse the repository at this point in the history
  18. Add windows_host_engine_test to presubmit, remove lint exceptions. (#…

    …162032)
    
    Closes #161990.
    Closes #161406.
    
    I don't have a Windows machine present, so will let CI tell me if it's
    good to go.
    matanlurey authored Jan 23, 2025
    Configuration menu
    Copy the full SHA
    c1561a4 View commit details
    Browse the repository at this point in the history
Loading