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

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

Issue 266033002: Collect data reduction proxy UMA on all platforms (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Remove ifdef from HttpResponseHeaders::IsDataReductionProxyResponse() 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 f5e24f3ed002d455deede08c50ed186a15771bdd..f1139776adbb3ec45f7c89c6a2cad632b9a4ffe4 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,7 +15,6 @@
namespace {
-#if defined(OS_ANDROID) || defined(OS_IOS)
const size_t kNumDaysInHistory = 60;
int64 GetListPrefInt64Value(
@@ -28,8 +27,6 @@ int64 GetListPrefInt64Value(
return value;
}
-#endif // defined(OS_ANDROID) || defined(OS_IOS)
-
} // namespace
namespace data_reduction_proxy {
@@ -46,7 +43,6 @@ class ChromeNetworkDataSavingMetricsTest : public testing::Test {
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::
@@ -78,7 +74,6 @@ class ChromeNetworkDataSavingMetricsTest : public testing::Test {
registry->RegisterInt64Pref(
data_reduction_proxy::prefs::
kDailyHttpContentLengthLastUpdateDate, 0L);
-#endif // defined(OS_ANDROID) || defined(OS_IOS)
}
TestingPrefServiceSimple pref_service_;
@@ -110,8 +105,6 @@ TEST_F(ChromeNetworkDataSavingMetricsTest, TotalLengths) {
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.
// Note: No time zone is specified. Local time will be assumed by
@@ -679,6 +672,5 @@ TEST_F(ChromeNetworkDailyDataSavingMetricsTest, BackwardTwoDays) {
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