Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.
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/engine
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: 525bd7dcf7f3
Choose a base ref
...
head repository: flutter/engine
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: a65a1b55e06a
Choose a head ref
  • 11 commits
  • 102 files changed
  • 5 contributors

Commits on Jan 26, 2024

  1. Configuration menu
    Copy the full SHA
    65d1291 View commit details
    Browse the repository at this point in the history
  2. Fix Shell::Screenshot for Impeller (#50072)

    Fixes flutter/flutter#141571
    
    Shell::Screenshot was impelemnted in a Skia-only way and would crash when invoked. Adds test coverage for the breaking path on iOS that ends up calling `FlutterView drawLayer`.
    dnfield authored Jan 26, 2024
    Configuration menu
    Copy the full SHA
    b307a6c View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    ddb3b35 View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    45e1164 View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    ce10577 View commit details
    Browse the repository at this point in the history
  6. Finish landing missing/incorrect header guards across flutter/engine (

    #50069)
    
    Generated by #48903 (`dart ./tools/header_guard_check/bin/main.dart --fix`).
    
    As discussed with @cbracken and @jmagman, the guards are not technically needed on the Mac/iOS code, but they (a) do not hurt and (b) still provide value if for some reason `#include` is used instead of `#import` (though I suspect we could try to add that to the tool in the future as well).
    matanlurey authored Jan 26, 2024
    Configuration menu
    Copy the full SHA
    182b595 View commit details
    Browse the repository at this point in the history
  7. [web] Do not wipe the PlatformViewManager when disposing of a view. (#…

    …49991)
    
    When a view gets disposed of, its rasterizer completely clears up the singleton `PlatformViewManager`. Particularly, it removes all registered platform view factories.
    
    This is wrong because the remaining PlatformViews on the page cannot be re-rendered, and the default Platform View factories (used by `pointer_interceptor`, for example), disappear.
    
    This PR attempts to preserve the `dispose` logic of the canvaskit rasterizer, without using the `debugClear` method of the `PlatformViewManager` (which is supposedly test-only).
    
    ## Issues
    
    * Fixes flutter/flutter#142094
    
    ## Tests
    
    * Added unit-test
    * Deployed demo app: https://dit-maps-tests.web.app
    
    [C++, Objective-C, Java style guides]: https://github.com/flutter/engine/blob/main/CONTRIBUTING.md#style
    ditman authored Jan 26, 2024
    Configuration menu
    Copy the full SHA
    70039d3 View commit details
    Browse the repository at this point in the history
  8. Configuration menu
    Copy the full SHA
    daf139b View commit details
    Browse the repository at this point in the history
  9. Introduce a prototype of a "header guard enforcement" tool (#48903)

    Closes flutter/flutter#133415.
    
    ---
    
    This is a prototype I threw together in about 1-2 hours. It enforces and
    automatically fixes header guards that don't match the [the Google C++
    style
    guide](https://google.github.io/styleguide/cppguide.html#The__define_Guard).
    For example, here is (trimmed) output at HEAD:
    
    ```txt
    line 5, column 1 of impeller/aiks/picture.h: Unexpected #pragma once
      ╷
    5 │ #pragma once
      │ ^^^^^^^^^^^^
      ╵
    line 5, column 1 of flow/stopwatch.h: Expected #ifndef FLUTTER_FLOW_STOPWATCH_H_
      ╷
    5 │ #ifndef FLUTTER_FLOW_INSTRUMENTATION_H_
      │ ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
      ╵
    The following 731 files have invalid header guards:
    ```
    matanlurey authored Jan 26, 2024
    Configuration menu
    Copy the full SHA
    d33b2dd View commit details
    Browse the repository at this point in the history
  10. [Impeller] add compute pass API for memory barriers, re-enable for Vu…

    …lkan. (#49946)
    
    Adds two new APIs that insert memory barriers for compute -> compute dependencies.
    
    ```c++
      // |ComputePass|
      void AddBufferMemoryBarrier() override;
    
      // |ComputePass|
      void AddTextureMemoryBarrier() override;
    ```
    
    Also makes the ComputePassVK automatically insert a compute -> vertex dependency when encoding.  This change is sufficient to let the GPU compute implementation of draw points work on Pixel and Samsung Android devices.
    
    For more explaination on these specific barriers, see the documentation added in this PR.
    
    Fixes #49946
    Jonah Williams authored Jan 26, 2024
    Configuration menu
    Copy the full SHA
    d27b18d View commit details
    Browse the repository at this point in the history
  11. Configuration menu
    Copy the full SHA
    a65a1b5 View commit details
    Browse the repository at this point in the history
Loading