Bug 1488908 - QI the elements of nsIGIOservice.getAppsForURIScheme() to nsIHandlerApp. r=kmag a=pascalc DEVEDITION_63_0b7_BUILD1 DEVEDITION_63_0b7_RELEASE FENNEC_63_0b7_BUILD1 FENNEC_63_0b7_RELEASE FIREFOX_63_0b7_BUILD1 FIREFOX_63_0b7_RELEASE
authorBotond Ballo <botond@mozilla.com>
Fri, 14 Sep 2018 17:59:28 +0000
changeset 492519 503cd479184519f50341493317e1aacf0d0d35ec
parent 492518 03508cc9a42b0114e1c0d9ad4cd145e42fe9b6df
child 492520 f7718b03f0a02a9eb57d03bf0b763c7377173c3f
push id1815
push userffxbld-merge
push dateMon, 15 Oct 2018 10:40:45 +0000
treeherdermozilla-release@18d4c09e9378 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerskmag, pascalc
bugs1488908
milestone63.0
Bug 1488908 - QI the elements of nsIGIOservice.getAppsForURIScheme() to nsIHandlerApp. r=kmag a=pascalc Differential Revision: https://phabricator.services.mozilla.com/D5886
browser/components/preferences/in-content/main.js
--- a/browser/components/preferences/in-content/main.js
+++ b/browser/components/preferences/in-content/main.js
@@ -1733,17 +1733,17 @@ var gMainPane = {
       possibleAppMenuItems.push(menuItem);
     }
     // Add gio handlers
     if (Cc["@mozilla.org/gio-service;1"]) {
       let gIOSvc = Cc["@mozilla.org/gio-service;1"].
                    getService(Ci.nsIGIOService);
       var gioApps = gIOSvc.getAppsForURIScheme(handlerInfo.type);
       let possibleHandlers = handlerInfo.possibleApplicationHandlers;
-      for (let handler of gioApps.enumerate()) {
+      for (let handler of gioApps.enumerate(Ci.nsIHandlerApp)) {
         // OS handler share the same name, it's most likely the same app, skipping...
         if (handler.name == handlerInfo.defaultDescription) {
           continue;
         }
         // Check if the handler is already in possibleHandlers
         let appAlreadyInHandlers = false;
         for (let i = possibleHandlers.length - 1; i >= 0; --i) {
           let app = possibleHandlers.queryElementAt(i, Ci.nsIHandlerApp);