New issue
Advanced search Search tips

Issue 680644 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jan 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

[Remoting Android] Telemetry should report host OS & version

Project Member Reported by yuweih@chromium.org, Jan 12 2017

Issue description

Currently telemetry doesn't report host OS & host version, without which we can't use dashboard to filter Android stats by host.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Jan 13 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/c8bf2f02a0eca1ab4d9ad73dd376172f8f95c5b5

commit c8bf2f02a0eca1ab4d9ad73dd376172f8f95c5b5
Author: yuweih <yuweih@chromium.org>
Date: Fri Jan 13 03:50:53 2017

[Chromoting.com] Make Android telemetry report host version/os/os version

Previously the host telemetry logger doesn't report these information. This
CL fixes this problem so that we can use host info to filter stats.

BUG= 680644 

Review-Url: https://codereview.chromium.org/2629593003
Cr-Commit-Position: refs/heads/master@{#443481}

[modify] https://crrev.com/c8bf2f02a0eca1ab4d9ad73dd376172f8f95c5b5/remoting/android/java/src/org/chromium/chromoting/HostInfo.java
[modify] https://crrev.com/c8bf2f02a0eca1ab4d9ad73dd376172f8f95c5b5/remoting/android/java/src/org/chromium/chromoting/SessionConnector.java
[modify] https://crrev.com/c8bf2f02a0eca1ab4d9ad73dd376172f8f95c5b5/remoting/android/java/src/org/chromium/chromoting/jni/Client.java
[modify] https://crrev.com/c8bf2f02a0eca1ab4d9ad73dd376172f8f95c5b5/remoting/base/chromoting_event.cc
[modify] https://crrev.com/c8bf2f02a0eca1ab4d9ad73dd376172f8f95c5b5/remoting/base/chromoting_event.h
[modify] https://crrev.com/c8bf2f02a0eca1ab4d9ad73dd376172f8f95c5b5/remoting/client/client_telemetry_logger.cc
[modify] https://crrev.com/c8bf2f02a0eca1ab4d9ad73dd376172f8f95c5b5/remoting/client/client_telemetry_logger.h
[modify] https://crrev.com/c8bf2f02a0eca1ab4d9ad73dd376172f8f95c5b5/remoting/client/jni/chromoting_jni_instance.cc
[modify] https://crrev.com/c8bf2f02a0eca1ab4d9ad73dd376172f8f95c5b5/remoting/client/jni/chromoting_jni_instance.h
[modify] https://crrev.com/c8bf2f02a0eca1ab4d9ad73dd376172f8f95c5b5/remoting/client/jni/jni_client.cc
[modify] https://crrev.com/c8bf2f02a0eca1ab4d9ad73dd376172f8f95c5b5/remoting/client/jni/jni_client.h

Comment 2 by yuweih@chromium.org, Jan 13 2017

Status: Fixed (was: Started)

Sign in to add a comment