Closed Bug 1508165 Opened 2 years ago Closed 1 year ago

Remove support for treecol[type="password"]


(Core :: XUL, task, P5)




Tracking Status
firefox71 --- fixed


(Reporter: ntim, Assigned: WeirdAl)


(Blocks 1 open bug)



(2 files)

Support was introduced in bug 1192492 specially for bug 1121291, but that bug got backed out in bug 1208145.

So AFAIK, this can be removed.
Matt, can you please confirm that this feature is unused ?
Component: XUL Widgets → XUL
Flags: needinfo?(MattN+bmo)
Product: Toolkit → Core
It's supposed to be used again in bug 1257078 / bug 1209267 but nobody has been working on password manager for years so it hasn't happened yet. from that bug is changing the behaviour.
Flags: needinfo?(MattN+bmo)
Priority: -- → P5
Type: enhancement → task

I've just confirmed by visual inspection that mozilla-central and comm-central don't use treecol type="password".

That is correct. See comment 2 which points out bugs that aren't fixed.

Attachment #9099622 - Attachment description: Bug 1508165, remove support for treecol[type=password]. r=bgrins → Bug 1508165, remove support for treecol[type=password]. r=MattN
Assignee: nobody → ajvincent
Keywords: checkin-needed

hg error in cmd: hg push -r tip upstream: pushing to ssh:// searching for changes remote: adding changesets remote: adding manifests remote: adding file changes remote: added 1 changesets with 5 changes to 5 files remote: remote: ******************************* ERROR ******************************* remote: Changeset c11a77a16d77 alters WebIDL file(s) without DOM peer review: remote: dom/webidl/TreeColumn.webidl remote: remote: Please, request review from either: remote: - Andrea Marchesini (:baku) remote: - Andreas Farre (:farre) remote: - Andrew McCreight (:mccr8) remote: - Bobby Holley (:bholley) remote: - Boris Zbarsky (:bz) remote: - Ehsan Akhgari (:ehsan) remote: - Henri Sivonen (:hsivonen) remote: - Nika Layzell (:mystor) remote: - Olli Pettay (:smaug) remote: - Peter Van der Beken (:peterv) remote: ********************************************************************* remote: remote: transaction abort! remote: rollback completed remote: pretxnchangegroup.mozhooks hook failed abort: push failed on remote

Flags: needinfo?(ajvincent)
Keywords: checkin-needed

sigh Sorry about that... I've now asked #webidl to review.

Flags: needinfo?(ajvincent)
Keywords: checkin-needed

Pushed by
remove support for treecol[type=password]. r=MattN,webidl,smaug

Keywords: checkin-needed
Pushed by
build bustage fix for a no-longer-used variable.
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla71
You need to log in before you can comment on or make changes to this bug.