| Index: components/dom_distiller/core/distiller_page.cc
|
| diff --git a/components/dom_distiller/core/distiller_page.cc b/components/dom_distiller/core/distiller_page.cc
|
| index 94c37ae5c7857a9f9d69a34e6f959ff0a1141e72..1e70b83a2a51906811a7b6a879bd5a3ffb51644f 100644
|
| --- a/components/dom_distiller/core/distiller_page.cc
|
| +++ b/components/dom_distiller/core/distiller_page.cc
|
| @@ -84,35 +84,47 @@ void DistillerPage::OnDistillationDone(const GURL& page_url,
|
| value, distiller_result.get());
|
| if (!found_content) {
|
| DVLOG(1) << "Unable to parse DomDistillerResult.";
|
| - } else if (distiller_result->has_timing_info()) {
|
| - const dom_distiller::proto::TimingInfo& timing =
|
| - distiller_result->timing_info();
|
| - if (timing.has_markup_parsing_time()) {
|
| - UMA_HISTOGRAM_TIMES(
|
| - "DomDistiller.Time.MarkupParsing",
|
| - base::TimeDelta::FromMillisecondsD(timing.markup_parsing_time()));
|
| + } else {
|
| + if (distiller_result->has_timing_info()) {
|
| + const dom_distiller::proto::TimingInfo& timing =
|
| + distiller_result->timing_info();
|
| + if (timing.has_markup_parsing_time()) {
|
| + UMA_HISTOGRAM_TIMES(
|
| + "DomDistiller.Time.MarkupParsing",
|
| + base::TimeDelta::FromMillisecondsD(timing.markup_parsing_time()));
|
| + }
|
| + if (timing.has_document_construction_time()) {
|
| + UMA_HISTOGRAM_TIMES(
|
| + "DomDistiller.Time.DocumentConstruction",
|
| + base::TimeDelta::FromMillisecondsD(
|
| + timing.document_construction_time()));
|
| + }
|
| + if (timing.has_article_processing_time()) {
|
| + UMA_HISTOGRAM_TIMES(
|
| + "DomDistiller.Time.ArticleProcessing",
|
| + base::TimeDelta::FromMillisecondsD(
|
| + timing.article_processing_time()));
|
| + }
|
| + if (timing.has_formatting_time()) {
|
| + UMA_HISTOGRAM_TIMES(
|
| + "DomDistiller.Time.Formatting",
|
| + base::TimeDelta::FromMillisecondsD(timing.formatting_time()));
|
| + }
|
| + if (timing.has_total_time()) {
|
| + UMA_HISTOGRAM_TIMES(
|
| + "DomDistiller.Time.DistillationTotal",
|
| + base::TimeDelta::FromMillisecondsD(timing.total_time()));
|
| + }
|
| }
|
| - if (timing.has_document_construction_time()) {
|
| - UMA_HISTOGRAM_TIMES(
|
| - "DomDistiller.Time.DocumentConstruction",
|
| - base::TimeDelta::FromMillisecondsD(
|
| - timing.document_construction_time()));
|
| - }
|
| - if (timing.has_article_processing_time()) {
|
| - UMA_HISTOGRAM_TIMES(
|
| - "DomDistiller.Time.ArticleProcessing",
|
| - base::TimeDelta::FromMillisecondsD(
|
| - timing.article_processing_time()));
|
| - }
|
| - if (timing.has_formatting_time()) {
|
| - UMA_HISTOGRAM_TIMES(
|
| - "DomDistiller.Time.Formatting",
|
| - base::TimeDelta::FromMillisecondsD(timing.formatting_time()));
|
| - }
|
| - if (timing.has_total_time()) {
|
| - UMA_HISTOGRAM_TIMES(
|
| - "DomDistiller.Time.DistillationTotal",
|
| - base::TimeDelta::FromMillisecondsD(timing.total_time()));
|
| + if (distiller_result->has_statistics_info()) {
|
| + const dom_distiller::proto::StatisticsInfo& statistics =
|
| + distiller_result->statistics_info();
|
| + if (statistics.has_word_count()) {
|
| + UMA_HISTOGRAM_CUSTOM_COUNTS(
|
| + "DomDistiller.Statistics.WordCount",
|
| + statistics.word_count(),
|
| + 1, 4000, 50);
|
| + }
|
| }
|
| }
|
| }
|
|
|