New issue
Advanced search Search tips

Issue 703742 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug



Sign in to add a comment

Add more metrics for VideoPersistence

Project Member Reported by mlamouri@chromium.org, Mar 21 2017

Issue description

Just adding some entry in XML file.
 
Labels: Merge-Request-58
Project Member

Comment 3 by sheriffbot@chromium.org, Mar 22 2017

Labels: -Merge-Request-58 Hotlist-Merge-Approved Merge-Approved-58
Your change meets the bar and is auto-approved for M58. Please go ahead and merge the CL to branch 3029 manually. Please contact milestone owner if you have questions.
Owners: amineer@(Android), cmasso@(iOS), bhthompson@(ChromeOS), govind@(Desktop)

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

Comment 4 by bugdroid1@chromium.org, Mar 23 2017

Labels: -merge-approved-58 merge-merged-3029
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/279a0d7e4223871099f5be77b06cd60ba9223ae6

commit 279a0d7e4223871099f5be77b06cd60ba9223ae6
Author: Mounir Lamouri <mlamouri@chromium.org>
Date: Thu Mar 23 14:14:45 2017

Add more metrics for VideoPersistence.

BUG= 703742 
R=isherman@chromium.org

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

Review-Url: https://codereview.chromium.org/2771043002 .
Cr-Commit-Position: refs/branch-heads/3029@{#384}
Cr-Branched-From: 939b32ee5ba05c396eef3fd992822fcca9a2e262-refs/heads/master@{#454471}

[modify] https://crrev.com/279a0d7e4223871099f5be77b06cd60ba9223ae6/tools/metrics/histograms/histograms.xml
[modify] https://crrev.com/279a0d7e4223871099f5be77b06cd60ba9223ae6/tools/metrics/rappor/rappor.xml

This follow-up CL has landed yesterday: https://codereview.chromium.org/2769723002

I'm merging in to m58 too.
Project Member

Comment 6 by bugdroid1@chromium.org, Mar 23 2017

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

commit 7a1436f14727c7e141468579e879b3a2c96038f7
Author: Mounir Lamouri <mlamouri@chromium.org>
Date: Thu Mar 23 14:17:24 2017

Add new entry in VideoPersistenceEndReason.

BUG= 703742 
R=isherman@chromium.org

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

Review-Url: https://codereview.chromium.org/2774433003 .
Cr-Commit-Position: refs/branch-heads/3029@{#385}
Cr-Branched-From: 939b32ee5ba05c396eef3fd992822fcca9a2e262-refs/heads/master@{#454471}

[modify] https://crrev.com/7a1436f14727c7e141468579e879b3a2c96038f7/tools/metrics/histograms/histograms.xml

Status: Fixed (was: Started)

Comment 8 by dah...@chromium.org, Mar 29 2017

Blocking: -679364

Sign in to add a comment