Closed Bug 1433979 Opened 2 years ago Closed 2 years ago

Remove deprecated code from PlacesBackups.jsm

Categories

(Toolkit :: Places, enhancement, P2)

enhancement

Tracking

()

RESOLVED FIXED
mozilla60
Tracking Status
firefox60 --- fixed

People

(Reporter: standard8, Assigned: standard8)

References

Details

(Whiteboard: [fxsearch])

Attachments

(1 file)

PlaceBackup.jsm has some code that was deprecated in bug 859695, now that add-ons are no longer able to access it, we should remove the old code.

The deprecation was due to OS.File adoption.

There's also some double-handling of backup file lists, so we can simplify that as well.
Comment on attachment 8946318 [details]
Bug 1433979 - Remove deprecated code from PlacesBackups.jsm as it is no longer necessary.

https://reviewboard.mozilla.org/r/216296/#review222718

nice one!

::: browser/components/places/content/places.js:536
(Diff revision 1)
>      let fp = Cc["@mozilla.org/filepicker;1"].createInstance(Ci.nsIFilePicker);
>      let fpCallback = function fpCallback_done(aResult) {
>        if (aResult != Ci.nsIFilePicker.returnCancel) {
>          // There is no OS.File version of the filepicker yet (Bug 937812).
> -        PlacesBackups.saveBookmarksToJSONFile(fp.file.path);
> +        PlacesBackups.saveBookmarksToJSONFile(fp.file.path)
> +          .catch(Cu.reportError);

nit: please align dots

::: toolkit/components/places/PlacesBackups.jsm:20
(Diff revision 1)
>  XPCOMUtils.defineLazyModuleGetter(this, "BookmarkJSONUtils",
>    "resource://gre/modules/BookmarkJSONUtils.jsm");
> -XPCOMUtils.defineLazyModuleGetter(this, "Deprecated",
> -  "resource://gre/modules/Deprecated.jsm");
>  XPCOMUtils.defineLazyModuleGetter(this, "OS",
>    "resource://gre/modules/osfile.jsm");

please use defineLazyModuleGetters, and also include PlacesUtils into it.
Attachment #8946318 - Flags: review?(mak77) → review+
Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d5600d2ea770
Remove deprecated code from PlacesBackups.jsm as it is no longer necessary. r=mak
https://hg.mozilla.org/mozilla-central/rev/d5600d2ea770
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla60
You need to log in before you can comment on or make changes to this bug.