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

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

Issue 949533004: Rename DataReductionProxyStatisticsPrefs to DataReductionProxyCompressionStats (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 5 years, 9 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_statistics_prefs_unittest.cc
diff --git a/components/data_reduction_proxy/core/browser/data_reduction_proxy_statistics_prefs_unittest.cc b/components/data_reduction_proxy/core/browser/data_reduction_proxy_statistics_prefs_unittest.cc
deleted file mode 100644
index 004e96b593a533246ae69f5ae7eb9a7e574b691f..0000000000000000000000000000000000000000
--- a/components/data_reduction_proxy/core/browser/data_reduction_proxy_statistics_prefs_unittest.cc
+++ /dev/null
@@ -1,264 +0,0 @@
-// Copyright 2014 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "base/command_line.h"
-#include "base/memory/scoped_ptr.h"
-#include "base/prefs/pref_registry_simple.h"
-#include "base/prefs/testing_pref_service.h"
-#include "base/strings/string_number_conversions.h"
-#include "base/test/test_simple_task_runner.h"
-#include "base/time/time.h"
-#include "base/values.h"
-#include "components/data_reduction_proxy/core/browser/data_reduction_proxy_prefs.h"
-#include "components/data_reduction_proxy/core/browser/data_reduction_proxy_statistics_prefs.h"
-#include "components/data_reduction_proxy/core/common/data_reduction_proxy_pref_names.h"
-#include "components/data_reduction_proxy/core/common/data_reduction_proxy_switches.h"
-#include "testing/gtest/include/gtest/gtest.h"
-
-namespace {
-
-// TODO Make kNumDaysInHistory accessible from DataReductionProxySettings.
-const size_t kNumDaysInHistory = 60;
-const int kWriteDelayMinutes = 60;
-
-int64 GetListPrefInt64Value(
- const base::ListValue& list_update, size_t index) {
- std::string string_value;
- EXPECT_TRUE(list_update.GetString(index, &string_value));
-
- int64 value = 0;
- EXPECT_TRUE(base::StringToInt64(string_value, &value));
- return value;
-}
-
-} // namespace
-
-namespace data_reduction_proxy {
-
-class DataReductionProxyStatisticsPrefsTest : public testing::Test {
- protected:
- DataReductionProxyStatisticsPrefsTest()
- : task_runner_(scoped_refptr<base::TestSimpleTaskRunner>(
- new base::TestSimpleTaskRunner())) {}
-
- void SetUp() override { RegisterPrefs(simple_pref_service_.registry()); }
-
- void SetUpPrefs() {
- CreatePrefList(prefs::kDailyHttpOriginalContentLength);
- CreatePrefList(prefs::kDailyHttpReceivedContentLength);
-
- const int64 kOriginalLength = 150;
- const int64 kReceivedLength = 100;
-
- statistics_prefs_->SetInt64(
- prefs::kHttpOriginalContentLength, kOriginalLength);
- statistics_prefs_->SetInt64(
- prefs::kHttpReceivedContentLength, kReceivedLength);
-
- base::ListValue* original_daily_content_length_list =
- statistics_prefs_->GetList(prefs::kDailyHttpOriginalContentLength);
- base::ListValue* received_daily_content_length_list =
- statistics_prefs_->GetList(prefs::kDailyHttpReceivedContentLength);
-
- for (size_t i = 0; i < kNumDaysInHistory; ++i) {
- original_daily_content_length_list->Set(
- i, new base::StringValue(base::Int64ToString(i)));
- }
-
- received_daily_content_length_list->Clear();
- for (size_t i = 0; i < kNumDaysInHistory/2; ++i) {
- received_daily_content_length_list->Set(
- i, new base::StringValue(base::Int64ToString(i)));
- }
- }
-
- // Create daily pref list of |kNumDaysInHistory| zero values.
- void CreatePrefList(const char* pref) {
- base::ListValue* update = statistics_prefs_->GetList(pref);
- update->Clear();
- for (size_t i = 0; i < kNumDaysInHistory; ++i) {
- update->Insert(0, new base::StringValue(base::Int64ToString(0)));
- }
- }
-
- // Verify the pref list values in |pref_service_| are equal to those in
- // |simple_pref_service| for |pref|.
- void VerifyPrefListWasWritten(const char* pref) {
- const base::ListValue* delayed_list = statistics_prefs_->GetList(pref);
- const base::ListValue* written_list = simple_pref_service_.GetList(pref);
- ASSERT_EQ(delayed_list->GetSize(), written_list->GetSize());
- size_t count = delayed_list->GetSize();
-
- for (size_t i = 0; i < count; ++i) {
- EXPECT_EQ(GetListPrefInt64Value(*delayed_list, i),
- GetListPrefInt64Value(*written_list, i));
- }
- }
-
- // Verify the pref value in |pref_service_| are equal to that in
- // |simple_pref_service|.
- void VerifyPrefWasWritten(const char* pref) {
- int64 delayed_pref = statistics_prefs_->GetInt64(pref);
- int64 written_pref = simple_pref_service_.GetInt64(pref);
- EXPECT_EQ(delayed_pref, written_pref);
- }
-
- // Verify the pref values in |dict| are equal to that in |statistics_prefs_|.
- void VerifyPrefs(base::DictionaryValue* dict) {
- base::string16 dict_pref_string;
- int64 dict_pref;
- int64 service_pref;
-
- dict->GetString("historic_original_content_length", &dict_pref_string);
- base::StringToInt64(dict_pref_string, &dict_pref);
- service_pref = statistics_prefs_->GetInt64(
- prefs::kHttpOriginalContentLength);
- EXPECT_EQ(service_pref, dict_pref);
-
- dict->GetString("historic_received_content_length", &dict_pref_string);
- base::StringToInt64(dict_pref_string, &dict_pref);
- service_pref = statistics_prefs_->GetInt64(
- prefs::kHttpReceivedContentLength);
- EXPECT_EQ(service_pref, dict_pref);
- }
-
- scoped_refptr<base::TestSimpleTaskRunner> task_runner_;
- TestingPrefServiceSimple simple_pref_service_;
- scoped_ptr<DataReductionProxyStatisticsPrefs> statistics_prefs_;
-};
-
-TEST_F(DataReductionProxyStatisticsPrefsTest, WritePrefsDirect) {
- statistics_prefs_.reset(new DataReductionProxyStatisticsPrefs(
- &simple_pref_service_,
- task_runner_,
- base::TimeDelta()));
- SetUpPrefs();
-
- VerifyPrefWasWritten(prefs::kHttpOriginalContentLength);
- VerifyPrefWasWritten(prefs::kHttpReceivedContentLength);
- VerifyPrefListWasWritten(prefs::kDailyHttpOriginalContentLength);
- VerifyPrefListWasWritten(prefs::kDailyHttpReceivedContentLength);
-}
-
-TEST_F(DataReductionProxyStatisticsPrefsTest, WritePrefsDelayed) {
- statistics_prefs_.reset(new DataReductionProxyStatisticsPrefs(
- &simple_pref_service_,
- task_runner_,
- base::TimeDelta::FromMinutes(kWriteDelayMinutes)));
- SetUpPrefs();
-
- task_runner_->RunPendingTasks();
-
- VerifyPrefWasWritten(prefs::kHttpOriginalContentLength);
- VerifyPrefWasWritten(prefs::kHttpReceivedContentLength);
- VerifyPrefListWasWritten(prefs::kDailyHttpOriginalContentLength);
- VerifyPrefListWasWritten(prefs::kDailyHttpReceivedContentLength);
-}
-
-TEST_F(DataReductionProxyStatisticsPrefsTest,
- WritePrefsOnUpdateDailyReceivedContentLengths) {
- statistics_prefs_.reset(new DataReductionProxyStatisticsPrefs(
- &simple_pref_service_,
- task_runner_,
- base::TimeDelta::FromMinutes(kWriteDelayMinutes)));
- SetUpPrefs();
-
- simple_pref_service_.SetBoolean(
- prefs::kUpdateDailyReceivedContentLengths, true);
-
- VerifyPrefWasWritten(prefs::kHttpOriginalContentLength);
- VerifyPrefWasWritten(prefs::kHttpReceivedContentLength);
- VerifyPrefListWasWritten(prefs::kDailyHttpOriginalContentLength);
- VerifyPrefListWasWritten(prefs::kDailyHttpReceivedContentLength);
-}
-
-TEST_F(DataReductionProxyStatisticsPrefsTest,
- HistoricNetworkStatsInfoToValue) {
- const int64 kOriginalLength = 150;
- const int64 kReceivedLength = 100;
- statistics_prefs_.reset(new DataReductionProxyStatisticsPrefs(
- &simple_pref_service_,
- task_runner_,
- base::TimeDelta::FromMinutes(kWriteDelayMinutes)));
-
- base::DictionaryValue* dict = nullptr;
- scoped_ptr<base::Value> stats_value(
- statistics_prefs_->HistoricNetworkStatsInfoToValue());
- EXPECT_TRUE(stats_value->GetAsDictionary(&dict));
- VerifyPrefs(dict);
-
- statistics_prefs_->SetInt64(prefs::kHttpOriginalContentLength,
- kOriginalLength);
- statistics_prefs_->SetInt64(prefs::kHttpReceivedContentLength,
- kReceivedLength);
-
- stats_value.reset(statistics_prefs_->HistoricNetworkStatsInfoToValue());
- EXPECT_TRUE(stats_value->GetAsDictionary(&dict));
- VerifyPrefs(dict);
-}
-
-TEST_F(DataReductionProxyStatisticsPrefsTest,
- HistoricNetworkStatsInfoToValueDirect) {
- const int64 kOriginalLength = 150;
- const int64 kReceivedLength = 100;
- statistics_prefs_.reset(new DataReductionProxyStatisticsPrefs(
- &simple_pref_service_,
- task_runner_,
- base::TimeDelta()));
-
- base::DictionaryValue* dict = nullptr;
- scoped_ptr<base::Value> stats_value(
- statistics_prefs_->HistoricNetworkStatsInfoToValue());
- EXPECT_TRUE(stats_value->GetAsDictionary(&dict));
- VerifyPrefs(dict);
-
- statistics_prefs_->SetInt64(prefs::kHttpOriginalContentLength,
- kOriginalLength);
- statistics_prefs_->SetInt64(prefs::kHttpReceivedContentLength,
- kReceivedLength);
-
- stats_value.reset(statistics_prefs_->HistoricNetworkStatsInfoToValue());
- EXPECT_TRUE(stats_value->GetAsDictionary(&dict));
- VerifyPrefs(dict);
-}
-
-TEST_F(DataReductionProxyStatisticsPrefsTest,
- ClearPrefsOnRestartEnabled) {
- base::CommandLine* command_line = base::CommandLine::ForCurrentProcess();
- command_line->AppendSwitch(
- data_reduction_proxy::switches::kClearDataReductionProxyDataSavings);
-
- base::ListValue list_value;
- list_value.Insert(0, new base::StringValue(base::Int64ToString(1234)));
- simple_pref_service_.Set(prefs::kDailyHttpOriginalContentLength, list_value);
-
- statistics_prefs_.reset(new DataReductionProxyStatisticsPrefs(
- &simple_pref_service_,
- task_runner_,
- base::TimeDelta::FromMinutes(kWriteDelayMinutes)));
-
- const base::ListValue* value = simple_pref_service_.GetList(
- prefs::kDailyHttpOriginalContentLength);
- EXPECT_EQ(0u, value->GetSize());
-}
-
-TEST_F(DataReductionProxyStatisticsPrefsTest,
- ClearPrefsOnRestartDisabled) {
- base::ListValue list_value;
- list_value.Insert(0, new base::StringValue(base::Int64ToString(1234)));
- simple_pref_service_.Set(prefs::kDailyHttpOriginalContentLength, list_value);
-
- statistics_prefs_.reset(new DataReductionProxyStatisticsPrefs(
- &simple_pref_service_,
- task_runner_,
- base::TimeDelta::FromMinutes(kWriteDelayMinutes)));
-
- const base::ListValue* value = simple_pref_service_.GetList(
- prefs::kDailyHttpOriginalContentLength);
- std::string string_value;
- value->GetString(0, &string_value);
- EXPECT_EQ("1234", string_value);
-}
-
-} // namespace data_reduction_proxy

Powered by Google App Engine
This is Rietveld 408576698