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

Commits on Mar 5, 2024

  1. Revert "_DefaultTabControllerState should dispose all created TabCont…

    …oller instances. (#136608)" (#144579)
    
    This reverts commit 9fa9fd3.
    
    Fixes #144087.
    Fixes #138588.
    
    This crash has been reported previously from a customer in google3 in #138588, but we weren't able to track it down. Thankfully, a repro was now provided in #144087 (comment) which traced the crash down to a change made in #136608. This PR reverts that change to fix that crash for now. I will post a new PR shortly that will add a test to cover the use case that caused the crash with #136608 to make sure we don't re-introduce the crash in the future.
    goderbauer authored Mar 5, 2024
    Configuration menu
    Copy the full SHA
    d93f24a View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    4e71b4d View commit details
    Browse the repository at this point in the history
  3. remove unused firstBuildTime parameter in DevFS::update (#144576)

    The title says it all. This parameter is unused and serves no apparent purpose.
    andrewkolos authored Mar 5, 2024
    Configuration menu
    Copy the full SHA
    4e6de2b View commit details
    Browse the repository at this point in the history
  4. Roll Packages from 0625827 to 2aa6e3f (5 revisions) (#144628)

    flutter/packages@0625827...2aa6e3f
    
    2024-03-05 [email protected] [in_app_purchase_storekit] Fix type of error code returned from native code in `SKReceiptManager.retrieveReceiptData` (flutter/packages#6265)
    2024-03-05 [email protected] [video_player] Fix typo in readme (flutter/packages#6264)
    2024-03-05 [email protected] [pigeon] Separates message call code generation into separate methods in the SwiftGenerator (flutter/packages#5959)
    2024-03-05 [email protected] [flutter_markdown] Fix WidgetSpan Support in MarkdownElementBuilder (flutter/packages#6225)
    2024-03-04 [email protected] [webview_flutter] Make WebKitWebViewWidget unit tests stateless (flutter/packages#6228)
    
    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://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
    engine-flutter-autoroll authored Mar 5, 2024
    Configuration menu
    Copy the full SHA
    ba7749a View commit details
    Browse the repository at this point in the history
  5. Add regression test for TabBar crash (#144627)

    This is a regression test for #144087 and #138588.
    
    To be submitted after #144579.
    goderbauer authored Mar 5, 2024
    Configuration menu
    Copy the full SHA
    2aa1efc View commit details
    Browse the repository at this point in the history
  6. Configuration menu
    Copy the full SHA
    bb8fc6e View commit details
    Browse the repository at this point in the history
  7. Adds missing style to PopupMenuButton (#143392)

    Adds missing `style` to `PopupMenuButton`.
    
    Fixes: #114709
    piedcipher authored Mar 5, 2024
    Configuration menu
    Copy the full SHA
    311c006 View commit details
    Browse the repository at this point in the history
  8. Updated the smiley TextButton example again (#144630)

    Improved the smiley image TextButton example a little. Handling the case where the `Future.delayed` object that represents the button's one second long action is superseded by a second button press that triggers a new one-second action. This does complicate the example - just a little - but it's a little more robust. In case someone copy-and-pastes the code.
    
    The TextButton example was recently updated: #144583
    HansMuller authored Mar 5, 2024
    Configuration menu
    Copy the full SHA
    52961f7 View commit details
    Browse the repository at this point in the history
  9. Roll Flutter Engine from 3e8b0deffe4e to 49bc1577f317 (5 revisions) (#…

    …144639)
    
    flutter/engine@3e8b0de...49bc157
    
    2024-03-05 [email protected] Fix typo in githook message (flutter/engine#51205)
    2024-03-05 [email protected] Roll Skia from 453cb0d04584 to 875b356e4d96 (2 revisions) (flutter/engine#51202)
    2024-03-05 [email protected] Add device selection to `et run` (flutter/engine#51184)
    2024-03-05 [email protected] Roll Skia from de108a9c8b54 to 453cb0d04584 (2 revisions) (flutter/engine#51201)
    2024-03-05 [email protected] Adds githooks that reminds to run `gclient sync -D` (flutter/engine#51156)
    
    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://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
    engine-flutter-autoroll authored Mar 5, 2024
    Configuration menu
    Copy the full SHA
    15e8d32 View commit details
    Browse the repository at this point in the history
  10. Enable asset transformation feature in hot reload workflow (excluding…

    … Web) (#144161)
    
    Partial implementation of #143348
    
    This enables asset transformation during hot reload (except for web, because that has its own implementation of `DevFS` �). Asset transformers will be reapplied after changing any asset and performing a hot reload during `flutter run`.
    andrewkolos authored Mar 5, 2024
    Configuration menu
    Copy the full SHA
    ff3b6dc View commit details
    Browse the repository at this point in the history
  11. Configuration menu
    Copy the full SHA
    7e05bc4 View commit details
    Browse the repository at this point in the history
  12. Bump codecov/codecov-action from 4.0.2 to 4.1.0 (#144174)

    Bumps [codecov/codecov-action](https://github.com/codecov/codecov-action) from 4.0.2 to 4.1.0.
    <details>
    <summary>Release notes</summary>
    <p><em>Sourced from <a href="https://github.com/codecov/codecov-action/releases">codecov/codecov-action's releases</a>.</em></p>
    <blockquote>
    <h2>v4.1.0</h2>
    <h2>What's Changed</h2>
    <ul>
    <li>fix: set safe directory by <a href="https://github.com/thomasrockhu-codecov"><code>@�thomasrockhu-codecov</code></a> in <a href="https://redirect.github.com/codecov/codecov-action/pull/1304">codecov/codecov-action#1304</a></li>
    <li>build(deps): bump github/codeql-action from 3.24.3 to 3.24.5 by <a href="https://github.com/dependabot"><code>@�dependabot</code></a> in <a href="https://redirect.github.com/codecov/codecov-action/pull/1306">codecov/codecov-action#1306</a></li>
    <li>build(deps-dev): bump eslint from 8.56.0 to 8.57.0 by <a href="https://github.com/dependabot"><code>@�dependabot</code></a> in <a href="https://redirect.github.com/codecov/codecov-action/pull/1305">codecov/codecov-action#1305</a></li>
    <li>chore(release): v4.1.0 by <a href="https://github.com/thomasrockhu-codecov"><code>@�thomasrockhu-codecov</code></a> in <a href="https://redirect.github.com/codecov/codecov-action/pull/1307">codecov/codecov-action#1307</a></li>
    </ul>
    <p><strong>Full Changelog</strong>: <a href="https://github.com/codecov/codecov-action/compare/v4.0.2...v4.1.0">https://github.com/codecov/codecov-action/compare/v4.0.2...v4.1.0</a></p>
    </blockquote>
    </details>
    <details>
    <summary>Commits</summary>
    <ul>
    <li><a href="https://github.com/codecov/codecov-action/commit/54bcd8715eee62d40e33596ef5e8f0f48dbbccab"><code>54bcd87</code></a> chore(release): v4.1.0 (<a href="https://redirect.github.com/codecov/codecov-action/issues/1307">#1307</a>)</li>
    <li><a href="https://github.com/codecov/codecov-action/commit/8ba77ef8d7b55bda0f69297c841058adf8459f89"><code>8ba77ef</code></a> build(deps-dev): bump eslint from 8.56.0 to 8.57.0 (<a href="https://redirect.github.com/codecov/codecov-action/issues/1305">#1305</a>)</li>
    <li><a href="https://github.com/codecov/codecov-action/commit/c60aa801e394788e00c8c837b5e250085fbe5df8"><code>c60aa80</code></a> build(deps): bump github/codeql-action from 3.24.3 to 3.24.5 (<a href="https://redirect.github.com/codecov/codecov-action/issues/1306">#1306</a>)</li>
    <li><a href="https://github.com/codecov/codecov-action/commit/2fc4847d3f22c2ca7f5bab5504f530b55d4a9768"><code>2fc4847</code></a> fix: set safe directory (<a href="https://redirect.github.com/codecov/codecov-action/issues/1304">#1304</a>)</li>
    <li>See full diff in <a href="https://github.com/codecov/codecov-action/compare/0cfda1dd0a4ad9efc75517f399d859cd1ea4ced1...54bcd8715eee62d40e33596ef5e8f0f48dbbccab">compare view</a></li>
    </ul>
    </details>
    <br />
    
    [![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=codecov/codecov-action&package-manager=github_actions&previous-version=4.0.2&new-version=4.1.0)](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 Mar 5, 2024
    Configuration menu
    Copy the full SHA
    bbb0052 View commit details
    Browse the repository at this point in the history

Commits on Mar 6, 2024

  1. Roll Flutter Engine from 49bc1577f317 to effcf97a1f7c (10 revisions) (#…

    …144653)
    
    flutter/engine@49bc157...effcf97
    
    2024-03-05 98614782+auto-submit[bot]@users.noreply.github.com Reverts "[Impeller] Enable depth buffer clipping & Stencil-then-Cover path rendering. (#51209)" (flutter/engine#51217)
    2024-03-05 [email protected] [Impeller] implement blur style solid (flutter/engine#50892)
    2024-03-05 [email protected] [g3 roll] Revert "Remove unused drone_dimension field" (flutter/engine#51214)
    2024-03-05 [email protected] Roll Skia from 17677914dabf to 37947aec8c5c (5 revisions) (flutter/engine#51211)
    2024-03-05 [email protected] Fix git hooks on Windows (flutter/engine#51203)
    2024-03-05 [email protected] [Impeller] Enable depth buffer clipping & Stencil-then-Cover path rendering. (flutter/engine#51209)
    2024-03-05 [email protected] [Impeller] Fix convex triangulation winding bug for multi-contour paths. (flutter/engine#51198)
    2024-03-05 [email protected] Roll Skia from 875b356e4d96 to 17677914dabf (2 revisions) (flutter/engine#51207)
    2024-03-05 [email protected] Skip configuration dependency if unit tests are disabled (flutter/engine#51179)
    2024-03-05 [email protected] [Impeller] Implement YUV texture import and sampling for video player frames. (flutter/engine#50730)
    
    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://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
    engine-flutter-autoroll authored Mar 6, 2024
    Configuration menu
    Copy the full SHA
    a431dee View commit details
    Browse the repository at this point in the history
  2. Fill in SliverConstraints fields missing from ==, hashCode, toString (#…

    …143661)
    
    I was doing some debugging on a RenderSliver subclass, and found
    that SliverConstraints.toString was missing the precedingScrollExtent
    field.
    
    Add that, and add both that field and userScrollDirection to the
    `==` and hashCode implementations, which had been skipping them,
    so that all three methods now handle all the class's fields.
    gnprice authored Mar 6, 2024
    Configuration menu
    Copy the full SHA
    6b9d3ea View commit details
    Browse the repository at this point in the history
  3. Roll Flutter Engine from effcf97a1f7c to d374c78bcf52 (5 revisions) (#…

    …144659)
    
    flutter/engine@effcf97...d374c78
    
    2024-03-06 [email protected] Revamp the `scenario_app/../README.md` to encourage self-sustenance (flutter/engine#51196)
    2024-03-06 [email protected] [Impeller] disable blending in gaussian intermediate steps. (flutter/engine#51118)
    2024-03-06 [email protected] [Impeller] Enable depth buffer clipping & Stencil-then-Cover path rendering. (flutter/engine#51219)
    2024-03-06 [email protected] Roll Skia from 37947aec8c5c to 1d1fd7fe1a89 (1 revision) (flutter/engine#51216)
    2024-03-06 [email protected] [Impeller] Fix path winding when bridging from contours with an odd number of points. (flutter/engine#51218)
    
    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://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
    engine-flutter-autoroll authored Mar 6, 2024
    Configuration menu
    Copy the full SHA
    2ab00ee View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    e1013f2 View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    d0c8ad2 View commit details
    Browse the repository at this point in the history
  6. Roll Packages from 2aa6e3f to 9b88dbc (8 revisions) (#144693)

    flutter/packages@2aa6e3f...9b88dbc
    
    2024-03-06 [email protected] [image_picker_for_web] migrates to package:web (flutter/packages#5799)
    2024-03-06 [email protected] [video_player_web] migrates to package:web (flutter/packages#5800)
    2024-03-06 49699333+dependabot[bot]@users.noreply.github.com [in_app_pur]: Bump org.json:json from 20240205 to 20240303 in /packages/in_app_purchase/in_app_purchase/example/android/app (flutter/packages#6253)
    2024-03-05 [email protected] [rfw] Change test coverage logic to enforce 100% coverage (flutter/packages#6272)
    2024-03-05 [email protected] [in_app_purchase_storekit] backfill native tests for more complete test coverage (flutter/packages#6209)
    2024-03-05 [email protected] [tool] Add features to support GCB auto-publish flow (flutter/packages#6218)
    2024-03-05 [email protected] [web] Use TrustedTypes from pkg web. (flutter/packages#6273)
    2024-03-05 [email protected] Roll Flutter from 65cd84b to 3b5a2ec (26 revisions) (flutter/packages#6269)
    
    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://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
    engine-flutter-autoroll authored Mar 6, 2024
    Configuration menu
    Copy the full SHA
    8f84f3f View commit details
    Browse the repository at this point in the history
Loading