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

Issue 703267 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

Record the type of media controls used for video persistence

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

Issue description

native vs custom
 

Comment 1 by dah...@chromium.org, Mar 20 2017

Sorry for my ignorance, but what is video persistence?
Re #2: Talked offline and resolved
Labels: Merge-Request-58
Project Member

Comment 5 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 6 by bugdroid1@chromium.org, Mar 22 2017

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

commit 1a50efc6d0e39da9a6fb4e806828fd89890851c9
Author: Mounir Lamouri <mlamouri@chromium.org>
Date: Wed Mar 22 16:55:41 2017

Record the type of controls used for video persistence.

BUG= 703267 
R=zqzhang@chromium.org

Review-Url: https://codereview.chromium.org/2763723002
Cr-Commit-Position: refs/heads/master@{#458387}
(cherry picked from commit 9bab651a8a170a6016af11dc0ee4f6407456fcb9)

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

[modify] https://crrev.com/1a50efc6d0e39da9a6fb4e806828fd89890851c9/third_party/WebKit/Source/core/html/HTMLMediaElement.cpp
[modify] https://crrev.com/1a50efc6d0e39da9a6fb4e806828fd89890851c9/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