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

Unified Diff: components/data_reduction_proxy/core/common/data_reduction_proxy_params_test_utils.h

Issue 2922663002: Data Reduction Proxy: Remove duplicate functions (Closed)
Patch Set: megjablon comments Created 3 years, 6 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/common/data_reduction_proxy_params_test_utils.h
diff --git a/components/data_reduction_proxy/core/common/data_reduction_proxy_params_test_utils.h b/components/data_reduction_proxy/core/common/data_reduction_proxy_params_test_utils.h
index f779e291a6d6bcc1d373f41114aaa2447f3222fc..201ee9acd00942ffe631912085524f13b572b6ce 100644
--- a/components/data_reduction_proxy/core/common/data_reduction_proxy_params_test_utils.h
+++ b/components/data_reduction_proxy/core/common/data_reduction_proxy_params_test_utils.h
@@ -15,17 +15,7 @@ class DataReductionProxyServer;
class TestDataReductionProxyParams : public DataReductionProxyParams {
public:
- // Used to emulate having constants defined by the preprocessor.
- enum HasNames {
- HAS_NOTHING = 0x0,
- HAS_ORIGIN = 0x2,
- HAS_FALLBACK_ORIGIN = 0x4,
- HAS_SECURE_PROXY_CHECK_URL = 0x40,
- HAS_EVERYTHING = 0xff,
- };
-
- TestDataReductionProxyParams(int flags,
- unsigned int has_definitions);
+ TestDataReductionProxyParams();
bool init_result() const;
void SetProxiesForHttp(const std::vector<DataReductionProxyServer>& proxies);
@@ -47,10 +37,6 @@ class TestDataReductionProxyParams : public DataReductionProxyParams {
std::string GetDefaultSecureProxyCheckURL() const override;
private:
- std::string GetDefinition(unsigned int has_def,
- const std::string& definition) const;
-
- unsigned int has_definitions_;
bool init_result_;
};
} // namespace data_reduction_proxy

Powered by Google App Engine
This is Rietveld 408576698