commit c7a1d1f288e94304dc6db61a83a1bafd6b4fa62b
author Kelvin Zhang <zhangkelvin@google.com> Fri Jul 29 13:36:29 2022 -0700
committer Kelvin Zhang <zhangkelvin@google.com> Tue Aug 02 12:32:14 2022 -0700
tree 7210a32801832e012c52b83e2ed612743356b1dc
parent b4416c9ade0c509c199d7144380d8a75a0423f3d

Remove ProxyResolver from update_engine

Current ProxyResolver code only supports no-proxy. Therefore it is
barely any use. Remove current implementation for a better one.

Test: th
Bug: 235816007
Change-Id: Ieb46dedf6c6ea86b11c81d7691c2adb578d3d97d

common/multi_range_http_fetcher.h

diff --git a/common/multi_range_http_fetcher.h b/common/multi_range_http_fetcher.h
index ef32f0d..849ed32 100644
--- a/common/multi_range_http_fetcher.h
+++ b/common/multi_range_http_fetcher.h
@@ -46,7 +46,7 @@
  public:
   // Takes ownership of the passed in fetcher.
   explicit MultiRangeHttpFetcher(HttpFetcher* base_fetcher)
-      : HttpFetcher(base_fetcher->proxy_resolver()),
+      : HttpFetcher(),
         base_fetcher_(base_fetcher),
         base_fetcher_active_(false),
         pending_transfer_ended_(false),
@@ -101,7 +101,8 @@
   }
   // TODO(deymo): Determine if this method should be virtual in HttpFetcher so
   // this call is sent to the base_fetcher_.
-  virtual void SetProxies(const std::deque<std::string>& proxies) {
+  void SetProxies(const std::deque<std::string>& proxies) override {
+    HttpFetcher::SetProxies(proxies);
     base_fetcher_->SetProxies(proxies);
   }