Bug 220215 use the XPCNativeWrapper for Components.lookupMethod r=caillon sr=bz
git-svn-id: svn://10.0.0.236/trunk@205793 18797224-902f-48f8-a5cc-f745e15eee43
This commit is contained in:
parent
27533cb344
commit
77ddce8ae6
@ -52,6 +52,7 @@
|
||||
</script>
|
||||
<script type="application/x-javascript" src="chrome://communicator/content/nsContextMenu.js"/>
|
||||
<script type="application/x-javascript" src="chrome://communicator/content/contentAreaUtils.js"/>
|
||||
<script type="application/x-javascript" src="chrome://communicator/content/XPCNativeWrapper.js"/>
|
||||
|
||||
<popupset id="contentAreaContextSet">
|
||||
<!-- This is a generic context menu for a content area. It contains
|
||||
|
||||
@ -45,8 +45,7 @@ function isContentFrame(aFocusedWindow)
|
||||
if (!aFocusedWindow)
|
||||
return false;
|
||||
|
||||
var focusedTop = Components.lookupMethod(aFocusedWindow, 'top')
|
||||
.call(aFocusedWindow);
|
||||
var focusedTop = new XPCNativeWrapper(aFocusedWindow, 'top').top;
|
||||
|
||||
return (focusedTop == window.content);
|
||||
}
|
||||
@ -69,7 +68,7 @@ function urlSecurityCheck(url, doc)
|
||||
function getContentFrameURI(aFocusedWindow)
|
||||
{
|
||||
var contentFrame = isContentFrame(aFocusedWindow) ? aFocusedWindow : window.content;
|
||||
return Components.lookupMethod(contentFrame, 'location').call(contentFrame).href;
|
||||
return new XPCNativeWrapper(contentFrame, "location").location.href;
|
||||
}
|
||||
|
||||
function getReferrer(doc)
|
||||
|
||||
@ -479,9 +479,9 @@ nsContextMenu.prototype = {
|
||||
// initialize popupURL
|
||||
const IOS = Components.classes["@mozilla.org/network/io-service;1"]
|
||||
.getService(CI.nsIIOService);
|
||||
var spec = Components.lookupMethod(window.content.opener, "location")
|
||||
.call();
|
||||
this.popupURL = IOS.newURI(spec, null, null);
|
||||
var opener = new XPCNativeWrapper(window.content, "opener").opener;
|
||||
var location = new XPCNativeWrapper(opener, "location").location;
|
||||
this.popupURL = IOS.newURI(location.href, null, null);
|
||||
|
||||
// but cancel if it's an unsuitable URL
|
||||
const PM = Components.classes["@mozilla.org/PopupWindowManager;1"]
|
||||
@ -675,8 +675,8 @@ nsContextMenu.prototype = {
|
||||
// Let's try to unescape it using a character set
|
||||
// in case the address is not ASCII.
|
||||
try {
|
||||
var characterSet = Components.lookupMethod(this.target.ownerDocument, "characterSet")
|
||||
.call(this.target.ownerDocument);
|
||||
var ownerDocument = new XPCNativeWrapper(this.target, "ownerDocument").ownerDocument;
|
||||
var characterSet = new XPCNativeWrapper(ownerDocument, "characterSet").characterSet;
|
||||
const textToSubURI = Components.classes["@mozilla.org/intl/texttosuburi;1"]
|
||||
.getService(Components.interfaces.nsITextToSubURI);
|
||||
addresses = textToSubURI.unEscapeNonAsciiURI(characterSet, addresses);
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user