Skip to content

Commit

Permalink
Remove usage of nsISupportsArray (#779)
Browse files Browse the repository at this point in the history
nsISupportsArray was deprecated a long time ago and finally removed in
Firefox 55 (see https://bugzilla.mozilla.org/show_bug.cgi?id=792209).

This should be backwards compatible, but was only tested against FF55.
  • Loading branch information
SammysHP authored and maxauthority committed Aug 14, 2017
1 parent eeae20b commit ae0df80
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions common/content/liberator.js
Original file line number Diff line number Diff line change
Expand Up @@ -846,14 +846,14 @@ const Liberator = Module("liberator", {

case liberator.NEW_PRIVATE_WINDOW:
case liberator.NEW_WINDOW:
let sa = Cc["@mozilla.org/supports-array;1"].createInstance(Ci.nsISupportsArray);
let sa = Cc["@mozilla.org/array;1"].createInstance(Ci.nsIMutableArray);
let wuri = Cc["@mozilla.org/supports-string;1"].createInstance(Ci.nsISupportsString);
wuri.data = url;
sa.AppendElement(wuri);
sa.AppendElement(null); // charset
sa.AppendElement(null); // referrerURI
sa.AppendElement(postdata);
sa.AppendElement(null); // allowThirdPartyFixup
sa.appendElement(wuri, false);
sa.appendElement(null, false); // charset
sa.appendElement(null, false); // referrerURI
sa.appendElement(postdata, false);
sa.appendElement(null, false); // allowThirdPartyFixup

let features = "chrome,dialog=no,all" + (where === liberator.NEW_PRIVATE_WINDOW ? ",private" : "");
let win = services.get("ww").openWindow(window, "chrome://browser/content/browser.xul", null, features, sa);
Expand Down

0 comments on commit ae0df80

Please sign in to comment.