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

Issue 29497591: Issue 5180 - remove synchronous WebRequest interface and stop exposing of DefaultWebRequest (Closed)

Created:
July 25, 2017, 2:31 p.m. by sergei
Modified:
July 26, 2017, 1:12 p.m.
Reviewers:
hub
CC:
Felix Dahlke
Base URL:
https://github.com/adblockplus/libadblockplus.git
Visibility:
Public.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+88 lines, -209 lines) Patch
M include/AdblockPlus.h View 1 chunk +0 lines, -2 lines 0 comments Download
D include/AdblockPlus/DefaultWebRequest.h View 1 chunk +0 lines, -47 lines 0 comments Download
M include/AdblockPlus/JsEngine.h View 3 chunks +1 line, -10 lines 0 comments Download
D include/AdblockPlus/WebRequest.h View 1 chunk +0 lines, -48 lines 0 comments Download
M libadblockplus.gyp View 2 chunks +1 line, -1 line 0 comments Download
A + src/DefaultWebRequest.h View 2 chunks +5 lines, -4 lines 0 comments Download
M src/DefaultWebRequest.cpp View 2 chunks +5 lines, -6 lines 0 comments Download
M src/DefaultWebRequestCurl.cpp View 1 chunk +1 line, -1 line 0 comments Download
M src/DefaultWebRequestDummy.cpp View 1 chunk +1 line, -1 line 0 comments Download
M src/DefaultWebRequestWinInet.cpp View 1 chunk +1 line, -1 line 0 comments Download
M src/JsEngine.cpp View 3 chunks +2 lines, -9 lines 0 comments Download
M src/WebRequestJsObject.cpp View 1 chunk +0 lines, -9 lines 0 comments Download
M test/Notification.cpp View 2 chunks +5 lines, -5 lines 0 comments Download
M test/WebRequest.cpp View 10 chunks +66 lines, -65 lines 0 comments Download

Messages

Total messages: 2
sergei
July 25, 2017, 2:34 p.m. (2017-07-25 14:34:54 UTC) #1
hub
July 25, 2017, 5:26 p.m. (2017-07-25 17:26:02 UTC) #2
LGTM

Powered by Google App Engine
This is Rietveld