Need to report active users for iOS client |
||||||||
Issue descriptionCurrently we do not capture active user counts for the iOS clients because iOS uses the older telemetry API. We need to update, so that we can get an accurate user count for all platforms.
,
May 19 2016
Hey Scott, there is a class called ClientTelemetryLogger under remoting/client. You can reuse that for iOS. The interface is similar to the existing ClientStatusLogger and you can take a look at ChromotingJniRuntime::StartLoggerOnNetworkThread and some function calls in remoting/client/jni/chromoting_jni_instance.cc if you like. BTW just to mention that the OnConnectionState function of ClientUserInterface will not be called if disconnection is triggered on the client but it will be called if it is triggered on the host. You may need to modify the Disconnect() function to get the correct stat of disconnect events.
,
Jul 10 2016
Moving this nonessential bug to the next milestone. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Oct 10 2016
,
Jan 4 2017
,
Jan 27 2017
,
May 25 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/f9ad97d4a86c4820fb02d73fb28bcd34ec4e3428 commit f9ad97d4a86c4820fb02d73fb28bcd34ec4e3428 Author: yuweih <yuweih@chromium.org> Date: Thu May 25 22:58:18 2017 [CRD iOS] Fix telemetry issues This CL fixes these issues: * Previously the client OS is reported as Mac. That's because the OS_MACOSX macro is actually defined on iOS. * Added the HostOs/HostVersion/HostOsVersion into the telemetry pipeline. BUG= 610866 Review-Url: https://codereview.chromium.org/2910513002 Cr-Commit-Position: refs/heads/master@{#474835} [modify] https://crrev.com/f9ad97d4a86c4820fb02d73fb28bcd34ec4e3428/remoting/base/chromoting_event.cc [modify] https://crrev.com/f9ad97d4a86c4820fb02d73fb28bcd34ec4e3428/remoting/ios/domain/host_info.h [modify] https://crrev.com/f9ad97d4a86c4820fb02d73fb28bcd34ec4e3428/remoting/ios/domain/host_info.mm [modify] https://crrev.com/f9ad97d4a86c4820fb02d73fb28bcd34ec4e3428/remoting/ios/facade/host_info.cc [modify] https://crrev.com/f9ad97d4a86c4820fb02d73fb28bcd34ec4e3428/remoting/ios/facade/host_info.h [modify] https://crrev.com/f9ad97d4a86c4820fb02d73fb28bcd34ec4e3428/remoting/ios/facade/remoting_service.mm [modify] https://crrev.com/f9ad97d4a86c4820fb02d73fb28bcd34ec4e3428/remoting/ios/session/remoting_client.mm
,
Sep 18 2017
,
Sep 18 2017
,
Apr 20 2018
|
||||||||
►
Sign in to add a comment |
||||||||
Comment 1 by jamiewa...@chromium.org
, May 12 2016Owner: nicho...@chromium.org
Status: Assigned (was: Untriaged)