From dcc60711b9c1e8358cc6c961db4e6b5cf05e1472 Mon Sep 17 00:00:00 2001 From: "vandebo@chromium.org" Date: Mon, 30 May 2011 17:36:07 +0000 Subject: Add metrics to determine what font types are being used with print preview. This depends on Skia r1444. BUG=80918 TEST=NONE Review URL: http://codereview.chromium.org/7084021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@87242 0039d316-1c4b-4281-b951-d872f2087c98 --- printing/pdf_metafile_skia.cc | 27 +++++++++++++++++++++++++++ 1 file changed, 27 insertions(+) (limited to 'printing/pdf_metafile_skia.cc') diff --git a/printing/pdf_metafile_skia.cc b/printing/pdf_metafile_skia.cc index f76eb70..2558a84 100644 --- a/printing/pdf_metafile_skia.cc +++ b/printing/pdf_metafile_skia.cc @@ -7,23 +7,33 @@ #include "base/eintr_wrapper.h" #include "base/file_descriptor_posix.h" #include "base/file_util.h" +#include "base/hash_tables.h" +#include "base/metrics/histogram.h" #include "skia/ext/vector_platform_device_skia.h" #include "third_party/skia/include/core/SkRefCnt.h" #include "third_party/skia/include/core/SkScalar.h" #include "third_party/skia/include/core/SkStream.h" +#include "third_party/skia/include/core/SkTypeface.h" #include "third_party/skia/include/pdf/SkPDFDevice.h" #include "third_party/skia/include/pdf/SkPDFDocument.h" +#include "third_party/skia/include/pdf/SkPDFFont.h" #include "third_party/skia/include/pdf/SkPDFPage.h" #include "ui/gfx/point.h" #include "ui/gfx/rect.h" #include "ui/gfx/size.h" +namespace { + typedef base::hash_map + FontTypeMap; +}; + namespace printing { struct PdfMetafileSkiaData { SkRefPtr current_page_; SkPDFDocument pdf_doc_; SkDynamicMemoryWStream pdf_stream_; + FontTypeMap font_type_stats_; }; PdfMetafileSkia::~PdfMetafileSkia() {} @@ -71,6 +81,13 @@ bool PdfMetafileSkia::StartPage(const gfx::Size& page_size, bool PdfMetafileSkia::FinishPage() { DCHECK(data_->current_page_.get()); + const SkTDArray& font_resources = + data_->current_page_->getFontResources(); + for (int i = 0; i < font_resources.count(); i++) { + SkFontID key = font_resources[i]->typeface()->uniqueID(); + data_->font_type_stats_[key] = font_resources[i]->getType(); + } + data_->pdf_doc_.appendPage(data_->current_page_); data_->current_page_ = NULL; return true; @@ -83,6 +100,16 @@ bool PdfMetafileSkia::FinishDocument() { if (data_->current_page_.get()) FinishPage(); + + for (FontTypeMap::const_iterator it = data_->font_type_stats_.begin(); + it != data_->font_type_stats_.end(); + it++) { + UMA_HISTOGRAM_ENUMERATION( + "PrintPreview.FontType", + it->second, + SkAdvancedTypefaceMetrics::kNotEmbeddable_Font + 1); + } + return data_->pdf_doc_.emitPDF(&data_->pdf_stream_); } -- cgit v1.1