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

Unified Diff: src/WebRequestJsObject.cpp

Issue 29497591: Issue 5180 - remove synchronous WebRequest interface and stop exposing of DefaultWebRequest (Closed) Base URL: https://github.com/adblockplus/libadblockplus.git
Patch Set: Created July 25, 2017, 2:31 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/JsEngine.cpp ('k') | test/Notification.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/WebRequestJsObject.cpp
diff --git a/src/WebRequestJsObject.cpp b/src/WebRequestJsObject.cpp
index d4288a637725aeb012c09d2857bfdb5304faab6c..7aad0ca25dda4b1d7a417e56bcab4d4bd92a5978 100644
--- a/src/WebRequestJsObject.cpp
+++ b/src/WebRequestJsObject.cpp
@@ -80,15 +80,6 @@ void JsEngine::ScheduleWebRequest(const v8::FunctionCallbackInfo<v8::Value>& arg
webRequestParams[2].Call(resultObject);
};
- if (jsEngine->webRequestLegacy)
- {
- std::thread([jsEngine, url, headers, getCallback]
- {
- getCallback(jsEngine->webRequestLegacy->GET(url, headers));
- }).detach();
- return;
- }
-
jsEngine->webRequest->GET(url, headers, getCallback);
}
« no previous file with comments | « src/JsEngine.cpp ('k') | test/Notification.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld