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/packages
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: 5b1bea87e8b9
Choose a base ref
...
head repository: flutter/packages
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: 3bddf2c57eed
Choose a head ref
  • 5 commits
  • 93 files changed
  • 4 contributors

Commits on Feb 3, 2026

  1. Roll Flutter from c305f1f7ad35 to bf701fefec86 (9 revisions) (#10957)

    flutter/flutter@c305f1f...bf701fe
    
    2026-02-03 engine-flutter-autoroll@skia.org Roll Dart SDK from 01ba84eeb32f to 56294a92d5cc (2 revisions) (flutter/flutter#181851)
    2026-02-03 brackenavaron@gmail.com Remove material from scrollable_helpers_test.dart (flutter/flutter#181460)
    2026-02-03 brackenavaron@gmail.com Reduce reliance on Material in page_transitions_test.dart (flutter/flutter#181467)
    2026-02-03 116356835+AbdeMohlbi@users.noreply.github.com Fix todo about `replacing URL with constants` in `gradle_errors.dart` (flutter/flutter#181673)
    2026-02-03 engine-flutter-autoroll@skia.org Roll Fuchsia Linux SDK from 1L4m9qCikk-JzrNWE... to UmQaaNuhkiuE8Dzug... (flutter/flutter#181841)
    2026-02-03 engine-flutter-autoroll@skia.org Roll Dart SDK from 1aa8f2de7587 to 01ba84eeb32f (2 revisions) (flutter/flutter#181836)
    2026-02-03 137456488+flutter-pub-roller-bot@users.noreply.github.com Roll pub packages (flutter/flutter#181738)
    2026-02-03 brackenavaron@gmail.com Remove material from slivers_appbar_floating_pinned_test.dart + slivers_appbar_stretch_test.dart + sliver_persistent_header_test.dart + slivers_appbar_floating_test.dart (flutter/flutter#181469)
    2026-02-02 engine-flutter-autoroll@skia.org Roll Skia from 43fa79e1c51f to f37a22506eb4 (1 revision) (flutter/flutter#181814)
    
    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
    Please CC stuartmorgan@google.com on the revert to ensure that a human
    is aware of the problem.
    
    To file a bug in Packages: 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 Feb 3, 2026
    Configuration menu
    Copy the full SHA
    f07e754 View commit details
    Browse the repository at this point in the history

Commits on Feb 4, 2026

  1. [go_router_builder] Add support for @TypedGoRouteParameter to custo…

    …mize parameter names (#10793)
    
    Relates to flutter/flutter#112152
    
    Add support for `@TypedGoRouteParameter` to customize parameter names
    
    Needs #10792 to get merged first
    
    ## Pre-Review Checklist
    
    **Note**: The Flutter team is currently trialing the use of [Gemini Code Assist for GitHub](https://developers.google.com/gemini-code-assist/docs/review-github-code). Comments from the `gemini-code-assist` bot should not be taken as authoritative feedback from the Flutter team. If you find its comments useful you can update your code accordingly, but if you are unsure or disagree with the feedback, please feel free to wait for a Flutter team member's review for guidance on which automated comments should be addressed.
    
    [^1]: Regular contributors who have demonstrated familiarity with the repository guidelines only need to comment if the PR is not auto-exempted by repo tooling.
    ValentinVignal authored Feb 4, 2026
    Configuration menu
    Copy the full SHA
    df85ef0 View commit details
    Browse the repository at this point in the history
  2. [google_maps_flutter] Remove use of OCMock (#10863)

    Eliminates usage of OCMock in the native iOS unit tests:
    - Creates a protocol for image asset loading that wraps that part of the registrar and the UIImage class method to replace mocking with injecting a protocol implementation.
    - Creates an abstract delegate for event reporting, instead of all of the sub-controllers communicating directly with the FlutterAPI instance.
    - Creates an abstract delegate for tile queries, instead of all of the tile controller communicating directly with the FlutterAPI instance.
    - Replaces mocking of our test subclass of the Google Maps view object with slightly more logic on that existing test subclass.
    - Replaces mocking of our existing protocols with simple manual fakes that lets us test the same assertions.
    - Eliminates static mock testing of GMSCameraUpdate.
    
    Unblocks flutter/flutter#119108
    
    ## Pre-Review Checklist
    
    **Note**: The Flutter team is currently trialing the use of [Gemini Code Assist for GitHub](https://developers.google.com/gemini-code-assist/docs/review-github-code). Comments from the `gemini-code-assist` bot should not be taken as authoritative feedback from the Flutter team. If you find its comments useful you can update your code accordingly, but if you are unsure or disagree with the feedback, please feel free to wait for a Flutter team member's review for guidance on which automated comments should be addressed.
    
    [^1]: Regular contributors who have demonstrated familiarity with the repository guidelines only need to comment if the PR is not auto-exempted by repo tooling.
    stuartmorgan-g authored Feb 4, 2026
    Configuration menu
    Copy the full SHA
    f3ee98a View commit details
    Browse the repository at this point in the history
  3. [video_player] Remove OCMock (#10932)

    Adds protocols to wrap usage of many AVFoundation objects in order to eliminate OCMock from the existing tests. This intentionally doesn't wrap everything, because it's already a large change; increasing test coverage in the future will almost certainly require adding more protocols.
    
    As with other PRs following this approach, in the short term this adds fair amount of production boilerplate (all the passthrough implementations in AVFVPFactory), but in the long term that boilerplate will almost all be eliminated in favor of using Swift protocol conformance in extensions, to just graft the protocols directly onto the real objects.
    
    This also requires more manual stub/mock test objects, but it does have some advantages beyond unblocking migration to Swift tests:
    - In many cases the tests themselves are less verbose, because some operations like triggering callbacks is cumbersome with OCMock.
    - It's much clearer where the test are using real AVFoundation objects that are doing things like loading network resources. (The answer is "far more than actually need to", but I deemed fixing that out of scope for this PR. This structure should make it easier to move most tests away from real objects in future test improvement work.)
    
    As an opportunistic fix while touching a lot of files anyway, I updated all the imports to the modern module import style.
    
    Helps unblock flutter/flutter#119105
    
    ## Pre-Review Checklist
    
    - [x ] I read the [Tree Hygiene] page, which explains my responsibilities.
    
    [^1]: Regular contributors who have demonstrated familiarity with the repository guidelines only need to comment if the PR is not auto-exempted by repo tooling.
    stuartmorgan-g authored Feb 4, 2026
    Configuration menu
    Copy the full SHA
    f4ea763 View commit details
    Browse the repository at this point in the history
  4. [camera_avfoundation] ios saving path (#10832)

    ## Description
    
    Changed iOS file storage path from `documentDirectory` to `temporaryDirectory` to ensure consistency with Android where files are stored in cache directory.
    
    ## Changes
    - iOS now uses `temporaryDirectory` instead of `documentDirectory` for consistency with Android
    
    ## Code changes
    ```swift
    // Before:
    let documentDirectory = FileManager.default.urls(
          for: .documentDirectory,
          in: .userDomainMask)[0]
    
    // After:
    let documentDirectory = FileManager.default.temporaryDirectory
    ```
    
    ## Issues 
    flutter/flutter#166933
    
    ## Pre-Review Checklist
    
    **Note**: The Flutter team is currently trialing the use of [Gemini Code Assist for GitHub](https://developers.google.com/gemini-code-assist/docs/review-github-code). Comments from the `gemini-code-assist` bot should not be taken as authoritative feedback from the Flutter team. If you find its comments useful you can update your code accordingly, but if you are unsure or disagree with the feedback, please feel free to wait for a Flutter team member's review for guidance on which automated comments should be addressed.
    
    [^1]: Regular contributors who have demonstrated familiarity with the repository guidelines only need to comment if the PR is not auto-exempted by repo tooling.
    ivan-vanyusho authored Feb 4, 2026
    Configuration menu
    Copy the full SHA
    3bddf2c View commit details
    Browse the repository at this point in the history
Loading