Skip to content

[url_launcher] Remove renderView usage #6137

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

Merged

Conversation

stuartmorgan-g
Copy link
Contributor

Removes calls to the deprecated renderView method, replacing them with best-effort lookup of the implicit view. This only affects an API that has been deprecated for almost two years, and only when using a specific optional parameter on that method, so the potential impact here is minimal, and this avoids the need for a breaking change.

In the future, when we remove this deprecated API, the workaround will go away as well.

Also opportunistically removes _ambiguate and _anonymize as they were only needed for versions of Flutter we no longer support.

Fixes flutter/flutter#143449

Pre-launch Checklist

Removes calls to the deprecated `renderView` method, replacing them with
best-effort lookup of the implicit view. This only affects an API that
has been deprecated for almost two years, and only when using a specific
optional parameter on that method, so the potential impact here is
minimal, and this avoids the need for a breaking change.

In the future, when we remove this deprecated API, the workaround will
go away as well.

Fixes flutter/flutter#143449
Copy link
Contributor

@bparrishMines bparrishMines left a comment

Choose a reason for hiding this comment

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

LGTM for the change assuming the analyze_legacy test failure has a trivial fix.

@stuartmorgan-g
Copy link
Contributor Author

assuming the analyze_legacy test failure has a trivial fix

The deprecation message turns out to have been very misleading; the fix for that failure was to require 3.16 since apparently the API didn't make 3.13.

@stuartmorgan-g stuartmorgan-g added the autosubmit Merge PR when tree becomes green via auto submit App label Feb 21, 2024
@auto-submit auto-submit bot merged commit 5f44e3d into flutter:main Feb 21, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Feb 22, 2024
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Feb 22, 2024
flutter/packages@48048f6...078c2a3

2024-02-22 [email protected] Manual roll Flutter from 5129806 to efee280 (47 revisions) (flutter/packages#6180)
2024-02-22 [email protected] Manual roll Flutter (stable) from bae5e49 to abb292a (1 revision) (flutter/packages#6179)
2024-02-21 [email protected] [in_app_purchase] Convert refreshReceipt(), startObservingPaymentQueue(), stopObservingPaymentQueue(), registerPaymentQueueDelegate(), removePaymentQueueDelegate(), showPriceConsentIfNeeded() to Pigeon (flutter/packages#6165)
2024-02-21 [email protected] [camerax] Modifies initialized camera info to reflect default AF/AE modes and the ability to set focus points for each (flutter/packages#6109)
2024-02-21 [email protected] [camerax] Implements `setFocusPoint`, `setExposurePoint`, `setExposureOffset` (flutter/packages#6059)
2024-02-21 [email protected] [various] Commit Windows build migrations (flutter/packages#6175)
2024-02-21 [email protected] [flutter_markdown] Support wasm (flutter/packages#6168)
2024-02-21 [email protected] [ci] Run Windows Arm64 build tests post-submit (flutter/packages#6166)
2024-02-21 [email protected] [url_launcher] Remove `renderView` usage (flutter/packages#6137)
2024-02-21 [email protected] [camera_android_camerax] Fix typo in readme (flutter/packages#6143)
2024-02-21 [email protected] [local_auth] Switch iOS endorsement to `local_auth_darwin` (flutter/packages#6107)

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-flutter-autoroll
Please CC [email protected],[email protected] 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
arc-yong pushed a commit to Arctuition/packages-arc that referenced this pull request Jun 14, 2024
Removes calls to the deprecated `renderView` method, replacing them with best-effort lookup of the implicit view. This only affects an API that has been deprecated for almost two years, and only when using a specific optional parameter on that method, so the potential impact here is minimal, and this avoids the need for a breaking change.

In the future, when we remove this deprecated API, the workaround will go away as well.

Also opportunistically removes `_ambiguate` and `_anonymize` as they were only needed for versions of Flutter we no longer support.

Fixes flutter/flutter#143449
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
autosubmit Merge PR when tree becomes green via auto submit App p: url_launcher
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[url_launcher] Replace deprecated renderView
3 participants