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

Issue 629313 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jul 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug

Blocking:
issue 629130



Sign in to add a comment

track SurfaceTexture vs SurfaceView frames from AVDA

Project Member Reported by liber...@chromium.org, Jul 18 2016

Issue description

track how many frames are sent from AVDA as allow_overlay or not.  this will let us track improvements over time.

 
Project Member

Comment 1 by bugdroid1@chromium.org, Jul 20 2016

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

commit ba0333059d47c4a9baa604d0def3ce2b3f6afee3
Author: liberato <liberato@chromium.org>
Date: Wed Jul 20 23:37:46 2016

Add FramesSentAsOverlay UMA metric to AVDA.

This metric records frames sent by AVDA, bucketed by whether they
are or are not overlays.

The goal is to provide an approximate count of how often SurfaceView
is used versus SurfaceTexture.

BUG= 629313 

Review-Url: https://codereview.chromium.org/2163513002
Cr-Commit-Position: refs/heads/master@{#406712}

[modify] https://crrev.com/ba0333059d47c4a9baa604d0def3ce2b3f6afee3/media/gpu/android_video_decode_accelerator.cc
[modify] https://crrev.com/ba0333059d47c4a9baa604d0def3ce2b3f6afee3/tools/metrics/histograms/histograms.xml

Labels: Merge-Request-53
requesting merge to m53 so we get more historical data.

Comment 3 by dimu@google.com, Jul 20 2016

Labels: -Merge-Request-53 Merge-Approved-53 Hotlist-Merge-Approved
Your change meets the bar and is auto-approved for M53 (branch: 2785)
Project Member

Comment 4 by bugdroid1@chromium.org, Jul 21 2016

Labels: -merge-approved-53 merge-merged-2785
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/64b719b4aa1328c47db531cc8438e4bee24d1637

commit 64b719b4aa1328c47db531cc8438e4bee24d1637
Author: liberato <liberato@chromium.org>
Date: Thu Jul 21 16:54:31 2016

[M53] Add FramesSentAsOverlay UMA metric to AVDA.

Merge to M53 of https://codereview.chromium.org/2163513002 .

This metric records frames sent by AVDA, bucketed by whether they
are or are not overlays.

The goal is to provide an approximate count of how often SurfaceView
is used versus SurfaceTexture.

BUG= 629313 
NOPRESUBMIT=true
NOTRY=true
TBR=watk@chromium.org,isherman@chromium.org

Review-Url: https://codereview.chromium.org/2166953003
Cr-Commit-Position: refs/branch-heads/2785@{#264}
Cr-Branched-From: 68623971be0cfc492a2cb0427d7f478e7b214c24-refs/heads/master@{#403382}

[modify] https://crrev.com/64b719b4aa1328c47db531cc8438e4bee24d1637/media/gpu/android_video_decode_accelerator.cc
[modify] https://crrev.com/64b719b4aa1328c47db531cc8438e4bee24d1637/tools/metrics/histograms/histograms.xml

Status: Fixed (was: Started)

Sign in to add a comment