With the fix to bug # 56062 and 82236, the semantics of this
notification has slightly changed. Until now, onLocationChange() was
called to notify of a new urlload in the root docshell. Therefore most
of the clients of this call, used it to update the location bar
(urlbar) with the new url. But with the fixes to the above bugs,
onLocationChange() will *also* be called when ever a new url is loaded
in a subframe. Therefore, if you used onLocationChange() to update a
urlbar, you will notice that the urlbar is updated for
subframenavigations too, (which is not a preferable behavior for most
applications). You can avoid this annoying behavior by using a simple
code snippet like below:
onLocationChange : function(aWebProgress, aRequest, aLocation)
{
........
domWindow = aWebProgress.DOMWindow;
// Update urlbar only if a new page was loaded on the primary
content area
if (domWindow == domWindow.top) {
// This is a load to the root docshell
this.urlBar.value = location;
}
// Update other things like Back/forward buttons
.........
}
You have been informed!
Radha
