New issue
Advanced search Search tips

Issue 840556 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Aug 10
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android , iOS
Pri: 3
Type: Bug



Sign in to add a comment

[CRD Remoting] Add SessionEntryPoint to telemetry logs

Project Member Reported by yuweih@chromium.org, May 7 2018

Issue description

This allows us to compare metrics on initial vs reconnect attempt, and reuse error breakdown dashboards without unchecking the SessionEntryPoint filter.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Jul 20

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

commit 6bb3d58456f03944dff92f01d812ad71cac795ed
Author: Yuwei Huang <yuweih@chromium.org>
Date: Fri Jul 20 19:35:02 2018

[CRD iOS] Report session entry point in telemetry

This CL adds the session entry point info to the telemetry reported by
the iOS client. This allows us to compare metrics on initial vs reconnect
attempt, and reuse error breakdown dashboard without unchecking the
SessionEntryPoint filter.

Bug:  840556 
Change-Id: I27b5fd644f024b10953b5bbf919047d8f370b753
Reviewed-on: https://chromium-review.googlesource.com/1141147
Reviewed-by: Jamie Walch <jamiewalch@chromium.org>
Commit-Queue: Yuwei Huang <yuweih@chromium.org>
Cr-Commit-Position: refs/heads/master@{#576952}
[modify] https://crrev.com/6bb3d58456f03944dff92f01d812ad71cac795ed/remoting/base/chromoting_event.cc
[modify] https://crrev.com/6bb3d58456f03944dff92f01d812ad71cac795ed/remoting/base/chromoting_event.h
[modify] https://crrev.com/6bb3d58456f03944dff92f01d812ad71cac795ed/remoting/client/chromoting_session.cc
[modify] https://crrev.com/6bb3d58456f03944dff92f01d812ad71cac795ed/remoting/client/client_telemetry_logger.cc
[modify] https://crrev.com/6bb3d58456f03944dff92f01d812ad71cac795ed/remoting/client/client_telemetry_logger.h
[modify] https://crrev.com/6bb3d58456f03944dff92f01d812ad71cac795ed/remoting/client/client_telemetry_logger_unittest.cc
[modify] https://crrev.com/6bb3d58456f03944dff92f01d812ad71cac795ed/remoting/client/connect_to_host_info.h
[modify] https://crrev.com/6bb3d58456f03944dff92f01d812ad71cac795ed/remoting/ios/app/client_connection_view_controller.mm
[modify] https://crrev.com/6bb3d58456f03944dff92f01d812ad71cac795ed/remoting/ios/session/remoting_client.h
[modify] https://crrev.com/6bb3d58456f03944dff92f01d812ad71cac795ed/remoting/ios/session/remoting_client.mm

Status: Fixed (was: Assigned)

Sign in to add a comment