Replace deprecated openURL API call #164247
Conversation
| [UIApplication.sharedApplication openURL:url]; | ||
| [UIApplication.sharedApplication openURL:url | ||
| options:@{} | ||
| completionHandler:nil]; |
There was a problem hiding this comment.
cc @hannah-hyj i may need your help testing this out on a physical device. Do you remember what you did before to trigger this line?
There was a problem hiding this comment.
yes, to trigger this line, you should set up AASA file for deep link in certain domain, and not support that deep link in your flutter app, so the link will be opened again in browser.
There was a problem hiding this comment.
i will help testing if this is working
|
tested on my device, the new API works well and can open browser when needed. |
5b33671 to
4bffc71
Compare
|
Nvm, i realized that i don't have a valid reason to skip the tests. will add some tests. |
|
@hannah-hyj Thanks for testing it. Could you stamp it? Thanks! |
|
Drive by questions: Could we have caught this sooner? My understanding is this deprecated API should've produced a warning? Are there other deprecated APIs we're using that are causing warnings? Do we have warnings as errors disabled on Flutter iOS? If yes, should/could we enable that? |
|
This needs a rebase for the presub to pass. |
It's not enabled yet. I believe we still have a few usage of deprecated API in the engine. I have seen some PRs of removing some of them over the years. |
baae1f2 to
a2cd20e
Compare
|
autosubmit label was removed for flutter/flutter/164247, because - The status or check suite Google testing has failed. Please fix the issues identified (or deflake) before re-applying this label. |
a2cd20e to
1e2e442
Compare
This PR replace the deprecated
openURL:API with equivalentopenURL:options:completionHandler:API.List which issues are fixed by this PR. You must list at least one issue. An issue is not required if the PR fixes something trivial like a typo.
#164047
If you had to change anything in the flutter/tests repo, include a link to the migration guide as per the breaking change policy.
Pre-launch Checklist
///).If you need help, consider asking for advice on the #hackers-new channel on Discord.