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

[pigeon] Removes the @protected annotation from the InstanceManager field of the PigeonInternalProxyApiBaseClass #8125

Merged
merged 2 commits into from
Nov 21, 2024

Conversation

bparrishMines
Copy link
Contributor

@bparrishMines bparrishMines commented Nov 19, 2024

It's common to access the InstanceManager when creating platform views, so this removes the @protected annotation to prevent lint warnings.

See #7794 (comment)

Pre-launch Checklist

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

@bparrishMines bparrishMines changed the title update field [pigeon] Removes the @protected annotation from the InstanceManager field of the PigeonInternalProxyApiBaseClass Nov 19, 2024
@bparrishMines bparrishMines marked this pull request as ready for review November 19, 2024 18:39
Copy link
Contributor

@stuartmorgan stuartmorgan left a comment

Choose a reason for hiding this comment

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

LGTM

@bparrishMines bparrishMines added the autosubmit Merge PR when tree becomes green via auto submit App label Nov 21, 2024
@auto-submit auto-submit bot merged commit 5180c03 into flutter:main Nov 21, 2024
77 checks passed
@bparrishMines bparrishMines deleted the pigeon_no_protected branch November 21, 2024 21:54
sinyu1012 added a commit to sinyu1012/packages that referenced this pull request Nov 22, 2024
* main: (64 commits)
  [quick_actions_plaform_interface] add localizedSubtitle (flutter#8112)
  [tools] Don't check license of generated Swift package (flutter#8137)
  Roll Flutter from 8536b96ebb3e to 93d772c5cdd8 (37 revisions) (flutter#8147)
  [go_router] Fix: Consistent PopScope Handling on Root Routes issue #140869 (flutter#8045)
  [in_app_purchase_storekit] fix price displayed with wrong precision (flutter#8127)
  [pigeon] Removes the `@protected` annotation from the InstanceManager field of the   `PigeonInternalProxyApiBaseClass` (flutter#8125)
  [google_maps_flutter] Use structured Pigeon data on iOS (flutter#8142)
  [vector_graphics] handle errors from bytes loader (flutter#8080)
  [flutter_svg] Fix SvgNetworkLoader not closing internal http client (flutter#8126)
  [video_player_avfoundation] send video load failure even when eventsink was initialized late (flutter#7194)
  [flutter_markdown] enable Wasm support (flutter#8120)
  Reverts "[url_launcher] Add Swift Package Manager integration to example app (flutter#8128)" (flutter#8136)
  [url_launcher] Add Swift Package Manager integration to example app (flutter#8128)
  [pigeon] Enable example app build in CI (flutter#8119)
  [in_app_purchase_storekit] disallow ios versions lower than supported from enabling storekit (flutter#8110)
  [interactive_media_ads]: Bump com.google.ads.interactivemedia.v3:interactivemedia from 3.35.1 to 3.36.0 in /packages/interactive_media_ads/android (flutter#8046)
  [interactive_media_ads]: Bump androidx.annotation:annotation from 1.8.2 to 1.9.1 in /packages/interactive_media_ads/android (flutter#7980)
  [webview_flutter_android] Updates plugin to use `ProxyApis`s (flutter#7794)
  [interactive_media_ads] Adds support to define parameters that control the rendering of ads (flutter#8057)
  Roll Flutter from b3818f6b5979 to 8536b96ebb3e (22 revisions) (flutter#8124)
  ...

# Conflicts:
#	packages/quick_actions/quick_actions_platform_interface/CHANGELOG.md
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 22, 2024
…nceManager field of the `PigeonInternalProxyApiBaseClass` (flutter/packages#8125)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 22, 2024
…nceManager field of the `PigeonInternalProxyApiBaseClass` (flutter/packages#8125)
github-merge-queue bot pushed a commit to flutter/flutter that referenced this pull request Nov 22, 2024
flutter/packages@913b99e...9203213

2024-11-22 [email protected] Reland
"[url_launcher] Add Swift Package Manager integration to example app"
(flutter/packages#8148)
2024-11-22 [email protected] [webview_flutter_wkwebview] Webkit
webview controller multiple registration fix (flutter/packages#8078)
2024-11-22 [email protected] [quick_actions_plaform_interface] add
localizedSubtitle (flutter/packages#8112)
2024-11-22 [email protected] [tools] Don't
check license of generated Swift package (flutter/packages#8137)
2024-11-21 [email protected] Roll Flutter from
8536b96 to 93d772c (37 revisions) (flutter/packages#8147)
2024-11-21 [email protected] [go_router] Fix: Consistent PopScope
Handling on Root Routes issue #140869 (flutter/packages#8045)
2024-11-21 [email protected] [in_app_purchase_storekit] fix price
displayed with wrong precision (flutter/packages#8127)
2024-11-21 [email protected] [pigeon]
Removes the `@protected` annotation from the InstanceManager field of
the `PigeonInternalProxyApiBaseClass` (flutter/packages#8125)
2024-11-21 [email protected] [google_maps_flutter] Use structured
Pigeon data on iOS (flutter/packages#8142)

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] 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
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: pigeon
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants