Rename ProxyResolutionRequestImpl

Since the ProxyResolutionRequestImpl is specific to the
ConfiguredProxyResolutionService, it is appropriate to rename it to
ConfiguredProxyResolutionRequest.

Bug: 1032820
Change-Id: If8691fa721b5a41229b534ed7ad164d0aa7bd5a3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2111835
Reviewed-by: Eric Roman <[email protected]>
Commit-Queue: Eric Roman <[email protected]>
Cr-Commit-Position: refs/heads/master@{#752234}
diff --git a/net/proxy_resolution/configured_proxy_resolution_request.h b/net/proxy_resolution/configured_proxy_resolution_request.h
new file mode 100644
index 0000000..b7b25efa
--- /dev/null
+++ b/net/proxy_resolution/configured_proxy_resolution_request.h
@@ -0,0 +1,99 @@
+// Copyright 2020 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.
+
+#ifndef NET_PROXY_RESOLUTION_CONFIGURED_PROXY_RESOLUTION_REQUEST_H_
+#define NET_PROXY_RESOLUTION_CONFIGURED_PROXY_RESOLUTION_REQUEST_H_
+
+#include <memory>
+#include <string>
+
+#include "base/time/time.h"
+#include "net/base/completion_once_callback.h"
+#include "net/base/network_isolation_key.h"
+#include "net/log/net_log_with_source.h"
+#include "net/proxy_resolution/proxy_resolution_request.h"
+#include "net/proxy_resolution/proxy_resolver.h"
+#include "net/traffic_annotation/network_traffic_annotation.h"
+#include "url/gurl.h"
+
+namespace net {
+
+class ProxyInfo;
+class ConfiguredProxyResolutionService;
+
+// ConfiguredProxyResolutionRequest is the concrete implementation of
+// ProxyResolutionRequest used by ConfiguredProxyResolutionService. Manages a
+// single asynchronous proxy resolution request.
+class ConfiguredProxyResolutionRequest final : public ProxyResolutionRequest {
+ public:
+  ConfiguredProxyResolutionRequest(
+      ConfiguredProxyResolutionService* service,
+      const GURL& url,
+      const std::string& method,
+      const NetworkIsolationKey& network_isolation_key,
+      ProxyInfo* results,
+      const CompletionOnceCallback user_callback,
+      const NetLogWithSource& net_log);
+
+  ConfiguredProxyResolutionRequest(const ConfiguredProxyResolutionRequest&) =
+      delete;
+  ConfiguredProxyResolutionRequest& operator=(
+      const ConfiguredProxyResolutionRequest&) = delete;
+
+  ~ConfiguredProxyResolutionRequest() override;
+
+  // Starts the resolve proxy request.
+  int Start();
+
+  bool is_started() const {
+    // Note that !! casts to bool. (VS gives a warning otherwise).
+    return !!resolve_job_.get();
+  }
+
+  void StartAndCompleteCheckingForSynchronous();
+
+  void CancelResolveJob();
+
+  // Returns true if the request has been completed.
+  bool was_completed() const { return user_callback_.is_null(); }
+
+  // Callback for when the ProxyResolver request has completed.
+  void QueryComplete(int result_code);
+
+  // Helper to call after ProxyResolver completion (both synchronous and
+  // asynchronous). Fixes up the result that is to be returned to user.
+  int QueryDidComplete(int result_code);
+
+  // Helper to call if the request completes synchronously, since in that case
+  // the request will not be added to |pending_requests_| (in
+  // |ConfiguredProxyResolutionService|).
+  int QueryDidCompleteSynchronously(int result_code);
+
+  NetLogWithSource* net_log() { return &net_log_; }
+
+  // Request implementation:
+  LoadState GetLoadState() const override;
+
+ private:
+  // Note that Request holds a bare pointer to the
+  // ConfiguredProxyResolutionService. Outstanding requests are cancelled during
+  // ~ConfiguredProxyResolutionService, so this is guaranteed to be valid
+  // throughout our lifetime.
+  ConfiguredProxyResolutionService* service_;
+  CompletionOnceCallback user_callback_;
+  ProxyInfo* results_;
+  const GURL url_;
+  const std::string method_;
+  const NetworkIsolationKey network_isolation_key_;
+  std::unique_ptr<ProxyResolver::Request> resolve_job_;
+  MutableNetworkTrafficAnnotationTag traffic_annotation_;
+  NetLogWithSource net_log_;
+  // Time when the request was created.  Stored here rather than in |results_|
+  // because the time in |results_| will be cleared.
+  base::TimeTicks creation_time_;
+};
+
+}  // namespace net
+
+#endif  // NET_PROXY_RESOLUTION_CONFIGURED_PROXY_RESOLUTION_REQUEST_H_