tree 39063e178a49235f2cbddcc608794abb580c666b
parent ca9e2a41c18a60ddd290156059e733eb92cd5968
author Konstantin Shcheglov <scheglov@google.com> 1632862442 +0000
committer Alexander Thomas <athom@google.com> 1634116004 +0200

Issue 90868. Fix race condition in PluginManager.addPluginToContextRoot

1. We start stopping the plugin in removedContextRoot.
2. We add the plugin to the map in addPluginToContextRoot.
3. We receive the notification that the plugin stopped, and remove the
   plugin by its path from the map.

The issue is that the map has already been updated to contain the newly
started plugin, with the same path. As a result, we forget that we have
a plugin started, and never stop it.


Bug: https://github.com/flutter/flutter/issues/90868
Change-Id: I46c294c555905f0e9f298044718b281cb890e8ac
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/214862
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
Commit-Queue: Konstantin Shcheglov <scheglov@google.com>