Bug 789469 - Get the executable path directly from the directory service instead of guesswork in WindowsJumpLists.jsm. r=jimm
authorMike Hommey <mh+mozilla@glandium.org>
Fri, 14 Dec 2012 18:40:20 +0100
changeset 125185 b8d79efc85416374a5795c323ed36c73d12dcabf
parent 125184 627a842914d0132d9177a522e0c9708e6ee27ae7
child 125186 3b334850ce94ca4dc7bba485f77e4ba9e96102a3
push id2151
push userlsblakk@mozilla.com
push dateTue, 19 Feb 2013 18:06:57 +0000
treeherdermozilla-beta@4952e88741ec [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjimm
bugs789469
milestone20.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 789469 - Get the executable path directly from the directory service instead of guesswork in WindowsJumpLists.jsm. r=jimm
browser/modules/WindowsJumpLists.jsm
--- a/browser/modules/WindowsJumpLists.jsm
+++ b/browser/modules/WindowsJumpLists.jsm
@@ -418,20 +418,17 @@ this.WinTaskbarJumpList =
 
   /**
    * Jump list item creation helpers
    */
 
   _getHandlerAppItem: function WTBJL__getHandlerAppItem(name, description, 
                                                         args, iconIndex, 
                                                         faviconPageUri) {
-    var file = Services.dirsvc.get("XCurProcD", Ci.nsILocalFile);
-
-    // XXX where can we grab this from in the build? Do we need to?
-    file.append("firefox.exe");
+    var file = Services.dirsvc.get("XREExeF", Ci.nsILocalFile);
 
     var handlerApp = Cc["@mozilla.org/uriloader/local-handler-app;1"].
                      createInstance(Ci.nsILocalHandlerApp);
     handlerApp.executable = file;
     // handlers default to the leaf name if a name is not specified
     if (name && name.length != 0)
       handlerApp.name = name;
     handlerApp.detailedDescription = description;