Bommels05/Mozilla
Bommels05/Mozilla
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

bug 397690: display full executable name instead of leaf name for applications in Applications prefpane; Patch by Jim Mathies, r=myk, a=mconnor

Browse Source 
git-svn-id: svn://10.0.0.236/trunk@236887 18797224-902f-48f8-a5cc-f745e15eee43
This commit is contained in:
myk%mozilla.org 2007-09-28 22:50:24 +00:00
parent 1fa9404ffe
commit 78fa3cdb02
1 changed files with 6 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
mozilla/browser/components/preferences/applications.js
View File

@ -1005,7 +1005,7 @@ var gApplicationsPane = {
return this._prefsBundle.getString("saveFile");
case Ci.nsIHandlerInfo.useHelperApp:
return aHandlerInfo.preferredApplicationHandler.name;
return getDisplayNameForFile(aHandlerInfo.preferredApplicationHandler.executable);
case Ci.nsIHandlerInfo.handleInternally:
// For the feed type, handleInternally means live bookmarks.
@ -1065,7 +1065,7 @@ var gApplicationsPane = {
isValidHandlerApp: function(aHandlerApp) {
if (!aHandlerApp)
return false;
if (aHandlerApp instanceof Ci.nsILocalHandlerApp)
return aHandlerApp.executable &&
aHandlerApp.executable.exists() &&
@ -1150,7 +1150,10 @@ var gApplicationsPane = {
let menuItem = document.createElementNS(kXULNS, "menuitem");
menuItem.setAttribute("action", Ci.nsIHandlerInfo.useHelperApp);
menuItem.setAttribute("label", possibleApp.name);
if (possibleApp instanceof Ci.nsILocalHandlerApp)
menuItem.setAttribute("label", getDisplayNameForFile(possibleApp.executable));
else
menuItem.setAttribute("label", possibleApp.name);
menuItem.setAttribute("image", this._getIconURLForHandlerApp(possibleApp));
// Attach the handler app object to the menu item so we can use it