From 3f8e7b60690c70a9378fef4ef45bfc4a79563ea0 Mon Sep 17 00:00:00 2001
From: "blakeross%telocity.com"
 <blakeross%telocity.com@18797224-902f-48f8-a5cc-f745e15eee43>
Date: Mon, 18 Feb 2002 22:58:54 +0000
Subject: [PATCH] 120838 - fix most of the 'theme switching doesn't work from
 prefs' bugs. r=ben sr=hewitt

git-svn-id: svn://10.0.0.236/trunk@114821 18797224-902f-48f8-a5cc-f745e15eee43
---
 mozilla/chrome/src/nsChromeRegistry.cpp     | 17 +++++++++--------
 mozilla/rdf/chrome/src/nsChromeRegistry.cpp | 17 +++++++++--------
 2 files changed, 18 insertions(+), 16 deletions(-)

diff --git a/mozilla/chrome/src/nsChromeRegistry.cpp b/mozilla/chrome/src/nsChromeRegistry.cpp
index c640970342d..09b440a4944 100644
--- a/mozilla/chrome/src/nsChromeRegistry.cpp
+++ b/mozilla/chrome/src/nsChromeRegistry.cpp
@@ -1862,7 +1862,6 @@ NS_IMETHODIMP nsChromeRegistry::IsProviderSelected(const nsCString& aProvider,
   resourceStr += aProvider;
   resourceStr += ":";
   resourceStr.AppendWithConversion(aProviderName);
-
   // Obtain the provider resource.
   nsresult rv = NS_OK;
   nsCOMPtr<nsIRDFResource> resource;
@@ -1896,6 +1895,8 @@ NS_IMETHODIMP nsChromeRegistry::IsProviderSelected(const nsCString& aProvider,
   // For each skin/package entry, follow the arcs to the real package
   // resource.
   PRBool more;
+  PRInt32 numSet = 0;
+  PRInt32 numPackages = 0;
   rv = arcs->HasMoreElements(&more);
   if (NS_FAILED(rv)) return rv;
   while (more) {
@@ -1921,19 +1922,19 @@ NS_IMETHODIMP nsChromeRegistry::IsProviderSelected(const nsCString& aProvider,
              NS_ERROR("Unable to set provider for package resource.");
              return rv;
            }
-           if (isSet && !*aResult)
-             *aResult = FULL;
-           else if (!isSet && *aResult) {
-             *aResult = PARTIAL;
-             return NS_OK;
-           }
+           ++numPackages;
+           if (isSet)
+             ++numSet;
          }
       }
     }
     rv = arcs->HasMoreElements(&more);
     if (NS_FAILED(rv)) return rv;
   }
-
+  if (numPackages == numSet)
+    *aResult = FULL;
+  else if (numSet)
+    *aResult = PARTIAL;
   return NS_OK;
 }
 
diff --git a/mozilla/rdf/chrome/src/nsChromeRegistry.cpp b/mozilla/rdf/chrome/src/nsChromeRegistry.cpp
index c640970342d..09b440a4944 100644
--- a/mozilla/rdf/chrome/src/nsChromeRegistry.cpp
+++ b/mozilla/rdf/chrome/src/nsChromeRegistry.cpp
@@ -1862,7 +1862,6 @@ NS_IMETHODIMP nsChromeRegistry::IsProviderSelected(const nsCString& aProvider,
   resourceStr += aProvider;
   resourceStr += ":";
   resourceStr.AppendWithConversion(aProviderName);
-
   // Obtain the provider resource.
   nsresult rv = NS_OK;
   nsCOMPtr<nsIRDFResource> resource;
@@ -1896,6 +1895,8 @@ NS_IMETHODIMP nsChromeRegistry::IsProviderSelected(const nsCString& aProvider,
   // For each skin/package entry, follow the arcs to the real package
   // resource.
   PRBool more;
+  PRInt32 numSet = 0;
+  PRInt32 numPackages = 0;
   rv = arcs->HasMoreElements(&more);
   if (NS_FAILED(rv)) return rv;
   while (more) {
@@ -1921,19 +1922,19 @@ NS_IMETHODIMP nsChromeRegistry::IsProviderSelected(const nsCString& aProvider,
              NS_ERROR("Unable to set provider for package resource.");
              return rv;
            }
-           if (isSet && !*aResult)
-             *aResult = FULL;
-           else if (!isSet && *aResult) {
-             *aResult = PARTIAL;
-             return NS_OK;
-           }
+           ++numPackages;
+           if (isSet)
+             ++numSet;
          }
       }
     }
     rv = arcs->HasMoreElements(&more);
     if (NS_FAILED(rv)) return rv;
   }
-
+  if (numPackages == numSet)
+    *aResult = FULL;
+  else if (numSet)
+    *aResult = PARTIAL;
   return NS_OK;
 }