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

Issue 29855555: Issue 6849 - Remove filter list checksum verification (Closed)

Created:
Aug. 14, 2018, 9:23 a.m. by Manish Jethani
Modified:
Aug. 14, 2018, 5:31 p.m.
Reviewers:
Sebastian Noack, hub
CC:
kzar
Base URL:
https://hg.adblockplus.org/adblockpluscore/
Visibility:
Public.

Patch Set 1 #

Patch Set 2 : Remove generateChecksum stub #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -75 lines) Patch
M lib/synchronizer.js View 4 chunks +0 lines, -15 lines 0 comments Download
M test/stub-modules/utils.js View 1 1 chunk +0 lines, -9 lines 0 comments Download
M test/synchronizer.js View 2 chunks +0 lines, -51 lines 0 comments Download

Messages

Total messages: 5
Manish Jethani
Aug. 14, 2018, 9:23 a.m. (2018-08-14 09:23:30 UTC) #1
Manish Jethani
Patch Set 1 Patch Set 2: Remove generateChecksum stub
Aug. 14, 2018, 9:27 a.m. (2018-08-14 09:27:57 UTC) #2
Manish Jethani
On 2018/08/14 09:27:57, Manish Jethani wrote: > Patch Set 1 > Patch Set 2: Remove ...
Aug. 14, 2018, 4:41 p.m. (2018-08-14 16:41:18 UTC) #3
Sebastian Noack
LGTM
Aug. 14, 2018, 5:01 p.m. (2018-08-14 17:01:48 UTC) #4
hub
Aug. 14, 2018, 5:01 p.m. (2018-08-14 17:01:53 UTC) #5
LGTM

Powered by Google App Engine
This is Rietveld