Reoder some checks to do the fast one that will test false most often

first. Bug 398222, r+sr+a=sicking


git-svn-id: svn://10.0.0.236/trunk@237379 18797224-902f-48f8-a5cc-f745e15eee43
This commit is contained in:
bzbarsky%mit.edu
2007-10-07 18:23:52 +00:00
parent ea85959965
commit c393f33945

View File

@@ -488,7 +488,7 @@ nsScriptLoader::ProcessScriptElement(nsIScriptElement *aElement)
// If we've got existing pending requests, add ourselves
// to this list.
if (ReadyToExecuteScripts() && mPendingRequests.Count() == 0) {
if (mPendingRequests.Count() == 0 && ReadyToExecuteScripts()) {
return ProcessRequest(request);
}
}
@@ -654,13 +654,13 @@ void
nsScriptLoader::ProcessPendingRequests()
{
nsRefPtr<nsScriptLoadRequest> request;
while (ReadyToExecuteScripts() && mPendingRequests.Count() &&
while (mPendingRequests.Count() && ReadyToExecuteScripts() &&
!(request = mPendingRequests[0])->mLoading) {
mPendingRequests.RemoveObjectAt(0);
ProcessRequest(request);
}
while (ReadyToExecuteScripts() && !mPendingChildLoaders.IsEmpty()) {
while (!mPendingChildLoaders.IsEmpty() && ReadyToExecuteScripts()) {
nsRefPtr<nsScriptLoader> child = mPendingChildLoaders[0];
mPendingChildLoaders.RemoveElementAt(0);
child->RemoveExecuteBlocker();