421092 - De-uglify references to prefpane variables

Status: RESOLVED FIXED

(SeaMonkey :: Preferences, defect)

Description

[Robert Kaiser]

Currently, we need to reference prefpane variables with document.getElementById('foo_pane').* when they come from XBL, which includes the helper app listbox stuff and even the menuitems added to it via JS, but also onsyncfrompreference attributes that actually get called through the toolkit preference XBL. We should look into "de-uglification" of that, as Neil named it in review comments to bug 417590.

Comment 1

[[e-mail]]

Attachment: Fix pref-applications

For KaiRo, just looking to double-check I didn't break anything ;-)

* Removed the XUL event listeners and added them in JS instead [optional]
* Moved code to handleEvent to cope with the extra events and targets
* Swapped from alwaysAsk and action attributes to value attributes
* Swapped from var to let, because we shouldn't have used the vars again anyway
* Added two new action types for chooseApp and manageApp
* Simplified the code to set the initial menuitem using its value
* Simplified the code to store the alwaysAsk setting
* Fired events from XBL to avoid calling global methods

Comment 2

[jag (Peter Annema)]

Comment on attachment 308332 [details] [diff] [review]
Fix pref-applications

>Index: suite/common/pref/pref-applications.js
>===================================================================

> // But since nsIHandlerInfo doesn't support plugins, there's no value
> // identifying the "use plugin" action, so we use this constant instead.
> const kActionUsePlugin = 5;
>+const kActionChooseApp = 6;
>+const kActionManageApp = 7;

Could we maybe change these to e.g. 1001, 1002, and 1003 so there's less of a chance of a collision? Or maybe 0x101, 0x102, 0x103, if you're into that sort of thing.


>@@ -1462,26 +1502,13 @@
>     // the item identified by the preferred action (when the preferred action
>     // is to use a helper app, we have to pick the specific helper app item).
>     if (handlerInfo.alwaysAskBeforeHandling)
>+      menu.value = nsIHandlerInfo.alwaysAsk;
>+    else if (handlerInfo.preferredAction == nsIHandlerInfo.userHelperApp &&
>+             preferredApp)

Typo: use, not user, in useHelperApp.

sr=jag with these addressed

Comment 3

[Robert Kaiser]

Comment on attachment 308332 [details] [diff] [review]
Fix pref-applications

Seems to be working fine here, from what I see.

Comment 4

[[e-mail]]

Fix checked in.