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

Issue 29391555: Issue 4948 - add possibility to not send data depending on connection properties (Closed)

Created:
March 22, 2017, 5:25 a.m. by anton
Modified:
March 28, 2017, 5:49 a.m.
Reviewers:
sergei, Felix Dahlke
CC:
René Jeschke, diegocarloslima
Visibility:
Public.

Description

Issue 4948 - add possibility to not send data depending on connection properties This is update to libadblockplus-binaries repo

Patch Set 1 : update to libadblockplus hg:ec8e2edc66af #

Patch Set 2 : update to libadblockplus hg:1d708e673634 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+126 lines, -32 lines) Patch
M android_armeabi-v7a/libadblockplus.a View Binary file 0 comments Download
M android_armeabi-v7a/libv8_base.a View Binary file 0 comments Download
M android_armeabi-v7a/libv8_snapshot.a View 1 Binary file 0 comments Download
M android_x86/libadblockplus.a View Binary file 0 comments Download
M android_x86/libv8_base.a View Binary file 0 comments Download
M android_x86/libv8_snapshot.a View 1 Binary file 0 comments Download
M include/AdblockPlus.h View 1 chunk +1 line, -1 line 0 comments Download
M include/AdblockPlus/AppInfo.h View 1 chunk +1 line, -1 line 0 comments Download
M include/AdblockPlus/DefaultFileSystem.h View 1 chunk +1 line, -1 line 0 comments Download
M include/AdblockPlus/DefaultLogSystem.h View 1 chunk +1 line, -1 line 0 comments Download
M include/AdblockPlus/DefaultWebRequest.h View 1 chunk +1 line, -1 line 0 comments Download
M include/AdblockPlus/FileSystem.h View 1 chunk +1 line, -1 line 0 comments Download
M include/AdblockPlus/FilterEngine.h View 1 12 chunks +70 lines, -18 lines 0 comments Download
M include/AdblockPlus/JsEngine.h View 1 6 chunks +41 lines, -1 line 0 comments Download
M include/AdblockPlus/JsValue.h View 1 4 chunks +5 lines, -3 lines 0 comments Download
M include/AdblockPlus/LogSystem.h View 1 chunk +1 line, -1 line 0 comments Download
M include/AdblockPlus/Notification.h View 1 chunk +1 line, -1 line 0 comments Download
M include/AdblockPlus/ReferrerMapping.h View 1 chunk +1 line, -1 line 0 comments Download
M include/AdblockPlus/WebRequest.h View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 6
anton
March 22, 2017, 5:27 a.m. (2017-03-22 05:27:49 UTC) #1
sergei
LGTM, though I think that it should be either another issue or noissue.
March 22, 2017, 8:41 a.m. (2017-03-22 08:41:36 UTC) #2
anton
On 2017/03/22 08:41:36, sergei wrote: > LGTM, though I think that it should be either ...
March 22, 2017, 9:55 a.m. (2017-03-22 09:55:26 UTC) #3
sergei
On 2017/03/22 09:55:26, anton wrote: > On 2017/03/22 08:41:36, sergei wrote: > > LGTM, though ...
March 22, 2017, 10:05 a.m. (2017-03-22 10:05:26 UTC) #4
anton
On 2017/03/22 10:05:26, sergei wrote: > On 2017/03/22 09:55:26, anton wrote: > > On 2017/03/22 ...
March 22, 2017, 10:07 a.m. (2017-03-22 10:07:49 UTC) #5
Felix Dahlke
March 27, 2017, 2:39 p.m. (2017-03-27 14:39:03 UTC) #6
LGTM - yeah let's not reference the issue in the commit messages, we never do
that in libadblockplus-android.

Moved Diego to CC since two reviewers to what is ultimately just a bunch of
binary updates and copied headers should be plenty.

Powered by Google App Engine
This is Rietveld