New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 896861 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Nov 9
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Feature



Sign in to add a comment

Add metrics for AndroidSms components.

Project Member Reported by azeemarshad@chromium.org, Oct 18

Issue description

We should add metrics to track important parameters for Android Messages for Web integration.
 
Cc: jessejames@chromium.org elizabethchiu@chromium.org jordynass@chromium.org khorimoto@chromium.org shibasheikh@chromium.org shihuis@chromium.org nohle@chromium.org jhawkins@chromium.org hansberry@chromium.org
Project Member

Comment 2 by bugdroid1@chromium.org, Nov 6

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

commit 74873eda7b56409211c03f3c29bb7506c2e0a48f
Author: Azeem Arshad <azeemarshad@chromium.org>
Date: Tue Nov 06 05:55:42 2018

[AndroidSms] Add metrics for Service Worker lifetime and PWA install code.

Bug:  896861 
Change-Id: Ida470c2da886409de0864da4f3a4dfdca117e03e
Reviewed-on: https://chromium-review.googlesource.com/c/1289575
Reviewed-by: Kyle Horimoto <khorimoto@chromium.org>
Reviewed-by: Steven Holte <holte@chromium.org>
Reviewed-by: Alan Cutter <alancutter@chromium.org>
Reviewed-by: Ryan Hansberry <hansberry@chromium.org>
Commit-Queue: Azeem Arshad <azeemarshad@chromium.org>
Cr-Commit-Position: refs/heads/master@{#605606}
[modify] https://crrev.com/74873eda7b56409211c03f3c29bb7506c2e0a48f/chrome/browser/chromeos/android_sms/BUILD.gn
[modify] https://crrev.com/74873eda7b56409211c03f3c29bb7506c2e0a48f/chrome/browser/chromeos/android_sms/android_sms_service.cc
[modify] https://crrev.com/74873eda7b56409211c03f3c29bb7506c2e0a48f/chrome/browser/chromeos/android_sms/connection_establisher_impl.cc
[modify] https://crrev.com/74873eda7b56409211c03f3c29bb7506c2e0a48f/chrome/browser/chromeos/android_sms/connection_establisher_impl.h
[modify] https://crrev.com/74873eda7b56409211c03f3c29bb7506c2e0a48f/chrome/browser/chromeos/android_sms/connection_establisher_impl_unittest.cc
[modify] https://crrev.com/74873eda7b56409211c03f3c29bb7506c2e0a48f/chrome/browser/chromeos/multidevice_setup/android_sms_app_helper_delegate_impl.cc
[modify] https://crrev.com/74873eda7b56409211c03f3c29bb7506c2e0a48f/chrome/browser/chromeos/multidevice_setup/android_sms_app_helper_delegate_impl_unittest.cc
[modify] https://crrev.com/74873eda7b56409211c03f3c29bb7506c2e0a48f/chrome/browser/web_applications/components/web_app_constants.h
[modify] https://crrev.com/74873eda7b56409211c03f3c29bb7506c2e0a48f/tools/metrics/histograms/enums.xml
[modify] https://crrev.com/74873eda7b56409211c03f3c29bb7506c2e0a48f/tools/metrics/histograms/histograms.xml

Status: Fixed (was: Started)

Sign in to add a comment