Cleanup some tag queries related code

RESOLVED FIXED in Firefox 61

Status

()

enhancement
P2
normal
RESOLVED FIXED
Last year
Last year

People

(Reporter: mak, Assigned: mak)

Tracking

unspecified
mozilla61
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox61 fixed)

Details

(Whiteboard: [fxsearch])

Attachments

(1 attachment)

Assignee

Description

Last year
I found a few code lines that could be simplified considered the direction we are taking.
For example, most of the exceptions in serializeNode are not really that useful.
Comment hidden (mozreview-request)
Comment hidden (obsolete)
Assignee

Updated

Last year
Attachment #8966222 - Flags: review?(standard8)
Assignee

Comment 3

Last year
So what actually happens is that with my change Properties would be disabled on the roots, because the virtual roots have a concreteId but not an id. It may even be ok, there's no much point in opening the read-only properties dialog of a root.
Though, I must figure out what to do of the mochitest-browser that checks first if Properties is enabled. Probably the test should use the Library pane instead to check readonly.
Comment hidden (mozreview-request)

Comment 5

Last year
mozreview-review
Comment on attachment 8966222 [details]
Bug 1452621 - Cleanup some tag queries related code.

https://reviewboard.mozilla.org/r/234976/#review240980

Yay, especially nice to get rid of the grandParentId code, and to simplify serializeNode in general.
Attachment #8966222 - Flags: review?(standard8) → review+

Comment 6

Last year
Pushed by mak77@bonardo.net:
https://hg.mozilla.org/integration/autoland/rev/5b9d9f133bee
Cleanup some tag queries related code. r=standard8

Comment 7

Last year
bugherder
https://hg.mozilla.org/mozilla-central/rev/5b9d9f133bee
Status: ASSIGNED → RESOLVED
Closed: Last year
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
Depends on: 1453545
You need to log in before you can comment on or make changes to this bug.