Closed Bug 1338121 Opened 7 years ago Closed 7 years ago

Remove old tests for bookmarks caching, as it was removed when the async bookmarking API was created

Categories

(Toolkit :: Places, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla54
Tracking Status
firefox54 --- fixed

People

(Reporter: standard8, Assigned: standard8)

References

Details

Attachments

(1 file)

Talking with Marco on irc, toolkit/components/places/tests/bookmarks/test_711914.js can be removed as it tests bookmark caching, which was removed when the async bookmark API was started to be worked on.

I think toolkit/components/places/tests/bookmarks/test_675416.js is a similar test and can be removed as well.
Comment on attachment 8835475 [details]
Bug 1338121 - Remove old tests for bookmarks caching, as it was removed when the async bookmarking API was created.

https://reviewboard.mozilla.org/r/111222/#review112356
Attachment #8835475 - Flags: review?(mak77) → review+
Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/39081b4ed108
Remove old tests for bookmarks caching, as it was removed when the async bookmarking API was created. r=mak
https://hg.mozilla.org/mozilla-central/rev/39081b4ed108
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla54
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: