As far as I can tell we use it to show the Window list in the Window menu. +++ This bug was initially created as a clone of Bug #1425363 +++ As far as we can tell, we have an RDF data source for the single purpose of doing this menu only on mac. Instead we could just implement it with some simple JS.

Old menu with numbers and hightlighted current window.

New menu

Comment on attachment 8941179 [details] [diff] [review] Straight port Window list works fine but the active window is no longer checked. This does not seem to work: > if (win == window) { > - sep.setAttribute("checked", "true"); > - break; > + item.setAttribute("checked", "true"); Not sure if this is because of a de-xbl bug? The old menu had type="radio" in it. Do you want to try to fix it or should we file a followup bug? The old menu did had the windows numbered and you could use the numbers as an accelerator key but personally I don't think we need to port this. f+ for now.

Ah, the missing radio must be because I just create a plain menuitem (on Mac, this works fine). Maybe we can file a follow-up for the number/accesskey? It's certainly doable.

Comment on attachment 8941542 [details] [diff] [review] Now with type="radio" That did the trick. Thanks++. Could you remove the period from "r=frg." when checking it in.

Pushed by stefanh@inbox.com: https://hg.mozilla.org/comm-central/rev/4de3d93be690 Port bug 1425363 to SeaMonkey: Switch window menu to JS instead of RDF. r=frg.

(In reply to Frank-Rainer Grahl (:frg) from comment #6) > Could you remove the period from "r=frg." when > checking it in. Oops (I just did an hg import and a push).

Comment on attachment 8941542 [details] [diff] [review] Now with type="radio" This would be good for 2.53

Comment on attachment 8941542 [details] [diff] [review] Now with type="radio" Added to the unofficial builds already