[PATCH v1 10/22] chnet: Split construct URL req creation into a new function

[Ammar Faizi <ammarfaizi2@gnuweeb.org>]

This is a preparation to add the set header function. We can efficiently
set a new HTTP header if we create the URL request first as we don't
have to have the copy of HTTP headers in a vector.

Signed-off-by: Ammar Faizi <ammarfaizi2@gnuweeb.org>
---
 chnet/chnet.cc | 9 ++++++++-
 chnet/chnet.h  | 1 +
 2 files changed, 9 insertions(+), 1 deletion(-)

diff --git a/chnet/chnet.cc b/chnet/chnet.cc
index 058af89..67d420b 100644
--- a/chnet/chnet.cc
+++ b/chnet/chnet.cc
@@ -111,7 +111,7 @@ CHNetDelegate::~CHNetDelegate(void)
 	sig.Wait();
 }
 
-void CHNetDelegate::_Start(Waiter *sig)
+void CHNetDelegate::ConstructURLRequest(void)
 {
 	net::URLRequestContextBuilder url_req_ctx_b;
 
@@ -128,6 +128,13 @@ void CHNetDelegate::_Start(Waiter *sig)
 	url_req_ctx_ = url_req_ctx_b.Build();
 	url_req_ = url_req_ctx_->CreateRequest(url_, DEFAULT_PRIORITY, this,
 					       traffic_annotation, false);
+}
+
+void CHNetDelegate::_Start(Waiter *sig)
+{
+	if (!url_req_)
+		ConstructURLRequest();
+
 	url_req_->set_method(method_);
 	sig->Signal();
 
diff --git a/chnet/chnet.h b/chnet/chnet.h
index 7f6359b..5e2392c 100644
--- a/chnet/chnet.h
+++ b/chnet/chnet.h
@@ -109,6 +109,7 @@ public:
 
 private:
 	void _Start(Waiter *sig);
+	void ConstructURLRequest(void);
 
 	std::unique_ptr<net::URLRequestContext> url_req_ctx_;
 	scoped_refptr<net::IOBufferWithSize> read_buf_;
-- 
Ammar Faizi