Issue 5180 - remove synchronous WebRequest interface and stop exposing of DefaultWebRequest

src/DefaultWebRequest.h

Index: src/DefaultWebRequest.h
diff --git a/include/AdblockPlus/DefaultWebRequest.h b/src/DefaultWebRequest.h
similarity index 86%
rename from include/AdblockPlus/DefaultWebRequest.h
rename to src/DefaultWebRequest.h
index 0c7e8721f04e6f01080a10c220c6011b4c399765..aefa5e6e1fe7ff21b5247242b0d2a9322ed2573e 100644
--- a/include/AdblockPlus/DefaultWebRequest.h
+++ b/src/DefaultWebRequest.h
@@ -18,7 +18,7 @@
 #ifndef ADBLOCK_PLUS_DEFAULT_WEB_REQUEST_H
 #define ADBLOCK_PLUS_DEFAULT_WEB_REQUEST_H
 
-#include "WebRequest.h"
+#include <AdblockPlus/IWebRequest.h>
 
 namespace AdblockPlus
 {
@@ -27,20 +27,21 @@ namespace AdblockPlus
    * on other platforms. A dummy implementation that always reports failure is
    * used if libcurl is not available.
    */
-  class DefaultWebRequestSync : public WebRequest
+  class DefaultWebRequestSync
   {
+  public:
     ServerResponse GET(const std::string& url, const HeaderList& requestHeaders) const;
   };
 
   class DefaultWebRequest : public IWebRequest
   {
   public:
-    explicit DefaultWebRequest(const WebRequestSharedPtr& syncImpl);
+    explicit DefaultWebRequest(std::unique_ptr<DefaultWebRequestSync>&& syncImpl);
     ~DefaultWebRequest();
 
     void GET(const std::string& url, const HeaderList& requestHeaders, const GetCallback& getCallback) override;
   private:
-    WebRequestSharedPtr syncImpl;
+    std::unique_ptr<DefaultWebRequestSync> syncImpl;
   };
 }