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

Revert "iOS: Migrate FlutterEngine to ARC (#55590)" #55699

Merged
merged 1 commit into from
Oct 7, 2024

Conversation

cbracken
Copy link
Member

@cbracken cbracken commented Oct 7, 2024

Causing flakes on the Mac mac_unopt (Cocoon) shard.

This reverts commit 449df25.

Issue: flutter/flutter#156177

Pre-launch Checklist

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • I read the Tree Hygiene wiki page, which explains my responsibilities.
  • I read and followed the Flutter Style Guide and the C++, Objective-C, Java style guides.
  • I listed at least one issue that this PR fixes in the description above.
  • I added new tests to check the change I am making or feature I am adding, or the PR is test-exempt. See testing the engine for instructions on writing and running engine tests.
  • I updated/added relevant documentation (doc comments with ///).
  • I signed the CLA.
  • All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Causing flakes on the _Mac mac_unopt (Cocoon)_ shard.

This reverts commit 449df25.

Issue: flutter/flutter#156177
Copy link
Contributor

@stuartmorgan-g stuartmorgan-g left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

RSLGTM

@cbracken cbracken added the autosubmit Merge PR when tree becomes green via auto submit App label Oct 7, 2024
@auto-submit auto-submit bot merged commit 2484ae1 into flutter:main Oct 7, 2024
30 checks passed
@cbracken cbracken deleted the revert-engine-arc branch October 7, 2024 16:57
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Oct 7, 2024
…156347)

flutter/engine@ffacad9...0c9d3f1

2024-10-07 skia-flutter-autoroll@skia.org Roll Dart SDK from 5e6945a72200 to 902494bb4c8b (1 revision) (flutter/engine#55700)
2024-10-07 chris@bracken.jp Revert "iOS: Migrate FlutterEngine to ARC (#55590)" (flutter/engine#55699)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC matanl@google.com,zra@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter: 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
nick9822 pushed a commit to nick9822/flutter that referenced this pull request Dec 18, 2024
…ngine#55699)

Causing flakes on the _Mac mac_unopt (Cocoon)_ shard.

This reverts commit a2ac734.

Issue: flutter#156177

[C++, Objective-C, Java style guides]: https://github.com/flutter/engine/blob/main/CONTRIBUTING.md#style
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
autosubmit Merge PR when tree becomes green via auto submit App platform-ios
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants