Closed Bug 859151 Opened 8 years ago Closed 8 years ago

Check if we can remove PlacesUtils.backups.profileRelativeFolderPath

Categories

(Toolkit :: Places, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: raymondlee, Assigned: raymondlee)

References

Details

Attachments

(1 obsolete file)

No description provided.
No addons use reply on this property so we can remove it
Attached patch v1 (obsolete) — Splinter Review
Attachment #734409 - Flags: review?(mak77)
Comment on attachment 734409 [details] [diff] [review]
v1

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

we should rather first move it to PlacesBackups, and then use the new one
Attachment #734409 - Flags: review?(mak77)
Depends on: 855638
(In reply to Marco Bonardo [:mak] from comment #3)
> Comment on attachment 734409 [details] [diff] [review]
> v1
> 
> Review of attachment 734409 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> we should rather first move it to PlacesBackups, and then use the new one

We can just need to remove PlacesUtils.backups.profileRelativeFolderPath as no addons are using it.  It's best to leave this to bug 857429 which would remove and deprecate PlacesUtils.backups.
Blocks: 857429
Status: NEW → ASSIGNED
Summary: Check if we can remove PlacesUtils.backup.profileRelativeFolderPath → Check if we can remove PlacesUtils.backups.profileRelativeFolderPath
Attachment #734409 - Attachment is obsolete: true
Bug 855638 moved the PlacesUtils.backups.profileRelativeFolderPath to PlacesBackups
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.