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

Issue 29468579: Issue 5332 - Do not filter out rules containing non-ASCII characters (Closed)

Created:
June 19, 2017, 4:09 p.m. by Manish Jethani
Modified:
July 9, 2017, 1:13 p.m.
Reviewers:
kzar
CC:
Sebastian Noack
Base URL:
https://hg.adblockplus.org/abp2blocklist
Visibility:
Public.

Description

Issue 5332 - Do not filter out rules containing non-ASCII characters

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -27 lines) Patch
M lib/abp2blocklist.js View 2 chunks +1 line, -25 lines 0 comments Download
M test/abp2blocklist.js View 1 chunk +4 lines, -2 lines 1 comment Download

Messages

Total messages: 3
Manish Jethani
June 19, 2017, 4:10 p.m. (2017-06-19 16:10:01 UTC) #1
Manish Jethani
Patch Set 1 There's no need to filter out non-ASCII characters here as Safari supports ...
June 19, 2017, 4:12 p.m. (2017-06-19 16:12:16 UTC) #2
kzar
July 7, 2017, 12:19 p.m. (2017-07-07 12:19:30 UTC) #3
(Moving Sebastian to Cc.)

LGTM

Powered by Google App Engine
This is Rietveld