Skip to content

[webview_flutter_wkwebview] Only set limitsNavigationsToAppBoundDomains when it is set to true - #5137

Merged
auto-submit[bot] merged 3 commits into
flutter:mainfrom
bparrishMines:wk_limits_fix
Oct 24, 2023
Merged

[webview_flutter_wkwebview] Only set limitsNavigationsToAppBoundDomains when it is set to true#5137
auto-submit[bot] merged 3 commits into
flutter:mainfrom
bparrishMines:wk_limits_fix

Conversation

@bparrishMines

@bparrishMinesbparrishMines commented Oct 12, 2023

Copy link
Copy Markdown
Contributor

This changes the flag to only flag to only call the method if it sets it to true.

Fixesflutter/flutter#136449

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 relevant style guides and ran the auto-formatter. (Unlike the flutter/flutter repo, the flutter/packages repo does use dart format.)
  • I signed the CLA.
  • The title of the PR starts with the name of the package surrounded by square brackets, e.g. [shared_preferences]
  • I listed at least one issue that this PR fixes in the description above.
  • I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy, or this PR is exempt from version changes.
  • I updated CHANGELOG.md to add a description of the change, following repository CHANGELOG style.
  • I updated/added relevant documentation (doc comments with ///).
  • I added new tests to check the change I am making, or this PR is test-exempt.
  • All existing and new tests are passing.

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

@bparrishMinesbparrishMines changed the title only call on setting to true[webview_flutter_wkwebview] Only set limitsNavigationsToAppBoundDomains when it is set to trueOct 13, 2023

@stuartmorgan-gstuartmorgan-g left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM, thanks for the quick fix on this.

@bparrishMinesbparrishMines added the autosubmit Merge PR when tree becomes green via auto submit App label Oct 24, 2023
@auto-submit
auto-submitBot merged commit f22ff36 into flutter:mainOct 24, 2023
@bparrishMines
bparrishMines deleted the wk_limits_fix branch October 24, 2023 19:44
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 25, 2023
auto-submitBot pushed a commit to flutter/flutter that referenced this pull request Oct 25, 2023
flutter/packages@2faf992...f751ffb
2023-10-25 stuartmorgan@google.com [ci] Enable legacy Android emulator tests (flutter/packages#5190)
2023-10-25 49699333+dependabot[bot]@users.noreply.github.com Bump ossf/scorecard-action from 2.3.0 to 2.3.1 (flutter/packages#5209)
2023-10-25 godofredoc@google.com Replace gems dependency with ruby. (flutter/packages#5219)
2023-10-25 godofredoc@google.com Update the ftl hardware version. (flutter/packages#5224)
2023-10-25 737941+loic-sharma@users.noreply.github.com [ci] Disable Windows Arm64 stable CI (flutter/packages#5217)
2023-10-24 10687576+bparrishMines@users.noreply.github.com [webview_flutter_wkwebview] Only set `limitsNavigationsToAppBoundDomains` when it is set to true (flutter/packages#5137)
2023-10-24 stuartmorgan@google.com [url_launcher] Add an `inAppBrowserView` mode (flutter/packages#5155)
2023-10-24 43054281+camsim99@users.noreply.github.com [Android] Introduces API 34 emulators for subset of tests (flutter/packages#5105)
2023-10-24 stuartmorgan@google.com [url_launcher] Add an `inAppBrowserView` mode in implementations (flutter/packages#5211)
2023-10-24 stuartmorgan@google.com [ci] Fix legacy Android task names (flutter/packages#5191)
2023-10-24 reidbaker@google.com Update annotations deps to 1.7 (flutter/packages#4935)
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 flutter-ecosystem@google.com,rmistry@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
@reidbakerreidbaker mentioned this pull request Jan 12, 2024
creatorpiyush pushed a commit to creatorpiyush/packages that referenced this pull request Jun 10, 2026
…ins` when it is set to true (flutter#5137)
This changes the flag to only flag to only call the method if it sets it to `true`.
Fixesflutter/flutter#136449
bisor0627 pushed a commit to bisor0627/packages that referenced this pull request Jun 19, 2026
…ins` when it is set to true (flutter#5137)
This changes the flag to only flag to only call the method if it sets it to `true`.
Fixesflutter/flutter#136449
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autosubmitMerge PR when tree becomes green via auto submit Appp: webview_flutterplatform-ios

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[webview_flutter] PlatformException(FWFUnsupportedVersionError, setLimitsNavigationsToAppBoundDomains is only supported on versions 14+., null, null)

2 participants

@bparrishMines@stuartmorgan-g