SharedEffect is not thread safe

[dlamkins]

SharedEffect uses a collection type which is not thread safe and does not lock when iterated or when adding items. This can cause an incredibly rare crash during launch made more likely as more shared effects are added.