Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: src/DefaultWebRequest.h

Issue 29533641: Issue 5569 - add the possibility to use synchronous WebRequest and base path in File System (Closed) Base URL: https://github.com/adblockplus/libadblockplus.git
Patch Set: Created Sept. 1, 2017, 3:32 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « src/DefaultFileSystem.cpp ('k') | src/DefaultWebRequest.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/DefaultWebRequest.h
diff --git a/src/DefaultWebRequest.h b/src/DefaultWebRequest.h
index 054ef0d88850e2b48fa9638c45a6ef4a1f76f668..0c71583b763940800bc75cc72db1c090f8034100 100644
--- a/src/DefaultWebRequest.h
+++ b/src/DefaultWebRequest.h
@@ -28,22 +28,22 @@ namespace AdblockPlus
* on other platforms. A dummy implementation that always reports failure is
* used if libcurl is not available.
*/
- class DefaultWebRequestSync
+ class DefaultWebRequestSync : public IWebRequestSync
{
public:
- ServerResponse GET(const std::string& url, const HeaderList& requestHeaders) const;
+ ServerResponse GET(const std::string& url, const HeaderList& requestHeaders) const override;
};
class DefaultWebRequest : public IWebRequest
{
public:
- explicit DefaultWebRequest(const Scheduler& scheduler, std::unique_ptr<DefaultWebRequestSync>&& syncImpl);
+ explicit DefaultWebRequest(const Scheduler& scheduler, WebRequestSyncPtr syncImpl);
~DefaultWebRequest();
void GET(const std::string& url, const HeaderList& requestHeaders, const GetCallback& getCallback) override;
private:
Scheduler scheduler;
- std::unique_ptr<DefaultWebRequestSync> syncImpl;
+ WebRequestSyncPtr syncImpl;
};
}
« no previous file with comments | « src/DefaultFileSystem.cpp ('k') | src/DefaultWebRequest.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld