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

Issue 5719985141841920: issue #331 - Remove localStorage to FileSystem migration code (Closed)

Created:
Aug. 14, 2014, 11:03 a.m. by saroyanm
Modified:
Aug. 25, 2014, 10:39 a.m.
Visibility:
Public.

Description

This review is related to current ticket: https://issues.adblockplus.org/ticket/331

Patch Set 1 #

Total comments: 2

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -56 lines) Patch
M background.js View 3 chunks +1 line, -28 lines 0 comments Download
M lib/filesystem/io.js View 1 2 chunks +0 lines, -28 lines 0 comments Download

Messages

Total messages: 4
saroyanm
Sebastian, can you please have a look when you'll have a free time. Looks to ...
Aug. 14, 2014, 11:34 a.m. (2014-08-14 11:34:16 UTC) #1
Wladimir Palant
http://codereview.adblockplus.org/5719985141841920/diff/5629499534213120/background.js File background.js (left): http://codereview.adblockplus.org/5719985141841920/diff/5629499534213120/background.js#oldcode164 background.js:164: FilterStorage.loadFromDisk(localStorage["patterns.ini"]); There is a hack in lib/filesystem/io.js, function readFromFile() ...
Aug. 14, 2014, 2:16 p.m. (2014-08-14 14:16:13 UTC) #2
saroyanm
new patch uploaded. http://codereview.adblockplus.org/5719985141841920/diff/5629499534213120/background.js File background.js (left): http://codereview.adblockplus.org/5719985141841920/diff/5629499534213120/background.js#oldcode164 background.js:164: FilterStorage.loadFromDisk(localStorage["patterns.ini"]); On 2014/08/14 14:16:13, Wladimir Palant ...
Aug. 14, 2014, 4:09 p.m. (2014-08-14 16:09:53 UTC) #3
Sebastian Noack
Aug. 25, 2014, 10:04 a.m. (2014-08-25 10:04:31 UTC) #4
LGTM, from my side.

Powered by Google App Engine
This is Rietveld