Add FramesSentAsOverlay UMA metric to AVDA.

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
Committed: https://crrev.com/ba0333059d47c4a9baa604d0def3ce2b3f6afee3
Cr-Commit-Position: refs/heads/master@{#406712}

[liberato (no reviews please), 2016-07-18 23:42:59]

Description was changed from

==========
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
==========

to

==========
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
==========

[liberato (no reviews please), 2016-07-18 23:43:00]

liberato@chromium.org changed reviewers:
+ watk@chromium.org

[watk, 2016-07-19 19:51:54]

lgtm

[liberato (no reviews please), 2016-07-19 20:51:32]

liberato@chromium.org changed reviewers:
+ isherman@chromium.org

[liberato (no reviews please), 2016-07-19 20:52:05]

isherman: PTAL @ histograms.xml
watk: thanks

thanks
-fl

[Ilya Sherman, 2016-07-20 02:17:04]

LGTM % a comment:

https://codereview.chromium.org/2163513002/diff/1/tools/metrics/histograms/hi...
File tools/metrics/histograms/histograms.xml (right):

https://codereview.chromium.org/2163513002/diff/1/tools/metrics/histograms/hi...
tools/metrics/histograms/histograms.xml:67490: +  <int value="1" label="Frame
was not allow_overlay"/>
These labels seem reversed from what the histogram description says.  Please
double-check to make sure that the boolean logic matches up.

[liberato (no reviews please), 2016-07-20 22:01:50]

indeed, you're right.  thanks!

-fl

[liberato (no reviews please), 2016-07-20 22:02:04]

The CQ bit was checked by liberato@chromium.org

[liberato (no reviews please), 2016-07-20 22:02:04]

The patchset sent to the CQ was uploaded after l-g-t-m from watk@chromium.org,
isherman@chromium.org
Link to the patchset: https://codereview.chromium.org/2163513002/#ps20001
(title: "fixed boolean description")

[commit-bot: I haz the power, 2016-07-20 22:03:19]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-20 23:38:01]

Description was changed from

==========
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
==========

to

==========
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
==========

[commit-bot: I haz the power, 2016-07-20 23:38:02]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-07-20 23:40:57]

Description was changed from

==========
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
==========

to

==========
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

Committed: https://crrev.com/ba0333059d47c4a9baa604d0def3ce2b3f6afee3
Cr-Commit-Position: refs/heads/master@{#406712}
==========

[commit-bot: I haz the power, 2016-07-20 23:40:59]

Patchset 2 (id:??) landed as
https://crrev.com/ba0333059d47c4a9baa604d0def3ce2b3f6afee3
Cr-Commit-Position: refs/heads/master@{#406712}