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

Issue 29875591: Noissue - Simplify write throttling for blocked_total pref on Firefox (Closed)

Created:
Sept. 5, 2018, 3:27 p.m. by Sebastian Noack
Modified:
Sept. 6, 2018, 1:30 p.m.
Reviewers:
Manish Jethani, geo
Visibility:
Public.

Description

Noissue - Simplify write throttling for blocked_total pref on Firefox

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -4 lines) Patch
M lib/prefs.js View 2 chunks +1 line, -4 lines 0 comments Download

Messages

Total messages: 3
Sebastian Noack
This is consistent with a decision we recently made in a similar situation (see https://codereview.adblockplus.org/29860578/diff/29874601/lib/indexedDBBackup.js#newcode50). ...
Sept. 5, 2018, 3:35 p.m. (2018-09-05 15:35:12 UTC) #1
Manish Jethani
LGTM
Sept. 6, 2018, 7:19 a.m. (2018-09-06 07:19:46 UTC) #2
geo
Sept. 6, 2018, 1:26 p.m. (2018-09-06 13:26:16 UTC) #3
LGTM

Powered by Google App Engine
This is Rietveld