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: 18340ea16cee
Choose a base ref
...
head repository: flutter/flutter
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: 14774b95c250
Choose a head ref
  • 20 commits
  • 112 files changed
  • 11 contributors

Commits on Mar 21, 2024

  1. Configuration menu
    Copy the full SHA
    0646600 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    98d10b6 View commit details
    Browse the repository at this point in the history
  3. flutter test --wasm support (#145347)

    * Adds support for `flutter test --wasm`.
      * The test compilation flow is a bit different now, so that it supports compilers other than DDC. Specifically, when we run a set of unit tests, we generate a "switchboard" main function that imports each unit test and runs the main function for a specific one based off of a value set by the JS bootstrapping code. This way, there is one compile step and the same compile output is invoked for each unit test file.
    * Also, removes all references to `dart:html` from flutter/flutter.
    * Adds CI steps for running the framework unit tests with dart2wasm+skwasm
      * These steps are marked as `bringup: true`, so we don't know what kind of failures they will result in. Any failures they have will not block the tree at all yet while we're still in `bringup: true`. Once this PR is merged, I plan on looking at any failures and either fixing them or disabling them so we can get these CI steps running on presubmit.
    
    This fixes #126692
    eyebrowsoffire authored Mar 21, 2024
    Configuration menu
    Copy the full SHA
    31209d0 View commit details
    Browse the repository at this point in the history
  4. Eliminate more window singleton usages (#145560)

    These usages in strings slipped through the initial clean-up.
    goderbauer authored Mar 21, 2024
    Configuration menu
    Copy the full SHA
    eaf509a View commit details
    Browse the repository at this point in the history
  5. Clarify AutomaticKeepAliveClientMixin semantics in build method (#145297

    )
    
    This should help folks who trace the code before reading the documentation.
    Hixie authored Mar 21, 2024
    Configuration menu
    Copy the full SHA
    afe0150 View commit details
    Browse the repository at this point in the history
  6. Configuration menu
    Copy the full SHA
    d69833c View commit details
    Browse the repository at this point in the history
  7. Fixed race condition in PollingDeviceDiscovery. (#145506)

    There are two issues in the previous implementation:
    1. `_populateDevices` will return the devices from `deviceNotifier` if it had been initialized, assuming that once it's initialized, it has been properly populated. That assumption is not true because calling getters like `onAdded` would initialize `deviceNotifier` without populating it.
    2. `deviceNotifier` instance might be replaced in some cases, causing `onAdded` subscribers to lose any future updates.
    
    To fix (1), this commit added the `isPopulated` field in `deviceNotifier` as a more accurate flag to determine if we need to populate it.
    
    To fix (2), this commit made `deviceNotifier` a final member in `PolingDeviceDiscovery`.
    chingjun authored Mar 21, 2024
    Configuration menu
    Copy the full SHA
    c759c22 View commit details
    Browse the repository at this point in the history
  8. Configuration menu
    Copy the full SHA
    a36569d View commit details
    Browse the repository at this point in the history
  9. Add a --no-gradle-generation mode to the `generate_gradle_lockfiles…

    ….dart` script (#145568)
    
    The script currently overwrites existing `settings.gradle`, `build.gradle`, and `gradle-wrapper.properties` files in the directories it processes. This mode makes it not do that, and just generate the lockfiles themselves.
    
    Related to #145564 (comment)
    gmackall authored Mar 21, 2024
    Configuration menu
    Copy the full SHA
    3b390c5 View commit details
    Browse the repository at this point in the history
  10. Fix nullability of getFullHeightForCaret (#145554)

    Fixes #145507.
    
    Looks like this was accidentally migrated to nullable all the way back when we switched to NNBD.
    goderbauer authored Mar 21, 2024
    Configuration menu
    Copy the full SHA
    11c034f View commit details
    Browse the repository at this point in the history

Commits on Mar 22, 2024

  1. Configuration menu
    Copy the full SHA
    e1c6445 View commit details
    Browse the repository at this point in the history
  2. Fix BorderSide.none requiring explicit transparent color for `Under…

    …lineInputBorder` (#145329)
    
    Fix could have been "paint transparent when Border none" but, following other Borders, we will just not paint anything.
    
    Fix #143746
    bernaferrari authored Mar 22, 2024
    Configuration menu
    Copy the full SHA
    784f19c View commit details
    Browse the repository at this point in the history
  3. Add some cross references in the docs, move an example to a dartpad e…

    …xample (#145571)
    
    ## Description
    
    This adds some "See also" links to some docs for `TweenAnimationBuilder` and `ValueListenableBuilder`.
    
    Also, moved a "snippet" example in `ValueListenableBuilder` into the examples directory as a Dartpad example.
    
    ## Tests
     - Added test for the example.
    gspencergoog authored Mar 22, 2024
    Configuration menu
    Copy the full SHA
    0f685f8 View commit details
    Browse the repository at this point in the history
  4. Replace RenderBox.compute* with RenderBox.get* and add `@visibleF…

    …orOverriding` (#145503)
    
    `@visibleForOverriding` + `@protected` unfortunately does not catch the case where a `compute*` method was overridden in a subtype and the overide was called in that same type's implementation.
    
    I did not add a `flutter_ignore` for this because it doesn't seem there will be false positives.
    LongCatIsLooong authored Mar 22, 2024
    Configuration menu
    Copy the full SHA
    d755bc2 View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    5058c26 View commit details
    Browse the repository at this point in the history
  6. Configuration menu
    Copy the full SHA
    5fab92f View commit details
    Browse the repository at this point in the history
  7. Adds numpad navigation shortcuts for Linux (#145464)

    ## Description
    
    This PR adds shortcuts related to numpad keys on Linux.
    
    ## Related Issue
    
    Linux side for #144936
    
    ## Tests
    
    Adds 2 tests.
    bleroux authored Mar 22, 2024
    Configuration menu
    Copy the full SHA
    859eb2e View commit details
    Browse the repository at this point in the history
  8. Intensive if chain refactoring (#145194)

    This pull request refactors if-statements into switch expressions, as part of the effort to solve issue #144903.
    
    Making changes beyond just swapping syntax is more difficult (and also more difficult to review, I apologize), but much more satisfying too.
    nate-thegrate authored Mar 22, 2024
    Configuration menu
    Copy the full SHA
    7fb35db View commit details
    Browse the repository at this point in the history
  9. Configuration menu
    Copy the full SHA
    7ad647a View commit details
    Browse the repository at this point in the history
  10. Configuration menu
    Copy the full SHA
    14774b9 View commit details
    Browse the repository at this point in the history
Loading