Fix for TypeError: _this.isCollectingLocalStats is not a function (it is undefined)

[Romick2005]

#2489 Move isCollectingLocalStats to the statistics. Mark JitsiMeetJS.isCollectingLocalStats as deprecated with warning to use Statistics.isCollectingLocalStats instead.

[jitsi-jenkins]

Hi, thanks for your contribution!
If you haven't already done so, could you please make sure you sign our CLA (https://jitsi.org/icla for individuals and https://jitsi.org/ccla for corporations)? We would unfortunately be unable to merge your patch unless we have that piece :(.

[saghul]

Why is this change necessary?

[Romick2005]

Why is this change necessary?

In order to fix #2489. Also it helps to organize code in more obvious way as now method that depends on statistics is located directly in Statistics file. Don't you think it will look more as cleaner code and as a bonus without an error?

[saghul]

No, because on UMD builds Statistics is not exposed, JitsiMeetJS is the only exposed object.

In addition, current working code does not have this problem so I suspect is has to do with the environment perhaps.

[Romick2005]

No, because on UMD builds Statistics is not exposed, JitsiMeetJS is the only exposed object.

In addition, current working code does not have this problem so I suspect is has to do with the environment perhaps.

Then we can just remove deprecated warning. And isCollectingLocalStats method would be exposed as before via JitsiMeetJS.isCollectingLocalStats.

Should I remove deprecation warning?

[Romick2005]

Done sir. Please review and merge please.