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

Unified Diff: components/data_reduction_proxy/core/browser/data_reduction_proxy_compression_stats_unittest.cc

Issue 2622183002: Remove clear-data-reduction-proxy-data-savings flag (Closed)
Patch Set: Addressed comments Created 3 years, 11 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/core/browser/data_reduction_proxy_compression_stats_unittest.cc
diff --git a/components/data_reduction_proxy/core/browser/data_reduction_proxy_compression_stats_unittest.cc b/components/data_reduction_proxy/core/browser/data_reduction_proxy_compression_stats_unittest.cc
index 35ec52fb31ec81fde244767f660da2db94e86f06..365e4b2f2d29bd6f80c07218f5cf7a473a62d8fe 100644
--- a/components/data_reduction_proxy/core/browser/data_reduction_proxy_compression_stats_unittest.cc
+++ b/components/data_reduction_proxy/core/browser/data_reduction_proxy_compression_stats_unittest.cc
@@ -10,7 +10,6 @@
#include <string>
#include <utility>
-#include "base/command_line.h"
#include "base/memory/ptr_util.h"
#include "base/memory/ref_counted.h"
#include "base/run_loop.h"
@@ -543,27 +542,7 @@ TEST_F(DataReductionProxyCompressionStatsTest,
VerifyPrefs(dict);
}
-TEST_F(DataReductionProxyCompressionStatsTest,
- ClearPrefsOnRestartEnabled) {
- base::CommandLine* command_line = base::CommandLine::ForCurrentProcess();
- command_line->AppendSwitch(
- data_reduction_proxy::switches::kClearDataReductionProxyDataSavings);
-
- base::ListValue list_value;
- list_value.Insert(
- 0, base::MakeUnique<base::StringValue>(base::Int64ToString(1234)));
- pref_service()->Set(prefs::kDailyHttpOriginalContentLength, list_value);
-
- ResetCompressionStatsWithDelay(
- base::TimeDelta::FromMinutes(kWriteDelayMinutes));
-
- const base::ListValue* value = pref_service()->GetList(
- prefs::kDailyHttpOriginalContentLength);
- EXPECT_EQ(0u, value->GetSize());
-}
-
-TEST_F(DataReductionProxyCompressionStatsTest,
- ClearPrefsOnRestartDisabled) {
+TEST_F(DataReductionProxyCompressionStatsTest, StatsRestoredOnOnRestart) {
base::ListValue list_value;
list_value.Insert(
0, base::MakeUnique<base::StringValue>(base::Int64ToString(1234)));

Powered by Google App Engine
This is Rietveld 408576698