Closed Bug 709328 Opened 8 years ago Closed 8 years ago

Remove dead argument

Categories

(Toolkit :: Form Manager, defect)

x86_64
Linux
defect
Not set

Tracking

()

RESOLVED FIXED

People

(Reporter: espindola, Assigned: espindola)

Details

Attachments

(1 file)

the backup argument is always true. Remove it.
Assignee: nobody → respindola
Attachment #580556 - Flags: review?(mak77)
Comment on attachment 580556 [details] [diff] [review]
remove always true argument.

Review of attachment 580556 [details] [diff] [review]:
-----------------------------------------------------------------

it looks ok, this goes back to the first js rewrite, so doesn't look like a recent miss or regression.

::: toolkit/components/satchel/nsFormHistory.js
@@ +894,3 @@
>  
> +        let backupFile = this.dbFile.leafName + ".corrupt";
> +        storage.backupDatabaseFile(this.dbFile, backupFile);

nit: the newline in the middle (before "let backupFile") doesn't really look useful.
Attachment #580556 - Flags: review?(mak77) → review+
https://tbpl.mozilla.org/?rev=604ce2fde8ca
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.