Skip to content

Ensure Chrome is closed on tab close#43586

Merged
jonahwilliams merged 2 commits intoflutter:masterfrom
jonahwilliams:fix_tab_close
Oct 28, 2019
Merged

Ensure Chrome is closed on tab close#43586
jonahwilliams merged 2 commits intoflutter:masterfrom
jonahwilliams:fix_tab_close

Conversation

@jonahwilliams
Copy link
Contributor

Description

If the browser exit is initiated by the browser, we don't properly clean up chrome. Ensure we use the exit notification to use the existing cleanup mechanism.

Fixes #43486

@fluttergithubbot fluttergithubbot added tool Affects the "flutter" command-line tool. See also t: labels. work in progress; do not review labels Oct 27, 2019
@jonahwilliams jonahwilliams changed the title [WIP] Ensure Chrome is closed on tab close Ensure Chrome is closed on tab close Oct 27, 2019
@jonahwilliams jonahwilliams marked this pull request as ready for review October 27, 2019 20:07
@jonahwilliams jonahwilliams requested a review from yjbanov October 28, 2019 15:23
@jonahwilliams jonahwilliams merged commit d983c1d into flutter:master Oct 28, 2019
@jonahwilliams jonahwilliams deleted the fix_tab_close branch October 28, 2019 17:26
Inconnu08 pushed a commit to Inconnu08/flutter that referenced this pull request Nov 26, 2019
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 3, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

tool Affects the "flutter" command-line tool. See also t: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Flutter web does not close Chrome properly on quit.

4 participants