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

Side by Side Diff: third_party/WebKit/Source/platform/graphics/BitmapImageMetrics.cpp

Issue 2190653004: Add more detail to Dst Color Space UMA (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@histogram2
Patch Set: Use a switch Created 4 years, 4 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "platform/graphics/BitmapImageMetrics.h" 5 #include "platform/graphics/BitmapImageMetrics.h"
6 6
7 #include "platform/Histogram.h" 7 #include "platform/Histogram.h"
8 #include "wtf/Threading.h" 8 #include "wtf/Threading.h"
9 #include "wtf/text/WTFString.h" 9 #include "wtf/text/WTFString.h"
10 10
(...skipping 18 matching lines...) Expand all
29 DEFINE_THREAD_SAFE_STATIC_LOCAL(EnumerationHistogram, orientationHistogram, new EnumerationHistogram("Blink.DecodedImage.Orientation", ImageOrientationEnumE nd)); 29 DEFINE_THREAD_SAFE_STATIC_LOCAL(EnumerationHistogram, orientationHistogram, new EnumerationHistogram("Blink.DecodedImage.Orientation", ImageOrientationEnumE nd));
30 orientationHistogram.count(orientation); 30 orientationHistogram.count(orientation);
31 } 31 }
32 32
33 void BitmapImageMetrics::countGamma(SkColorSpace* colorSpace) 33 void BitmapImageMetrics::countGamma(SkColorSpace* colorSpace)
34 { 34 {
35 DEFINE_THREAD_SAFE_STATIC_LOCAL(EnumerationHistogram, gammaNamedHistogram, n ew EnumerationHistogram("Blink.ColorSpace.Destination", GammaEnd)); 35 DEFINE_THREAD_SAFE_STATIC_LOCAL(EnumerationHistogram, gammaNamedHistogram, n ew EnumerationHistogram("Blink.ColorSpace.Destination", GammaEnd));
36 36
37 if (colorSpace) { 37 if (colorSpace) {
38 SkColorSpace::GammaNamed skGamma = colorSpace->gammaNamed(); 38 SkColorSpace::GammaNamed skGamma = colorSpace->gammaNamed();
39 Gamma gamma = 39
40 (SkColorSpace::kLinear_GammaNamed == skGamma) ? GammaLinear : 40 Gamma gamma;
41 (SkColorSpace::kSRGB_GammaNamed == skGamma) ? GammaSRGB : 41 switch (skGamma) {
42 (SkColorSpace::k2Dot2Curve_GammaNamed == skGamma) ? Gamma2Dot2 : 42 case SkColorSpace::kLinear_GammaNamed:
43 (SkColorSpace::kNonStandard_GammaNamed == skGamma) ? GammaNonStandar d : 43 gamma = GammaLinear;
44 GammaFail; 44 break;
45 case SkColorSpace::kSRGB_GammaNamed:
46 gamma = GammaSRGB;
47 break;
48 case SkColorSpace::k2Dot2Curve_GammaNamed:
49 gamma = Gamma2Dot2;
50 break;
51 case SkColorSpace::kInvalid_GammaNamed:
52 gamma = GammaInvalid;
53 break;
54 default:
55 if (colorSpace->gammasAreMatching()) {
56 if (colorSpace->gammasAreValues()) {
57 gamma = GammaExponent;
58 } else if (colorSpace->gammasAreParams()) {
59 gamma = GammaParametric;
60 } else if (colorSpace->gammasAreTables()) {
61 gamma = GammaTable;
62 } else if (colorSpace->gammasAreNamed()) {
63 gamma = GammaNamed;
64 } else {
65 gamma = GammaFail;
66 }
67 } else {
68 gamma = GammaNonStandard;
69 }
70 break;
71 }
72
45 gammaNamedHistogram.count(gamma); 73 gammaNamedHistogram.count(gamma);
46 } else { 74 } else {
47 gammaNamedHistogram.count(GammaNull); 75 gammaNamedHistogram.count(GammaNull);
48 } 76 }
49 } 77 }
50 78
51 } // namespace blink 79 } // namespace blink
OLDNEW
« no previous file with comments | « third_party/WebKit/Source/platform/graphics/BitmapImageMetrics.h ('k') | tools/metrics/histograms/histograms.xml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698