Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1592)

Unified Diff: pdf/out_of_process_instance.h

Issue 2398763002: Add font substitution metric for Linux (Closed)
Patch Set: Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | pdf/out_of_process_instance.cc » ('j') | pdf/out_of_process_instance.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: pdf/out_of_process_instance.h
diff --git a/pdf/out_of_process_instance.h b/pdf/out_of_process_instance.h
index a998537dfd1e27548cf580d50715c68c928afb51..70fbfb58fd9b612fed17ba5c8897c72597d3f416 100644
--- a/pdf/out_of_process_instance.h
+++ b/pdf/out_of_process_instance.h
@@ -128,6 +128,7 @@ class OutOfProcessInstance : public pp::Instance,
void DocumentPaintOccurred() override;
void DocumentLoadComplete(int page_count) override;
void DocumentLoadFailed() override;
+ void FontSubstituted() override;
pp::Instance* GetPluginInstance() override;
void DocumentHasUnsupportedFeature(const std::string& feature) override;
void DocumentLoadProgress(uint32_t available, uint32_t doc_size) override;
@@ -309,6 +310,10 @@ class OutOfProcessInstance : public pp::Instance,
// the stats if a feature shows up many times per document.
std::set<std::string> unsupported_features_reported_;
+ // Keeps track of whether font substitution has been reported, so we avoid
+ // spamming the stats if a document requested multiple substitutes.
+ bool font_substitution_reported;
Lei Zhang 2016/10/05 20:48:09 members need a trailing underscore.
npm 2016/10/06 15:20:47 Done.
+
// Number of pages in print preview mode, 0 if not in print preview mode.
int print_preview_page_count_;
std::vector<int> print_preview_page_numbers_;
« no previous file with comments | « no previous file | pdf/out_of_process_instance.cc » ('j') | pdf/out_of_process_instance.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698