Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[macOS][Desktop]: UI is broken on latest master #104154

Closed
darshankawar opened this issue May 19, 2022 · 6 comments · Fixed by flutter/engine#33481
Closed

[macOS][Desktop]: UI is broken on latest master #104154

darshankawar opened this issue May 19, 2022 · 6 comments · Fixed by flutter/engine#33481
Labels
a: desktop Running on desktop c: regression It was better in the past than it is now c: rendering UI glitches reported at the engine/skia rendering level engine flutter/engine repository. See also e: labels. found in release: 3.1 Found to occur in 3.1 has reproducible steps The issue has been confirmed reproducible and is ready to work on platform-mac Building on or for macOS specifically r: fixed Issue is closed as already fixed in a newer version

Comments

@darshankawar
Copy link
Member

darshankawar commented May 19, 2022

UI seems to be broken and not rendering on macOS desktop using latest master (Channel master, 3.1.0-0.0.pre.799, at the time of writing this).

Steps to replicate:

  1. Create a new Flutter app (flutter create macos_issue)
  2. cd macos_issue
  3. flutter run -d macos and observe that UI is not rendered and shows a blank screen. If we try to resize the app window, then it shows flashes of the UI as we move the window size:
macos_desktop_broken.mov

The same works properly on latest stable (3.0) and beta (2.13.0-0.4.pre) as shown below:

macos_stable_rendering.mov

Moreover, this seems to be only happening on macOS desktop, whereas, macOS + web, macOS + android and Linux desktop seems to be working as expected.

master flutter doctor -v

[✓] Flutter (Channel master, 3.1.0-0.0.pre.799, on macOS 12.2.1 21D62
    darwin-x64, locale en-GB)
    • Flutter version 3.1.0-0.0.pre.799 at
      /Users/dhs/documents/fluttersdk/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 348a2b4f2f (29 minutes ago), 2022-05-19 00:33:08 -0400
    • Engine revision 1965c92ea4
    • Dart version 2.18.0 (build 2.18.0-130.0.dev)
    • DevTools version 2.13.1
    
[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.



@darshankawar darshankawar added in triage Presently being triaged by the triage team platform-mac Building on or for macOS specifically c: rendering UI glitches reported at the engine/skia rendering level a: desktop Running on desktop has reproducible steps The issue has been confirmed reproducible and is ready to work on found in release: 3.1 Found to occur in 3.1 c: regression It was better in the past than it is now framework flutter/packages/flutter repository. See also f: labels. f: material design flutter/packages/flutter/material repository. and removed in triage Presently being triaged by the triage team labels May 19, 2022
@dkwingsmt
Copy link
Contributor

dkwingsmt commented May 19, 2022

I bisected to the framework commit 348a2b4, i.e. #104137.

@TahaTesser TahaTesser added engine flutter/engine repository. See also e: labels. and removed framework flutter/packages/flutter repository. See also f: labels. f: material design flutter/packages/flutter/material repository. labels May 19, 2022
@dkwingsmt
Copy link
Contributor

dkwingsmt commented May 19, 2022

And further engine bisection points to flutter/engine#33461. In fact, when running with a local engine, instead of showing a grey window, the app simply crashes. @dnfield

Other than fixing the issue, we should probably add some smoke test.

@dnfield
Copy link
Contributor

dnfield commented May 19, 2022

Let's revert for now

@jmagman
Copy link
Member

jmagman commented May 19, 2022

@cbracken mentioned writing more macOS tests...

@cbracken
Copy link
Member

cbracken commented May 19, 2022

Indeed. There are many many many tests we need. We've been adding some, but desktop is pretty short-staffed right now. I'll gladly review patches if anyone wants to send them. :)

I just bisected this again without knowing it had already been bisected.

@darshankawar darshankawar added the r: fixed Issue is closed as already fixed in a newer version label May 20, 2022
@github-actions
Copy link

github-actions bot commented Jun 3, 2022

This thread has been automatically locked since there has not been any recent activity after it was closed. If you are still experiencing a similar issue, please open a new bug, including the output of flutter doctor -v and a minimal reproduction of the issue.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 3, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
a: desktop Running on desktop c: regression It was better in the past than it is now c: rendering UI glitches reported at the engine/skia rendering level engine flutter/engine repository. See also e: labels. found in release: 3.1 Found to occur in 3.1 has reproducible steps The issue has been confirmed reproducible and is ready to work on platform-mac Building on or for macOS specifically r: fixed Issue is closed as already fixed in a newer version
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants