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

Side by Side Diff: components/translate/core/browser/translate_browser_metrics.cc

Issue 794683005: replace COMPILE_ASSERT with static_assert in components/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: formatting fixup Created 5 years, 12 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 "components/translate/core/browser/translate_browser_metrics.h" 5 #include "components/translate/core/browser/translate_browser_metrics.h"
6 6
7 #include <string> 7 #include <string>
8 8
9 #include "base/basictypes.h" 9 #include "base/basictypes.h"
10 #include "base/metrics/histogram.h" 10 #include "base/metrics/histogram.h"
(...skipping 29 matching lines...) Expand all
40 { TranslateBrowserMetrics::UMA_LANGUAGE_DETECTION_ERROR, 40 { TranslateBrowserMetrics::UMA_LANGUAGE_DETECTION_ERROR,
41 kTranslateReportLanguageDetectionError }, 41 kTranslateReportLanguageDetectionError },
42 { TranslateBrowserMetrics::UMA_LOCALES_ON_DISABLED_BY_PREFS, 42 { TranslateBrowserMetrics::UMA_LOCALES_ON_DISABLED_BY_PREFS,
43 kTranslateLocalesOnDisabledByPrefs }, 43 kTranslateLocalesOnDisabledByPrefs },
44 { TranslateBrowserMetrics::UMA_UNDISPLAYABLE_LANGUAGE, 44 { TranslateBrowserMetrics::UMA_UNDISPLAYABLE_LANGUAGE,
45 kTranslateUndisplayableLanguage }, 45 kTranslateUndisplayableLanguage },
46 { TranslateBrowserMetrics::UMA_UNSUPPORTED_LANGUAGE_AT_INITIATION, 46 { TranslateBrowserMetrics::UMA_UNSUPPORTED_LANGUAGE_AT_INITIATION,
47 kTranslateUnsupportedLanguageAtInitiation }, 47 kTranslateUnsupportedLanguageAtInitiation },
48 }; 48 };
49 49
50 COMPILE_ASSERT(arraysize(kMetricsEntries) == TranslateBrowserMetrics::UMA_MAX, 50 static_assert(arraysize(kMetricsEntries) == TranslateBrowserMetrics::UMA_MAX,
51 arraysize_of_kMetricsEntries_should_be_UMA_MAX); 51 "kMetricsEntries should have UMA_MAX elements");
52 52
53 } // namespace 53 } // namespace
54 54
55 namespace TranslateBrowserMetrics { 55 namespace TranslateBrowserMetrics {
56 56
57 void ReportInitiationStatus(InitiationStatusType type) { 57 void ReportInitiationStatus(InitiationStatusType type) {
58 UMA_HISTOGRAM_ENUMERATION(kTranslateInitiationStatus, 58 UMA_HISTOGRAM_ENUMERATION(kTranslateInitiationStatus,
59 type, 59 type,
60 INITIATION_STATUS_MAX); 60 INITIATION_STATUS_MAX);
61 } 61 }
(...skipping 27 matching lines...) Expand all
89 if (kMetricsEntries[i].index == index) 89 if (kMetricsEntries[i].index == index)
90 return kMetricsEntries[i].name; 90 return kMetricsEntries[i].name;
91 } 91 }
92 NOTREACHED(); 92 NOTREACHED();
93 return NULL; 93 return NULL;
94 } 94 }
95 95
96 } // namespace TranslateBrowserMetrics 96 } // namespace TranslateBrowserMetrics
97 97
98 } // namespace translate 98 } // namespace translate
OLDNEW
« no previous file with comments | « components/sync_driver/model_association_manager.cc ('k') | components/translate/core/common/translate_metrics.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698