Add font failure telemetry

Add font failure telemetry

Adds some telemetry which might help us diagnose the cause
of issue 561873.

BUG=561873
Committed: https://crrev.com/9dde4efb888159a44ba56cdea254c15a984f69e3
Cr-Commit-Position: refs/heads/master@{#404064}

[Ilya Kulshin, 2016-06-30 18:21:26]

Description was changed from

==========
Add font failure telemetry


BUG=
==========

to

==========
Add font failure telemetry

Adds some telemetry which might help us diagnose the cause
of issue 561873.

BUG=561873
==========

[Ilya Kulshin, 2016-06-30 18:22:03]

kulshin@chromium.org changed reviewers:
+ ananta@chromium.org

[Ilya Kulshin, 2016-06-30 18:22:04]

ptal

[Ilya Kulshin, 2016-06-30 19:46:04]

kulshin@chromium.org changed reviewers:
+ asvitkine@chromium.org, rkaplow@chromium.org

[Ilya Kulshin, 2016-06-30 19:46:04]

+asvitkine,rkaplow - could you review histograms.xml?

[ananta, 2016-06-30 20:01:57]

lgtm

[rkaplow, 2016-07-04 14:45:02]

lgtm

https://codereview.chromium.org/2103293004/diff/20001/content/browser/rendere...
File content/browser/renderer_host/dwrite_font_proxy_message_filter_win.cc
(right):

https://codereview.chromium.org/2103293004/diff/20001/content/browser/rendere...
content/browser/renderer_host/dwrite_font_proxy_message_filter_win.cc:18:
#include "base/macros.h"
should be adding histogram_macros.h

https://codereview.chromium.org/2103293004/diff/20001/tools/metrics/histogram...
File tools/metrics/histograms/histograms.xml (left):

https://codereview.chromium.org/2103293004/diff/20001/tools/metrics/histogram...
tools/metrics/histograms/histograms.xml:8922: 
prefer more specific stuff for the unit - something like 'font families', 'font
files', etc.

[Ilya Kulshin, 2016-07-06 17:56:14]

https://codereview.chromium.org/2103293004/diff/20001/content/browser/rendere...
File content/browser/renderer_host/dwrite_font_proxy_message_filter_win.cc
(right):

https://codereview.chromium.org/2103293004/diff/20001/content/browser/rendere...
content/browser/renderer_host/dwrite_font_proxy_message_filter_win.cc:18:
#include "base/macros.h"
On 2016/07/04 14:45:01, rkaplow wrote:
> should be adding histogram_macros.h

histogram_macros.h is already #included (next line). macros.h is needed for
arraysize on line 396. Although this change still builds without macros.h, I'd
rather not depend on something else including macros.h.

https://codereview.chromium.org/2103293004/diff/20001/tools/metrics/histogram...
File tools/metrics/histograms/histograms.xml (left):

https://codereview.chromium.org/2103293004/diff/20001/tools/metrics/histogram...
tools/metrics/histograms/histograms.xml:8922: 
On 2016/07/04 14:45:01, rkaplow wrote:
> prefer more specific stuff for the unit - something like 'font families',
'font
> files', etc.

Done.

[Ilya Kulshin, 2016-07-06 17:56:18]

The CQ bit was checked by kulshin@chromium.org

[Ilya Kulshin, 2016-07-06 17:56:18]

The patchset sent to the CQ was uploaded after l-g-t-m from ananta@chromium.org,
rkaplow@chromium.org
Link to the patchset: https://codereview.chromium.org/2103293004/#ps40001
(title: "Change units in histograms")

[commit-bot: I haz the power, 2016-07-06 17:56:52]

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-06 20:17:59]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-07-06 20:18:00]

Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[Ilya Kulshin, 2016-07-07 02:55:39]

Patchset #4 (id:60001) has been deleted

[Ilya Kulshin, 2016-07-07 02:57:01]

The CQ bit was checked by kulshin@chromium.org

[Ilya Kulshin, 2016-07-07 02:57:01]

The patchset sent to the CQ was uploaded after l-g-t-m from ananta@chromium.org,
rkaplow@chromium.org
Link to the patchset: https://codereview.chromium.org/2103293004/#ps40001
(title: "Change units in histograms")

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

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-07 03:38:24]

Description was changed from

==========
Add font failure telemetry

Adds some telemetry which might help us diagnose the cause
of issue 561873.

BUG=561873
==========

to

==========
Add font failure telemetry

Adds some telemetry which might help us diagnose the cause
of issue 561873.

BUG=561873
==========

[commit-bot: I haz the power, 2016-07-07 03:38:25]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-07-07 03:39:54]

Description was changed from

==========
Add font failure telemetry

Adds some telemetry which might help us diagnose the cause
of issue 561873.

BUG=561873
==========

to

==========
Add font failure telemetry

Adds some telemetry which might help us diagnose the cause
of issue 561873.

BUG=561873

Committed: https://crrev.com/9dde4efb888159a44ba56cdea254c15a984f69e3
Cr-Commit-Position: refs/heads/master@{#404064}
==========

[commit-bot: I haz the power, 2016-07-07 03:39:55]

Patchset 3 (id:??) landed as
https://crrev.com/9dde4efb888159a44ba56cdea254c15a984f69e3
Cr-Commit-Position: refs/heads/master@{#404064}