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

Unified Diff: tools/metrics/histograms/histograms.xml

Side-by-side diff isn't available for this file because of its large size.
Issue 1488133002: Add usage metrics for display and image color profiles (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Changed histograms to use suffixes Created 5 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
Download patch
Index: tools/metrics/histograms/histograms.xml
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index 524e9c03947469e6e8e3c5dc9350bb22b79dc6d8..b9eeebcc243c501eb5765b87b7a588338b7afe28 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -4679,6 +4679,14 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
<summary>Number of tries before successful ping. 99 means giving up.</summary>
</histogram>
+<histogram name="ColorManagement.ColorProfileStatus" enum="ColorProfileStatus">
+ <owner>junov@chromium.org</owner>
+ <owner>noel@chromium.org</owner>
+ <summary>
+ Success or error status of retrieving, decoding and using a color profile.
+ </summary>
+</histogram>
+
<histogram
name="Compositing.Browser.DisplayListRecordingSource.UpdateInvalidatedAreaPerMs"
units="pixels/ms">
@@ -56795,6 +56803,19 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
<int value="30" label="More Than 30"/>
</enum>
+<enum name="ColorProfileStatus" type="int">
+ <int value="0" label="Has no color profile"/>
Noel Gordon 2015/12/23 15:28:48 My renaming suggestions mean these will need updat
+ <int value="1" label="Expected a color profile, but none found (WEBP)"/>
+ <int value="2" label="Color profile header data is corrupted"/>
+ <int value="3" label="Unsupported: Non-RGB profile"/>
+ <int value="4" label="Unsupported: Not an input profile"/>
+ <int value="5" label="Unsupported: ICC v4 profile"/>
+ <int value="6" label="Color profile decode failed"/>
+ <int value="7" label="Color profile has incoherent parameters"/>
+ <int value="8" label="Color profile successfully applied"/>
+ <int value="9" label="PNG tagged as sRGB (no embedded profile)"/>
+</enum>
+
<enum name="CombinedHttpResponseAndNetErrorCode" type="int">
<!-- Generated from net/base/net_error_list.h -->
@@ -79000,6 +79021,14 @@ To add a new entry, add it with any value and run test to compute valid value.
<affected-histogram name="CloudPrint.UrlFetcherUploadSize"/>
</histogram_suffixes>
+<histogram_suffixes name="ColorProfileSource" separator=".">
+ <suffix name="Display" label="Color profiles from display devices."/>
rkaplow 2015/12/03 15:42:42 ColorProfiles to be consistant?
Noel Gordon 2015/12/23 15:28:48 Perhaps ... Display device color profile. JPEG im
+ <suffix name="JPEG" label="ColorProfiles from JPEG files"/>
+ <suffix name="PNG" label="ColorProfiles from PNG files"/>
+ <suffix name="WEBP" label="ColorProfiles from WEBP files"/>
+ <affected-histogram name="ColorManagement.ColorProfileStatus"/>
+</histogram_suffixes>
+
<histogram_suffixes name="CompositorTimingHistoryProcess" separator="."
ordering="prefix">
<suffix name="Browser"/>

Powered by Google App Engine
This is Rietveld 408576698