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

Issue 29355386: Issue 4463 - Content blocker unavailable error message (Closed)

Created:
Sept. 30, 2016, 2:44 a.m. by diegocarloslima
Modified:
Oct. 14, 2016, 7:17 p.m.
Reviewers:
anton, Felix Dahlke
Visibility:
Public.

Description

Issue 4463 - Content blocker unavailable error message

Patch Set 1 #

Total comments: 3

Patch Set 2 : Reverting ABP version to hardcoded String #

Patch Set 3 : Adjusting indentation #

Unified diffs Side-by-side diffs Delta from patch set Stats (+96 lines, -49 lines) Patch
M res/values/sysstrings.xml View 1 chunk +2 lines, -0 lines 0 comments Download
M src/org/adblockplus/sbrowser/contentblocker/ContentBlockerContentProvider.java View 3 chunks +4 lines, -41 lines 0 comments Download
M src/org/adblockplus/sbrowser/contentblocker/engine/Engine.java View 1 2 6 chunks +87 lines, -4 lines 0 comments Download
M src/org/adblockplus/sbrowser/contentblocker/engine/Subscriptions.java View 2 chunks +3 lines, -4 lines 0 comments Download

Messages

Total messages: 5
diegocarloslima
Sept. 30, 2016, 2:50 a.m. (2016-09-30 02:50:32 UTC) #1
anton
On 2016/09/30 02:50:32, diegocarloslima wrote: LGTM
Sept. 30, 2016, 6:21 a.m. (2016-09-30 06:21:12 UTC) #2
Felix Dahlke
Looks good, just two comments. https://codereview.adblockplus.org/29355386/diff/29355387/src/org/adblockplus/sbrowser/contentblocker/engine/Engine.java File src/org/adblockplus/sbrowser/contentblocker/engine/Engine.java (right): https://codereview.adblockplus.org/29355386/diff/29355387/src/org/adblockplus/sbrowser/contentblocker/engine/Engine.java#newcode58 src/org/adblockplus/sbrowser/contentblocker/engine/Engine.java:58: public static final String ...
Sept. 30, 2016, 7:11 a.m. (2016-09-30 07:11:41 UTC) #3
diegocarloslima
https://codereview.adblockplus.org/29355386/diff/29355387/src/org/adblockplus/sbrowser/contentblocker/engine/Engine.java File src/org/adblockplus/sbrowser/contentblocker/engine/Engine.java (right): https://codereview.adblockplus.org/29355386/diff/29355387/src/org/adblockplus/sbrowser/contentblocker/engine/Engine.java#newcode58 src/org/adblockplus/sbrowser/contentblocker/engine/Engine.java:58: public static final String ABP_VERSION = "2.0"; On 2016/09/30 ...
Sept. 30, 2016, 2:06 p.m. (2016-09-30 14:06:38 UTC) #4
Felix Dahlke
Oct. 12, 2016, 4:56 a.m. (2016-10-12 04:56:37 UTC) #5
Sorry, missed your response for some reason - LGTM!

Powered by Google App Engine
This is Rietveld