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

Unified Diff: components/metrics/enabled_state_provider.cc

Issue 1958003003: Splitting the concept of UMA consent, and should UMA report. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 7 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « components/metrics/enabled_state_provider.h ('k') | components/metrics/metrics_service_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/metrics/enabled_state_provider.cc
diff --git a/ui/gfx/vector_icons/window_control_maximize.1x.icon b/components/metrics/enabled_state_provider.cc
similarity index 50%
copy from ui/gfx/vector_icons/window_control_maximize.1x.icon
copy to components/metrics/enabled_state_provider.cc
index f1c56bbd48d95762a1e6b4cf0ba8ce26f6bc9d4e..eb7b47a49c64b31202e48652e7e1a3994af9bb7c 100644
--- a/ui/gfx/vector_icons/window_control_maximize.1x.icon
+++ b/components/metrics/enabled_state_provider.cc
@@ -2,17 +2,12 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-CANVAS_DIMENSIONS, 12,
-MOVE_TO, 11, 1,
-R_V_LINE_TO, 10,
-H_LINE_TO, 1,
-V_LINE_TO, 1,
-R_H_LINE_TO, 10,
-CLOSE,
-MOVE_TO, 9, 9,
-H_LINE_TO, 3,
-V_LINE_TO, 3,
-R_H_LINE_TO, 6,
-R_V_LINE_TO, 6,
-CLOSE,
-END
+#include "components/metrics/enabled_state_provider.h"
+
+namespace metrics {
+
+bool EnabledStateProvider::IsReportingEnabled() {
+ return IsConsentGiven();
+}
+
+} // namespace metrics
« no previous file with comments | « components/metrics/enabled_state_provider.h ('k') | components/metrics/metrics_service_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698