Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Conversation

@akbiggs
Copy link
Contributor

@akbiggs akbiggs commented Aug 22, 2022

This crash has come up a number of times and we tend to
burn time looking in the wrong places for the fix. Documenting
some stuff about how we've fixed it previously in the error
message.

@flutter-dashboard
Copy link

It looks like this pull request may not have tests. Please make sure to add tests before merging. If you need an exemption to this rule, contact Hixie on the #hackers channel in Chat (don't just cc him here, he won't see it! He's on Discord!).

If you are not sure if you need tests, consider this rule of thumb: the purpose of a test is to make sure someone doesn't accidentally revert the fix. Ask yourself, is there anything in your PR that you feel it is important we not accidentally revert back to how it was before your fix?

Reviewers: Read the Tree Hygiene page and make sure this patch meets those guidelines before LGTMing.

This crash has come up a number of times and we tend to
burn time looking in the wrong places for the fix. Documenting
some stuff about how we've fixed it previously in the error
message.
@akbiggs akbiggs force-pushed the flatland_views_error branch from fe55b82 to d2806c0 Compare August 22, 2022 17:20
@akbiggs akbiggs merged commit 4740824 into flutter:main Aug 24, 2022
@akbiggs akbiggs deleted the flatland_views_error branch August 24, 2022 20:00
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Aug 24, 2022
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Aug 24, 2022
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Aug 24, 2022
zanderso pushed a commit to flutter/flutter that referenced this pull request Aug 24, 2022
…110219)

* fa86e2d82 Zip FlutterMacOS.framework. (flutter/engine#35623)

* ce987d4cb Roll Dart SDK from 859e9bbe6f0a to 691aa6246705 (1 revision) (flutter/engine#35672)

* ab3569684 Roll Skia from d965473d8719 to f0593b093a72 (2 revisions) (flutter/engine#35675)

* 474082412 [fuchsia] Document `found != flatland_views.end()`. (flutter/engine#35607)

* c6c872137 Roll Skia from f0593b093a72 to 8acbc3ce4e84 (7 revisions) (flutter/engine#35676)

* 741709107 Set temporary GN arg for Skia roll (flutter/engine#35651)

* 1c5c7052b Revert "Zip FlutterMacOS.framework. (#35623)" (flutter/engine#35680)

* 983e3abda [Impeller] Fix matrix printing (flutter/engine#35656)

* c9876d7a3 Roll Fuchsia Mac SDK from XrmzJCUPGAd4klJlw... to uiiEV3lEQoosICL43... (flutter/engine#35677)
GaryQian pushed a commit to GaryQian/engine that referenced this pull request Aug 25, 2022
This crash has come up a number of times and we tend to
burn time looking in the wrong places for the fix. Documenting
some stuff about how we've fixed it previously in the error
message.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants