579269 - Need a test to check for "Inaccessible <prefpane>'s if not explicitly referenced by a <preftreeitem>"

Status: NEW

(SeaMonkey :: Preferences, defect)

Description

[Justin Wood (:Callek)]

We should develop a test for our prefwindow that checks for inaccessible <prefpane>'s (due to a missing <preftreeitem> reference).

Found this fruit while reveiwing: https://wiki.mozilla.org/User:Callek/Prefwindow_API_Variants  We just need someone to pick it.

Comment 1

[Philip Chee]

preferences.xml:
          // append any still unreferenced <prefpane>s to the tree's top level
So how would you tell it's unreferenced?

Comment 2

[Justin Wood (:Callek)]

(In reply to comment #1)
> preferences.xml:
>           // append any still unreferenced <prefpane>s to the tree's top level
> So how would you tell it's unreferenced?

O suuure, lets believe the code rather than a pre-landing design document.

Given the code though, I wonder in rough passing if a console warning is worthwhile, or if doing this as "supported" XUL design for us is better.

However, this bug is not necessarily invalid; as a test that under normal preferences-XUL code we won't hit just ensures the correctness of our preferences-XUL code. but does lower the priority here for me.

Comment 3

[Philip Chee]

> Given the code though, I wonder in rough passing if a console warning is
> worthwhile, or if doing this as "supported" XUL design for us is better.
Sure nsIConsoleService::logStringMessage() appears to be suitable.