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

Unified Diff: components/data_reduction_proxy/browser/data_reduction_proxy_metrics_unittest.cc

Issue 274453011: Revert of Collect data reduction proxy UMA on all platforms (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 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
Index: components/data_reduction_proxy/browser/data_reduction_proxy_metrics_unittest.cc
diff --git a/components/data_reduction_proxy/browser/data_reduction_proxy_metrics_unittest.cc b/components/data_reduction_proxy/browser/data_reduction_proxy_metrics_unittest.cc
index f1139776adbb3ec45f7c89c6a2cad632b9a4ffe4..f5e24f3ed002d455deede08c50ed186a15771bdd 100644
--- a/components/data_reduction_proxy/browser/data_reduction_proxy_metrics_unittest.cc
+++ b/components/data_reduction_proxy/browser/data_reduction_proxy_metrics_unittest.cc
@@ -15,6 +15,7 @@
namespace {
+#if defined(OS_ANDROID) || defined(OS_IOS)
const size_t kNumDaysInHistory = 60;
int64 GetListPrefInt64Value(
@@ -26,6 +27,8 @@
EXPECT_TRUE(base::StringToInt64(string_value, &value));
return value;
}
+
+#endif // defined(OS_ANDROID) || defined(OS_IOS)
} // namespace
@@ -43,6 +46,7 @@
registry->RegisterInt64Pref(
data_reduction_proxy::prefs::kHttpOriginalContentLength, 0);
+#if defined(OS_ANDROID) || defined(OS_IOS)
registry->RegisterListPref(data_reduction_proxy::prefs::
kDailyHttpOriginalContentLength);
registry->RegisterListPref(data_reduction_proxy::prefs::
@@ -74,6 +78,7 @@
registry->RegisterInt64Pref(
data_reduction_proxy::prefs::
kDailyHttpContentLengthLastUpdateDate, 0L);
+#endif // defined(OS_ANDROID) || defined(OS_IOS)
}
TestingPrefServiceSimple pref_service_;
@@ -104,6 +109,8 @@
pref_service_.GetInt64(
data_reduction_proxy::prefs::kHttpOriginalContentLength));
}
+
+#if defined(OS_ANDROID) || defined(OS_IOS)
// The initial last update time used in test. There is no leap second a few
// days around this time used in the test.
@@ -672,5 +679,6 @@
original, 1, received, 1,
original, 1, received, 1);
}
+#endif // defined(OS_ANDROID) || defined(OS_IOS)
} // namespace data_reduction_proxy

Powered by Google App Engine
This is Rietveld 408576698