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

RESOLVED FIXED in Firefox 54

Status

()

Toolkit
Places
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: standard8, Assigned: standard8)

Tracking

unspecified
mozilla54
Points:
---

Firefox Tracking Flags

(firefox54 fixed)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

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 hidden (mozreview-request)

Comment 2

2 years ago
mozreview-review
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+

Comment 3

2 years ago
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

Comment 4

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/39081b4ed108
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox54: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla54
You need to log in before you can comment on or make changes to this bug.