Remove nsITreeContentView

RESOLVED FIXED in Firefox 62

Status

()

Product:
Core
Component:
Layout
Type:
enhancement
Importance:
P3
normal
Status:
RESOLVED FIXED
Opened:
Last year
Updated:
Last year

People

(Reporter: bzbarsky, Assigned: bzbarsky)

Tracking

Version:
unspecified
Target:
mozilla62
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox62 fixed)

Details

Attachments

(2 attachments)

part 1. Mark nsITreeContentView builtinclass
816 bytes, patch
mossop
: review+
Details | Diff | Splinter Review
part 2. Remove nsITreeContentView
4.40 KB, patch
mossop
: review+
Details | Diff | Splinter Review 
Seems to be unused in C++ and not implemented by JS.
Priority: -- → P3
There are no JS implementations that I can see, but this will get us nicer bisectability if it turns out I'm wrong.
Attachment #8983184 - Flags: review?(dtownsend)
Assignee: nobody → bzbarsky
Status: NEW → ASSIGNED
Attachment #8983184 - Flags: review?(dtownsend) → review+
Attachment #8983186 - Flags: review?(dtownsend) → review+
Pushed by bzbarsky@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/189125be1c9b
part 1.  Mark nsITreeContentView builtinclass.  r=mossop
https://hg.mozilla.org/integration/mozilla-inbound/rev/cc5f600eebd6
part 2.  Remove nsITreeContentView.  r=mosssop
https://hg.mozilla.org/mozilla-central/rev/189125be1c9b
https://hg.mozilla.org/mozilla-central/rev/cc5f600eebd6
Status: ASSIGNED → RESOLVED
Closed: Last year
status-firefox62: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
You need to log in before you can comment on or make changes to this bug.