Make SubscriptionController thread safe and some cleanup.

Use synchronized(mLock) on all public methods except getInstance which
is now synchronized on the class.

Change all members variables to private and non-static except sInstance
which will remain static.

Reduce logging in SubscriptionController.

Bug: 18716459
Change-Id: I679a7f184359ae8182c61308694354138a808b43