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

Issue 29372803: Issue 4774 - 'Block ads on this page' does not remove whitelisting when re-selected (Closed)

Created:
Jan. 20, 2017, 11:51 p.m. by diegocarloslima
Modified:
Jan. 31, 2017, 8:10 p.m.
Reviewers:
anton, Felix Dahlke
CC:
René Jeschke
Visibility:
Public.

Description

Issue 4774 - 'Block ads on this page' does not remove whitelisting when re-selected

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -2 lines) Patch
M adblockplus/Api.jsm View 2 chunks +2 lines, -2 lines 0 comments Download

Messages

Total messages: 3
diegocarloslima
Jan. 20, 2017, 11:54 p.m. (2017-01-20 23:54:17 UTC) #1
anton
On 2017/01/20 23:54:17, diegocarloslima wrote: LGTM but i'm not so deep in it.
Jan. 23, 2017, 7 a.m. (2017-01-23 07:00:41 UTC) #2
Felix Dahlke
Jan. 30, 2017, 3:40 p.m. (2017-01-30 15:40:04 UTC) #3
LGTM

Powered by Google App Engine
This is Rietveld