Bug 1454956 - change scheme loop to use a range-based for loop, r?nwgh draft
authorGijs Kruitbosch <gijskruitbosch@gmail.com>
Thu, 19 Apr 2018 16:13:00 +0100
changeset 785157 0079cfc102332c77bf099a494fb8f209094a188d
parent 784477 6480454995dae4a44c86887f4ae01dc998edb36f
push id107159
push userbmo:gijskruitbosch+bugs@gmail.com
push dateThu, 19 Apr 2018 18:43:47 +0000
reviewersnwgh
bugs1454956
milestone61.0a1
Bug 1454956 - change scheme loop to use a range-based for loop, r?nwgh MozReview-Commit-ID: C0a6urRjBzn
netwerk/base/nsIOService.cpp
--- a/netwerk/base/nsIOService.cpp
+++ b/netwerk/base/nsIOService.cpp
@@ -517,18 +517,18 @@ UsesExternalProtocolHandler(const char* 
         NS_LITERAL_CSTRING("chrome").Equals(aScheme) ||
         NS_LITERAL_CSTRING("resource").Equals(aScheme)) {
         // Don't allow file:, chrome: or resource: URIs to be handled with
         // nsExternalProtocolHandler, since internally we rely on being able to
         // use and read from these URIs.
         return false;
     }
 
-    for (unsigned int i = 0; i < NS_N(gForcedExternalSchemes); i++) {
-      if (!nsCRT::strcasecmp(gForcedExternalSchemes[i], aScheme)) {
+    for (const auto & forcedExternalScheme : gForcedExternalSchemes) {
+      if (!nsCRT::strcasecmp(forcedExternalScheme, aScheme)) {
         return true;
       }
     }
 
     nsAutoCString pref("network.protocol-handler.external.");
     pref += aScheme;
 
     return Preferences::GetBool(pref.get(), false);