📜 Description

• Most usages of synchronized keyword in method signatures and synchronized (...) { blocks have been replaced by using ReentrantLock
• Adds AutoClosableReentrantLock which offers an acquire method that locks and returns an AutoClosable that unlocks on close. This can be used in try blocks
• SynchronizedCollection and SynchronizedQueue now expect a AutoClosableReentrantLock instead of Object when passing a lock object
• Instance locks (previously synchronized methods and synchronized (this)) now use lock
• Static locks (previously static synchronized, synchronized (...class) and synchronized (...getInstance)) now use staticLock
• If there was a lock object before, the type has been changed from Object to AutoClosableReentrantLock
• Changing Android only classes wasn't totally necessary but IMO it's easier to maintain if there's only one way of synchronizing that's being used

💡 Motivation and Context

Fixes #3312

💚 How did you test it?

📝 Checklist

• I reviewed the submitted code.
• I added tests to verify the changes.
• No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled.
• I updated the docs if needed.
• Review from the native team if needed.
• No breaking change or entry added to the changelog.
• No breaking change for hybrid SDKs or communicated to hybrid SDKs.

🔮 Next steps