mirror of
https://github.com/mozilla/gecko-dev.git
synced 2024-10-23 02:05:42 +00:00
bug 404222: remove unnecessary calls to getBrowserIndexForDocument to improve performance; r=sayrer, a=schrep
This commit is contained in:
parent
05908dbfb0
commit
5ec205635e
@ -128,17 +128,6 @@ function pageShowEventHandlers(event)
|
|||||||
|
|
||||||
XULBrowserWindow.asyncUpdateUI();
|
XULBrowserWindow.asyncUpdateUI();
|
||||||
}
|
}
|
||||||
|
|
||||||
// some event handlers want to be told what the original browser/listener is
|
|
||||||
var targetBrowser = null;
|
|
||||||
if (gBrowser.mTabbedMode) {
|
|
||||||
var targetBrowserIndex = gBrowser.getBrowserIndexForDocument(event.originalTarget);
|
|
||||||
if (targetBrowserIndex == -1)
|
|
||||||
return;
|
|
||||||
targetBrowser = gBrowser.getBrowserAtIndex(targetBrowserIndex);
|
|
||||||
} else {
|
|
||||||
targetBrowser = gBrowser.mCurrentBrowser;
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
@ -998,8 +987,6 @@ function prepareForStartup()
|
|||||||
|
|
||||||
// setup our common DOMLinkAdded listener
|
// setup our common DOMLinkAdded listener
|
||||||
gBrowser.addEventListener("DOMLinkAdded", DOMLinkHandler, false);
|
gBrowser.addEventListener("DOMLinkAdded", DOMLinkHandler, false);
|
||||||
|
|
||||||
gBrowser.addEventListener("pagehide", FeedHandler.onPageHide, false);
|
|
||||||
}
|
}
|
||||||
|
|
||||||
function delayedStartup()
|
function delayedStartup()
|
||||||
@ -5295,12 +5282,6 @@ function convertFromUnicode(charset, str)
|
|||||||
* and shows UI when they are discovered.
|
* and shows UI when they are discovered.
|
||||||
*/
|
*/
|
||||||
var FeedHandler = {
|
var FeedHandler = {
|
||||||
onPageHide: function(event) {
|
|
||||||
var theBrowser = gBrowser.getBrowserForDocument(event.target);
|
|
||||||
if (theBrowser)
|
|
||||||
theBrowser.feeds = null;
|
|
||||||
},
|
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* The click handler for the Feed icon in the location bar. Opens the
|
* The click handler for the Feed icon in the location bar. Opens the
|
||||||
* subscription page if user is not given a choice of feeds.
|
* subscription page if user is not given a choice of feeds.
|
||||||
|
@ -455,6 +455,8 @@
|
|||||||
this.pageReport = null;
|
this.pageReport = null;
|
||||||
this.updatePageReport();
|
this.updatePageReport();
|
||||||
}
|
}
|
||||||
|
if (this.feeds)
|
||||||
|
this.feeds = null;
|
||||||
if (!this.docShell || !this.fastFind)
|
if (!this.docShell || !this.fastFind)
|
||||||
return;
|
return;
|
||||||
var tabBrowser = this.getTabBrowser();
|
var tabBrowser = this.getTabBrowser();
|
||||||
|
Loading…
Reference in New Issue
Block a user