From 5323073c13e3cc3c14aa77abb9ff56926d13d9b4 Mon Sep 17 00:00:00 2001 From: Brian Nicholson Date: Mon, 23 Apr 2012 10:34:14 -0400 Subject: [PATCH] Bug 746380 - Use URI of nsIChannel on state change. r=mfinkle a=blocking-fennec --- mobile/android/chrome/content/browser.js | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/mobile/android/chrome/content/browser.js b/mobile/android/chrome/content/browser.js index d74a8976bc4c..84a237dd9635 100644 --- a/mobile/android/chrome/content/browser.js +++ b/mobile/android/chrome/content/browser.js @@ -2044,11 +2044,6 @@ Tab.prototype = { return; } - let browser = BrowserApp.getBrowserForWindow(aWebProgress.DOMWindow); - let uri = ""; - if (browser) - uri = browser.currentURI.spec; - // Check to see if we restoring the content from a previous presentation (session) // since there should be no real network activity let restoring = aStateFlags & Ci.nsIWebProgressListener.STATE_RESTORING; @@ -2056,6 +2051,10 @@ Tab.prototype = { // true if the page loaded successfully (i.e., no 404s or other errors) let success = false; + let uri = ""; + try { + uri = aRequest.QueryInterface(Components.interfaces.nsIChannel).originalURI.spec; + } catch (e) { } try { success = aRequest.QueryInterface(Components.interfaces.nsIHttpChannel).requestSucceeded; } catch (e) { }