Chromium Code Reviews Chromium Code Reviews
Issue 1612803002:
Test distillability on all pages (Closed)
Base URL: https://chromium.googlesource.com/chromium/src.git@mobile-distillable| Index: tools/metrics/histograms/histograms.xml |
| diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml |
| index 1c55a93923c93de769e0b2bb5304b37b306c1403..39718be245beeab89432fd29894ff86d8a0e491c 100644 |
| --- a/tools/metrics/histograms/histograms.xml |
| +++ b/tools/metrics/histograms/histograms.xml |
| @@ -8496,6 +8496,10 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries. |
| </histogram> |
| <histogram name="DomDistiller.PageDistillable" enum="DistillableType"> |
| + <obsolete> |
| + PageDistillableAfterLoading and PageDistillableAfterParsing provide more |
| + details. |
|
jwd
2016/02/09 19:18:58
nit: can you add the MM/YYYY it was obsoleted.
|
| + </obsolete> |
| <owner>cjhopman@chromium.org</owner> |
| <summary> |
| Records the "Distillable Type" (not distillable, mobile-friendly |
| @@ -8503,7 +8507,30 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries. |
| </summary> |
| </histogram> |
| +<histogram name="DomDistiller.PageDistillableAfterLoading" |
| + enum="DistillableType2"> |
| + <owner>wychen@chromium.org</owner> |
| + <summary> |
| + Records the "Distillable Type" (mobile-friendly not distillable, |
| + mobile-friendly distillable, non-mobile-friendly not distillable, |
| + non-mobile-friendly distillable) for each analyzed page after loading. |
| + </summary> |
| +</histogram> |
| + |
| +<histogram name="DomDistiller.PageDistillableAfterParsing" |
| + enum="DistillableType2"> |
| + <owner>wychen@chromium.org</owner> |
| + <summary> |
| + Records the "Distillable Type" (mobile-friendly not distillable, |
| + mobile-friendly distillable, non-mobile-friendly not distillable, |
| + non-mobile-friendly distillable) for each analyzed page after parsing. |
| + </summary> |
| +</histogram> |
| + |
| <histogram name="DomDistiller.PageDistilledType" enum="DistillableType"> |
| + <obsolete> |
| + Not collected anymore. |
| + </obsolete> |
| <owner>cjhopman@chromium.org</owner> |
| <summary> |
| Records the "Distillable Type" (not distillable, mobile-friendly |
| @@ -59766,10 +59793,17 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries. |
| <enum name="DistillableType" type="int"> |
| <int value="0" label="Not distillable"/> |
| - <int value="1" label="Non-mobile-friendly Distillable"/> |
| + <int value="1" label="Non-mobile-friendly distillable"/> |
| <int value="2" label="Mobile-friendly distillable"/> |
| </enum> |
| +<enum name="DistillableType2" type="int"> |
| + <int value="0" label="Non-mobile-friendly, not distillable"/> |
| + <int value="1" label="Mobile-friendly, not distillable"/> |
| + <int value="2" label="Non-mobile-friendly, distillable"/> |
| + <int value="3" label="Mobile-friendly, distillable"/> |
| +</enum> |
| + |
| <enum name="DllHash" type="int"> |
| <!-- Generated by chrome_elf/dll_hash/dll_hash_main.cc --> |
