mozilla-mobile / prox

[INACTIVE] A search and discovery app for the "here & now". We're experimenting with ideas on mobile that can better surface content from the open web.
https://wiki.mozilla.org/New_Mobile_Experience
Mozilla Public License 2.0
22 stars 12 forks source link

Remove unused sessions, correct loading and permissions #485

Closed liuche closed 7 years ago

liuche commented 7 years ago

This fixes the biggest problem with session measuring where we were collecting data even if the app isn't visible. This shouldn't be a problem (while notifications are disabled) but this fixes differentiating between background sessions (testing with the Xcode "background fetch" scheme) and user-initiated sessions.

This also adds measurements for permissions, and also moved the analytics collection into user-initiated actions rather than including background fetches.

(I initially thought that this would require setting listeners for viewDidLoad but I had misconceptions about how granular iOS view management was, so that ended up not being relevant.)