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

Unified Diff: lib/filterStorage.js

Issue 4831974724206592: Issue 192 - Clean up from #153 (Closed)
Patch Set: Created April 14, 2014, 6:56 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « lib/elemHide.js ('k') | lib/io.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: lib/filterStorage.js
===================================================================
--- a/lib/filterStorage.js
+++ b/lib/filterStorage.js
@@ -368,17 +368,17 @@ let FilterStorage = exports.FilterStorag
TimeLine.enter("Entered FilterStorage.loadFromDisk()");
this._loading = true;
let readFile = function(sourceFile, backupIndex)
{
TimeLine.enter("FilterStorage.loadFromDisk() -> readFile()");
let parser = new INIParser();
- IO.readFromFile(sourceFile, true, parser, function(e)
+ IO.readFromFile(sourceFile, parser, function(e)
{
TimeLine.enter("FilterStorage.loadFromDisk() read callback");
if (!e && parser.subscriptions.length == 0)
{
// No filter subscriptions in the file, this isn't right.
e = new Error("No data in the file");
}
@@ -570,17 +570,17 @@ let FilterStorage = exports.FilterStorag
// Make sure the file's parent directory exists
try {
targetFile.parent.create(Ci.nsIFile.DIRECTORY_TYPE, FileUtils.PERMS_DIRECTORY);
} catch (e) {}
let writeFilters = function()
{
TimeLine.enter("FilterStorage.saveToDisk() -> writeFilters()");
- IO.writeToFile(targetFile, true, this._generateFilterData(subscriptions), function(e)
+ IO.writeToFile(targetFile, this._generateFilterData(subscriptions), function(e)
{
TimeLine.enter("FilterStorage.saveToDisk() write callback");
if (!explicitFile)
this._saving = false;
if (e)
Cu.reportError(e);
« no previous file with comments | « lib/elemHide.js ('k') | lib/io.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld