Bug 825627, part 2: Convert known-to-be-nonnegative variables 'i' and 'idx' to be unsigned, in Classifier::ReadNoiseEntries(), to fix signed/unsigned comparison build warnings. r=gcp

toolkit/components/url-classifier/Classifier.cpp

@@ -790,7 +790,7 @@ Classifier::ReadNoiseEntries(const Prefix& aPrefix,
   nsresult rv = cache->GetPrefixes(&prefixes);
   NS_ENSURE_SUCCESS(rv, rv);
 
-  int32_t idx = prefixes.BinaryIndexOf(aPrefix.ToUint32());
+  uint32_t idx = prefixes.BinaryIndexOf(aPrefix.ToUint32());
 
   if (idx == nsTArray<uint32_t>::NoIndex) {
     NS_WARNING("Could not find prefix in PrefixSet during noise lookup");
@@ -799,7 +799,7 @@ Classifier::ReadNoiseEntries(const Prefix& aPrefix,
 
   idx -= idx % aCount;
 
-  for (int32_t i = 0; (i < aCount) && ((idx+i) < prefixes.Length()); i++) {
+  for (uint32_t i = 0; (i < aCount) && ((idx+i) < prefixes.Length()); i++) {
     Prefix newPref;
     newPref.FromUint32(prefixes[idx+i]);
     if (newPref != aPrefix) {
@@ -810,5 +810,5 @@ Classifier::ReadNoiseEntries(const Prefix& aPrefix,
   return NS_OK;
 }
 
-}
-}
+} // namespace safebrowsing
+} // namespace mozilla