mirror of
https://github.com/mozilla/gecko-dev.git
synced 2024-10-09 19:35:51 +00:00
Bug #36221 --> throbber never stops when loading a frameset doc.
I was signaling the stop notification via a OnchildStatus in this scenario. We should always signal the start and stop of documents as a status change even if a child is originating the change.
This commit is contained in:
parent
89115d7adc
commit
9671270568
@ -545,11 +545,7 @@ void nsDocLoaderImpl::FireOnStartDocumentLoad(nsDocLoaderImpl* aLoadInitiator,
|
||||
if (mProgressListener)
|
||||
{
|
||||
mProgressStatusFlags = nsIWebProgress::flag_net_start;
|
||||
|
||||
if (aLoadInitiator == this)
|
||||
mProgressListener->OnStatusChange(mDocumentChannel, mProgressStatusFlags);
|
||||
else // the load must be initiated by a child...mscott: I'm passing the WRONG channel here! I need to add a get channel to the doc loader interface
|
||||
mProgressListener->OnChildStatusChange(mDocumentChannel, mProgressStatusFlags);
|
||||
mProgressListener->OnStatusChange(mDocumentChannel, mProgressStatusFlags);
|
||||
}
|
||||
|
||||
/*
|
||||
@ -615,11 +611,7 @@ void nsDocLoaderImpl::FireOnEndDocumentLoad(nsDocLoaderImpl* aLoadInitiator,
|
||||
if (mProgressListener)
|
||||
{
|
||||
mProgressStatusFlags = nsIWebProgress::flag_net_stop;
|
||||
|
||||
if (aLoadInitiator == this)
|
||||
mProgressListener->OnStatusChange(aDocChannel, mProgressStatusFlags);
|
||||
else // the load must be initiated by a child...mscott: I'm passing the WRONG channel here! I need to add a get channel to the doc loader interface
|
||||
mProgressListener->OnChildStatusChange(aDocChannel, mProgressStatusFlags);
|
||||
mProgressListener->OnStatusChange(aDocChannel, mProgressStatusFlags);
|
||||
}
|
||||
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user