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

Issue 685615 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Android MediaRouter only (left Chro...
Closed: Jan 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

Implement metrics to track usage of MediaSession API

Project Member Reported by zqzh...@chromium.org, Jan 26 2017

Issue description

We should probably merge it to M57
 
Summary: Implement metrics to track usage of MediaSession API (was: Add rappor metrics for tracking the usage of MediaSession API)
In addition of rappor metrics, I think we could use histograms that record usage of the three properties of the MediaSession interface. We can easily do that with [Measure] in IDl.
Labels: Merge-Request-57
We need to merge the two CLs into M57
Project Member

Comment 5 by sheriffbot@chromium.org, Jan 30 2017

Labels: -Merge-Request-57 Hotlist-Merge-Approved Merge-Approved-57
Your change meets the bar and is auto-approved for M57. Please go ahead and merge the CL to branch 2987 manually. Please contact milestone owner if you have questions.
Owners: amineer@(clank), cmasso@(bling), ketakid@(cros), govind@(desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 6 by bugdroid1@chromium.org, Jan 30 2017

Labels: -merge-approved-57 merge-merged-2987
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/4bb23bb916db20688d6bfc5139ce883128cbdf1b

commit 4bb23bb916db20688d6bfc5139ce883128cbdf1b
Author: Zhiqiang Zhang <zqzhang@google.com>
Date: Mon Jan 30 20:28:58 2017

Add rappor metrics to track Media Session API usage

BUG= 685615 

Review-Url: https://codereview.chromium.org/2651203005
Cr-Commit-Position: refs/heads/master@{#446839}
(cherry picked from commit ba54980e209fedd167b84a09b7a075f4c7d5bbf5)

Review-Url: https://codereview.chromium.org/2665733003 .
Cr-Commit-Position: refs/branch-heads/2987@{#180}
Cr-Branched-From: ad51088c0e8776e8dcd963dbe752c4035ba6dab6-refs/heads/master@{#444943}

[modify] https://crrev.com/4bb23bb916db20688d6bfc5139ce883128cbdf1b/third_party/WebKit/Source/modules/mediasession/MediaSession.cpp
[modify] https://crrev.com/4bb23bb916db20688d6bfc5139ce883128cbdf1b/tools/metrics/rappor/rappor.xml

Status: Fixed (was: Assigned)

Sign in to add a comment