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: 01db23b82233
Choose a base ref
...
head repository: flutter/flutter
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: 349ec7183cd0
Choose a head ref
  • 19 commits
  • 108 files changed
  • 16 contributors

Commits on Jun 13, 2024

  1. Configuration menu
    Copy the full SHA
    39edf2a View commit details
    Browse the repository at this point in the history
  2. Replace InputDecorator M3 golden test (#150111)

    Golden test replaced with a painting expectation.
    bleroux authored Jun 13, 2024
    Configuration menu
    Copy the full SHA
    651f25f View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    051b6b1 View commit details
    Browse the repository at this point in the history
  4. Update framework and flutter fix flutter.dev/docs links (#150174)

    Update links in the framework and fix data files to their newest, up-to-date destinations.
    parlough authored Jun 13, 2024
    Configuration menu
    Copy the full SHA
    15af844 View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    f56e0c2 View commit details
    Browse the repository at this point in the history
  6. Suppress Flutter update check if --machine is present at all. (#150138

    )
    
    Fixes #145158.
    
    In an ideal world, the `--machine` flag would be strictly a global flag which sub-commands can choose to use (or perhaps just to report a `toolExit` that they don't have a `--machine` supported-mode if not. However currently, there is both a global flag, and command-specific flags.
    
    This leads to the confusing scenario where:
    ```sh
    flutter devices --machine
    ```
    
    ... still checks for a Flutter update, printing a banner and breaking the JSON output.
    
    This PR "fixes" that by allowing `--machine` _anywhere_ in the command-line arguments to suppress the check.
    
    /cc @johnmccutchan.
    matanlurey authored Jun 13, 2024
    Configuration menu
    Copy the full SHA
    0cc27b9 View commit details
    Browse the repository at this point in the history
  7. Configuration menu
    Copy the full SHA
    32df505 View commit details
    Browse the repository at this point in the history
  8. Reland "Identify and re-throw our dependency checking errors in flutt…

    …er.groovy" (#150128)
    
    The original approach in #149609 didn't work when the Flutter Gradle plugin was applied using the deprecated script apply - the kotlin portion couldn't resolve the custom exception defined in `flutter.groovy`:
    ```
    e: /Users/mackall/development/flutter/flutter/packages/flutter_tools/gradle/src/main/kotlin/dependency_version_checker.gradle.kts:238:23: Unresolved reference: DependencyValidationException
    e: /Users/mackall/development/flutter/flutter/packages/flutter_tools/gradle/src/main/kotlin/dependency_version_checker.gradle.kts:263:23: Unresolved reference: DependencyValidationException
    e: /Users/mackall/development/flutter/flutter/packages/flutter_tools/gradle/src/main/kotlin/dependency_version_checker.gradle.kts:288:23: Unresolved reference: DependencyValidationException
    e: /Users/mackall/development/flutter/flutter/packages/flutter_tools/gradle/src/main/kotlin/dependency_version_checker.gradle.kts:313:23: Unresolved reference: DependencyValidationException
    Warning: Flutter was unable to detect project Gradle, Java, AGP, and KGP versions. Skipping dependency version checking. Error was: org.gradle.internal.exceptions.LocationAwareException: Script '/Users/mackall/development/flutter/flutter/packages/flutter_tools/gradle/src/main/kotlin/dependency_version_checker.gradle.kts' line: 238
    Script compilation errors:
    
      Line 238:                 throw DependencyValidationException(errorMessage)
                                      ^ Unresolved reference: DependencyValidationException
    
      Line 263:                 throw DependencyValidationException(errorMessage)
                                      ^ Unresolved reference: DependencyValidationException
    
      Line 288:                 throw DependencyValidationException(errorMessage)
                                      ^ Unresolved reference: DependencyValidationException
    
      Line 313:                 throw DependencyValidationException(errorMessage)
                                      ^ Unresolved reference: DependencyValidationException
    ```
    
    This new approach of setting one of the [`extra` properties](https://docs.gradle.org/current/dsl/org.gradle.api.Project.html#N14FF7) works in both cases (tested with the `camera_android` example app, which uses the script apply, and a freshly created counter app). 
    
    It also removes some brittleness in that we don't have to unwrap the exception anymore, and aren't subject to breaking if Gradle decides one day to wrap our custom exception 1 layer deeper in additional exceptions.
    gmackall authored Jun 13, 2024
    Configuration menu
    Copy the full SHA
    7e97cf7 View commit details
    Browse the repository at this point in the history
  9. Use --(no-)strip-wams instead of --(no-)-name-section in `dart compil…

    …e wasm` (#149641)
    
    The Dart SDK change in [0] introduced the `--(no-)strip-wasm` flag to align with the flag name in `flutter build web`, so we can use that flag now in flutter tools.
    
    [0] https://dart-review.googlesource.com/c/sdk/+/369062
    mkustermann authored Jun 13, 2024
    Configuration menu
    Copy the full SHA
    03b595e View commit details
    Browse the repository at this point in the history
  10. Configuration menu
    Copy the full SHA
    1330675 View commit details
    Browse the repository at this point in the history
  11. Roll pub packages (#150206)

    This PR was generated by `flutter update-packages --force-upgrade`.
    flutter-pub-roller-bot authored Jun 13, 2024
    Configuration menu
    Copy the full SHA
    397420d View commit details
    Browse the repository at this point in the history
  12. Update doc for ColorScheme.surface (#150212)

    This PR just has a tiny doc change to correct the doc for `ColorScheme.surface`. Previously, Card background color defaults to `ColorScheme.surface` but we updated the default since we've introduced the new tone-based surface colors, like `surfaceContainer`, `surfaceContainerLow` and etc. The `surface` in ColorScheme should now replace deprecated `background` which is the default background color of Scaffold.
    
    Related to the comment here: #149772 (comment)
    QuncCccccc authored Jun 13, 2024
    Configuration menu
    Copy the full SHA
    87a32dd View commit details
    Browse the repository at this point in the history
  13. Document CIPD role & login for upgrading Android engine (#149433)

    Uploading CIPD packages requires the `flutter-cipd-writers` role. This
    simply adds this detail to the instructions for updating the Android
    Engine's SDK.
    
    This is a documentation-only change.
    
    ## Pre-launch Checklist
    
    - [ ] 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].
    - [ ] 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 `///`).
    - [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/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
    [Features we expect every widget to implement]:
    https://github.com/flutter/flutter/wiki/Style-guide-for-Flutter-repo#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/wiki/Tree-hygiene#handling-breaking-changes
    [Discord]: https://github.com/flutter/flutter/wiki/Chat
    [Data Driven Fixes]:
    https://github.com/flutter/flutter/wiki/Data-driven-Fixes
    
    ---------
    
    Co-authored-by: Reid Baker <[email protected]>
    yaakovschectman and reidbaker authored Jun 13, 2024
    Configuration menu
    Copy the full SHA
    3bdc451 View commit details
    Browse the repository at this point in the history
  14. Fix leaky test. (#150235)

    polina-c authored Jun 13, 2024
    Configuration menu
    Copy the full SHA
    3226caf View commit details
    Browse the repository at this point in the history

Commits on Jun 14, 2024

  1. Bump github/codeql-action from 3.25.9 to 3.25.10 (#150228)

    Bumps [github/codeql-action](https://github.com/github/codeql-action) from 3.25.9 to 3.25.10.
    <details>
    <summary>Changelog</summary>
    <p><em>Sourced from <a href="https://github.com/github/codeql-action/blob/main/CHANGELOG.md">github/codeql-action's changelog</a>.</em></p>
    <blockquote>
    <h1>CodeQL Action Changelog</h1>
    <p>See the <a href="https://github.com/github/codeql-action/releases">releases page</a> for the relevant changes to the CodeQL CLI and language packs.</p>
    <p>Note that the only difference between <code>v2</code> and <code>v3</code> of the CodeQL Action is the node version they support, with <code>v3</code> running on node 20 while we continue to release <code>v2</code> to support running on node 16. For example <code>3.22.11</code> was the first <code>v3</code> release and is functionally identical to <code>2.22.11</code>. This approach ensures an easy way to track exactly which features are included in different versions, indicated by the minor and patch version numbers.</p>
    <h2>[UNRELEASED]</h2>
    <p>No user facing changes.</p>
    <h2>3.25.10 - 13 Jun 2024</h2>
    <ul>
    <li>Update default CodeQL bundle version to 2.17.5. <a href="https://redirect.github.com/github/codeql-action/pull/2327">#2327</a></li>
    </ul>
    <h2>3.25.9 - 12 Jun 2024</h2>
    <ul>
    <li>Avoid failing database creation if the database folder already exists and contains some unexpected files. Requires CodeQL 2.18.0 or higher. <a href="https://redirect.github.com/github/codeql-action/pull/2330">#2330</a></li>
    <li>The init Action will attempt to clean up the database cluster directory before creating a new database and at the end of the job. This will help to avoid issues where the database cluster directory is left in an inconsistent state. <a href="https://redirect.github.com/github/codeql-action/pull/2332">#2332</a></li>
    </ul>
    <h2>3.25.8 - 04 Jun 2024</h2>
    <ul>
    <li>Update default CodeQL bundle version to 2.17.4. <a href="https://redirect.github.com/github/codeql-action/pull/2321">#2321</a></li>
    </ul>
    <h2>3.25.7 - 31 May 2024</h2>
    <ul>
    <li>We are rolling out a feature in May/June 2024 that will reduce the Actions cache usage of the Action by keeping only the newest TRAP cache for each language. <a href="https://redirect.github.com/github/codeql-action/pull/2306">#2306</a></li>
    </ul>
    <h2>3.25.6 - 20 May 2024</h2>
    <ul>
    <li>Update default CodeQL bundle version to 2.17.3. <a href="https://redirect.github.com/github/codeql-action/pull/2295">#2295</a></li>
    </ul>
    <h2>3.25.5 - 13 May 2024</h2>
    <ul>
    <li>Add a compatibility matrix of supported CodeQL Action, CodeQL CLI, and GitHub Enterprise Server versions to the <a href="https://github.com/github/codeql-action/blob/main/README.md">https://github.com/github/codeql-action/blob/main/README.md</a>. <a href="https://redirect.github.com/github/codeql-action/pull/2273">#2273</a></li>
    <li>Avoid printing out a warning for a missing <code>on.push</code> trigger when the CodeQL Action is triggered via a <code>workflow_call</code> event. <a href="https://redirect.github.com/github/codeql-action/pull/2274">#2274</a></li>
    <li>The <code>tools: latest</code> input to the <code>init</code> Action has been renamed to <code>tools: linked</code>. This option specifies that the Action should use the tools shipped at the same time as the Action. The old name will continue to work for backwards compatibility, but we recommend that new workflows use the new name. <a href="https://redirect.github.com/github/codeql-action/pull/2281">#2281</a></li>
    </ul>
    <h2>3.25.4 - 08 May 2024</h2>
    <ul>
    <li>Update default CodeQL bundle version to 2.17.2. <a href="https://redirect.github.com/github/codeql-action/pull/2270">#2270</a></li>
    </ul>
    <h2>3.25.3 - 25 Apr 2024</h2>
    <ul>
    <li>Update default CodeQL bundle version to 2.17.1. <a href="https://redirect.github.com/github/codeql-action/pull/2247">#2247</a></li>
    <li>Workflows running on <code>macos-latest</code> using CodeQL CLI versions before v2.15.1 will need to either upgrade their CLI version to v2.15.1 or newer, or change the platform to an Intel MacOS runner, such as <code>macos-12</code>. ARM machines with SIP disabled, including the newest <code>macos-latest</code> image, are unsupported for CLI versions before 2.15.1. <a href="https://redirect.github.com/github/codeql-action/pull/2261">#2261</a></li>
    </ul>
    <h2>3.25.2 - 22 Apr 2024</h2>
    <p>No user facing changes.</p>
    </blockquote>
    <p>... (truncated)</p>
    </details>
    <details>
    <summary>Commits</summary>
    <ul>
    <li><a href="https://github.com/github/codeql-action/commit/23acc5c183826b7a8a97bce3cecc52db901f8251"><code>23acc5c</code></a> Merge pull request <a href="https://redirect.github.com/github/codeql-action/issues/2337">#2337</a> from github/update-v3.25.10-5bf6dad35</li>
    <li><a href="https://github.com/github/codeql-action/commit/9b72dbdc68c8ff81c1067a930bd2864ea707129b"><code>9b72dbd</code></a> Update changelog for v3.25.10</li>
    <li><a href="https://github.com/github/codeql-action/commit/5bf6dad35b41961d8779984f426d6a539bbe5d10"><code>5bf6dad</code></a> Merge pull request <a href="https://redirect.github.com/github/codeql-action/issues/2329">#2329</a> from github/henrymercer/csharp-buildless-rollback-me...</li>
    <li><a href="https://github.com/github/codeql-action/commit/feec81c66bfd25a8de4024a727431eb5a4b31317"><code>feec81c</code></a> Merge branch 'main' into henrymercer/csharp-buildless-rollback-mechanism</li>
    <li><a href="https://github.com/github/codeql-action/commit/789b5f86ef49249530b3d018f3ab10640eff231c"><code>789b5f8</code></a> Merge pull request <a href="https://redirect.github.com/github/codeql-action/issues/2328">#2328</a> from github/henrymercer/direct-tracing-fix</li>
    <li><a href="https://github.com/github/codeql-action/commit/c36b5fc54fb5e6fdd2c4942bcfa424f685b214a8"><code>c36b5fc</code></a> Merge pull request <a href="https://redirect.github.com/github/codeql-action/issues/2327">#2327</a> from github/update-bundle/codeql-bundle-v2.17.5</li>
    <li><a href="https://github.com/github/codeql-action/commit/b3642aa04dc0ec830af386ae4c9a130a2f322aab"><code>b3642aa</code></a> Merge branch 'main' into update-bundle/codeql-bundle-v2.17.5</li>
    <li><a href="https://github.com/github/codeql-action/commit/1fc6e20182346ec584a1a9968489d2d4daa96c3f"><code>1fc6e20</code></a> Merge pull request <a href="https://redirect.github.com/github/codeql-action/issues/2335">#2335</a> from github/mergeback/v3.25.9-to-main-530d4fea</li>
    <li><a href="https://github.com/github/codeql-action/commit/356bee4b4a909000f4530db85f1ea75c65cd550b"><code>356bee4</code></a> Update checked-in dependencies</li>
    <li><a href="https://github.com/github/codeql-action/commit/385808c715d3dc5b901e9fb07672dc363c43436b"><code>385808c</code></a> Update changelog and version after v3.25.9</li>
    <li>Additional commits viewable in <a href="https://github.com/github/codeql-action/compare/530d4feaa9c62aaab2d250371e2061eb7a172363...23acc5c183826b7a8a97bce3cecc52db901f8251">compare view</a></li>
    </ul>
    </details>
    <br />
    
    [![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=github/codeql-action&package-manager=github_actions&previous-version=3.25.9&new-version=3.25.10)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)
    
    Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`.
    
    ---
    
    <details>
    <summary>Dependabot commands and options</summary>
    <br />
    
    You can trigger Dependabot actions by commenting on this PR:
    - `@dependabot rebase` will rebase this PR
    - `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it
    - `@dependabot merge` will merge this PR after your CI passes on it
    - `@dependabot squash and merge` will squash and merge this PR after your CI passes on it
    - `@dependabot cancel merge` will cancel a previously requested merge and block automerging
    - `@dependabot reopen` will reopen this PR if it is closed
    - `@dependabot close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
    - `@dependabot show <dependency name> ignore conditions` will show all of the ignore conditions of the specified dependency
    - `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
    - `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
    - `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
    
    </details>
    dependabot[bot] authored Jun 14, 2024
    Configuration menu
    Copy the full SHA
    dd2ac7c View commit details
    Browse the repository at this point in the history
  2. Validate the contrastLevel during ColorScheme creation (#150176)

    Implement as an `assert` to help developers catch potential mistakes during development.
    
    Related to #149683
    parlough authored Jun 14, 2024
    Configuration menu
    Copy the full SHA
    9ea5b13 View commit details
    Browse the repository at this point in the history
  3. Include transform in static Gradient lerp methods (#149624)

    *Fixes #149534 Gradient subclasses' static lerp methods drop the GradientTransform of both `a` and `b`*
    
    LinearGradient.lerp(), RadialGradient.lerp() and SweepGradient.lerp() no longer drop the GradientTransform of `a` and/or `b`.
    
    The primitive interpolation is performed the same way TileMode is handled:  `transform: t < 0.5 ? a.transform : b.transform`.
    
    ## Media
    
    <details>
    <summary>Video demonstration</summary>
    
    ### Before
    https://github.com/flutter/flutter/assets/65806473/de14e201-b1a7-44ba-95ff-e62587c7f6ac
    
    ### After
    https://github.com/flutter/flutter/assets/65806473/d48f40e2-1b0f-4ac8-a45c-b3c423e3fd64
    
    </details>
    
      - Changed no documentation.
      - Non-breaking change.
    Zabadam authored Jun 14, 2024
    Configuration menu
    Copy the full SHA
    43e71ae View commit details
    Browse the repository at this point in the history
  4. Switch to Iterable.cast instance method (#150185)

    Switch away from the `Iterable.castFrom` static method to the `Iterable.cast` instance method which is more readable and more consistent with other iterable usages.
    parlough authored Jun 14, 2024
    Configuration menu
    Copy the full SHA
    d802df4 View commit details
    Browse the repository at this point in the history
  5. Add tests for navigator.0.dart (#150034)

    Contributes to #130459
    
    It adds a test for
    - `examples/api/lib/widgets/navigator/navigator.0.dart`
    ValentinVignal authored Jun 14, 2024
    Configuration menu
    Copy the full SHA
    349ec71 View commit details
    Browse the repository at this point in the history
Loading