Closed Bug 1428040 Opened 3 years ago Closed 3 years ago

PlacesUtils.isRootItem should take GUIDs as well as item IDs

Categories

(Toolkit :: Places, enhancement, P2)

enhancement

Tracking

()

RESOLVED FIXED
mozilla59
Tracking Status
firefox59 --- fixed

People

(Reporter: standard8, Assigned: standard8)

References

(Blocks 1 open bug)

Details

(Whiteboard: [fxsearch])

Attachments

(1 file)

As part of my work on bug 1423896 it would be useful if isRootItem can take a guid as well as an id.

Since there's a bit of test rework involved to obsolete the old test, and provide new tests to cover the work of the old test with the new apis, then I'm splitting this out to a separate bug.

It will also aid towards bug 1071511 - where we're replacing ids with guids.
Comment on attachment 8939846 [details]
Bug 1428040 - Allow PlacesUtils.isRootItem to take guids as well as ids.

https://reviewboard.mozilla.org/r/210154/#review215876
Attachment #8939846 - Flags: review?(mak77) → review+
Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/99f4d3702ed6
Allow PlacesUtils.isRootItem to take guids as well as ids. r=mak
Pushed by apavel@mozilla.com:
https://hg.mozilla.org/mozilla-central/rev/ed1d0d623ca4
Allow PlacesUtils.isRootItem to take guids as well as ids. r=mak
https://hg.mozilla.org/mozilla-central/rev/ed1d0d623ca4
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
You need to log in before you can comment on or make changes to this bug.