From c393f3394584ade99bfb62dafea240073f110fc8 Mon Sep 17 00:00:00 2001
From: "bzbarsky%mit.edu"
 <bzbarsky%mit.edu@18797224-902f-48f8-a5cc-f745e15eee43>
Date: Sun, 7 Oct 2007 18:23:52 +0000
Subject: [PATCH] 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
---
 mozilla/content/base/src/nsScriptLoader.cpp | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/mozilla/content/base/src/nsScriptLoader.cpp b/mozilla/content/base/src/nsScriptLoader.cpp
index 1bb68913f1c..7b7e306d8a6 100644
--- a/mozilla/content/base/src/nsScriptLoader.cpp
+++ b/mozilla/content/base/src/nsScriptLoader.cpp
@@ -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();