LPS-40873 Add skeleton for a few methods in WebRTCManager
LPS-40873 Our policy is to just include a person's name. If anyone searches online, they'll find you :)
LPS-40873 Empty constructor not needed
LPS-40873 Add line break
LPS-40873 Use protected in case somene wants to subclass it
LPS-40873 Ran "ant format-source" inside chat-video-portlet dir
LPS-40873 Method names should start with a verb where possible
LPS-40873 Ran "ant format-source" inside chat-video-portlet dir
LPS-40873 Use WebRTCClient vs. Client so things are not as confusing in case we have to deal with HttpClient or some other Client in the future
LPS-40873 Rename
LPS-40873 Implied...
LPS-40873 I think this is what you're really asking
LPS-40873 Implied
LPS-40873 What is a ConnectionState vs. a Presence? Do they have backing objects? This is the part that I got stumped on last time I was refactoring this… I saw multiple things that represented those 2 things. Should ConnectionState point to a WebRTCConnectionState enum class?  Removing until it's more obvious.
LPS-40873 Ran "ant format-source" inside chat-video-portlet dir