Do all the font UMA work in PdfMetafileSkia::FinishDocument.

Do all the font UMA work in PdfMetafileSkia::FinishDocument.

BUG=80918
TEST=NONE


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=87624

[Chris Guillory, 2011-05-31 18:15:45]

http://codereview.chromium.org/7057041/diff/1/printing/pdf_metafile_skia.cc
File printing/pdf_metafile_skia.cc (right):

http://codereview.chromium.org/7057041/diff/1/printing/pdf_metafile_skia.cc#n...
printing/pdf_metafile_skia.cc:100: SkFontID key =
font_resources[font]->typeface()->uniqueID();
We can call UMA_HISTOGRAM_ENUMERATION here when the hash_map doesn't contain the
key. Looks like we can also just use a simple set of font ids instead of a
hash_map.

[vandebo (ex-Chrome), 2011-05-31 18:55:11]

http://codereview.chromium.org/7057041/diff/1/printing/pdf_metafile_skia.cc
File printing/pdf_metafile_skia.cc (right):

http://codereview.chromium.org/7057041/diff/1/printing/pdf_metafile_skia.cc#n...
printing/pdf_metafile_skia.cc:100: SkFontID key =
font_resources[font]->typeface()->uniqueID();
On 2011/05/31 18:15:45, Chris Guillory wrote:
> We can call UMA_HISTOGRAM_ENUMERATION here when the hash_map doesn't contain
the
> key. Looks like we can also just use a simple set of font ids instead of a
> hash_map.

Good point. Done.

[Chris Guillory, 2011-05-31 18:57:29]

LGTM

[commit-bot: I haz the power, 2011-06-02 17:29:07]

Change committed as 87624