From 7ee672da11fb6d0ca6e0e12ff330944ab5de9501 Mon Sep 17 00:00:00 2001 From: "jst%mozilla.jstenback.com" Date: Sat, 29 Jul 2006 05:43:18 +0000 Subject: [PATCH] Fxing bug 249332. Don't use getters/setters for link.href in browser code. r=caillon@aillon.org, sr=dveditz@cruzio.com git-svn-id: svn://10.0.0.236/trunk@205802 18797224-902f-48f8-a5cc-f745e15eee43 --- mozilla/suite/common/contentAreaClick.js | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/mozilla/suite/common/contentAreaClick.js b/mozilla/suite/common/contentAreaClick.js index 0c9ef4b3f62..b08d17d71ac 100644 --- a/mozilla/suite/common/contentAreaClick.js +++ b/mozilla/suite/common/contentAreaClick.js @@ -150,19 +150,21 @@ } var href; if (linkNode) { - href = linkNode.href; + href = new XPCNativeWrapper(linkNode, "href").href; } else { // Try simple XLink linkNode = target; while (linkNode) { if (linkNode.nodeType == Node.ELEMENT_NODE) { - href = linkNode.getAttributeNS("http://www.w3.org/1999/xlink", "href"); + var wrapper = new XPCNativeWrapper(linkNode, "getAttributeNS()"); + href = wrapper.getAttributeNS("http://www.w3.org/1999/xlink", "href"); break; } linkNode = linkNode.parentNode; } if (href && href != "") { - href = makeURLAbsolute(target.baseURI,href); + var baseURI = new XPCNativeWrapper(linkNode, "baseURI").baseURI; + href = makeURLAbsolute(baseURI, href); } } return href;