Re-enable web-source icons in Stable and Preview builds (#19137)

Disables a controversial part of #19044.

Refs #19075

(cherry picked from commit 7b84162)
Service-Card-Id: PVTI_lADOAF3p4s4AmhmQzgcqq7E
Service-Version: 1.22

Author: DHowett

src/cascadia/TerminalSettingsModel/CascadiaSettings.cpp

@@ -483,9 +483,14 @@ static bool _validateSingleMediaResource(std::wstring_view resource)
             return false;
         }
 
-        const auto scheme{ resourceUri.SchemeName() };
-        // Only file: URIs and ms-* URIs are permissible. http, https, ftp, gopher, etc. are not.
-        return til::equals_insensitive_ascii(scheme, L"file") || til::starts_with_insensitive_ascii(scheme, L"ms-");
+        if constexpr (Feature_DisableWebSourceIcons::IsEnabled())
+        {
+            const auto scheme{ resourceUri.SchemeName() };
+            // Only file: URIs and ms-* URIs are permissible. http, https, ftp, gopher, etc. are not.
+            return til::equals_insensitive_ascii(scheme, L"file") || til::starts_with_insensitive_ascii(scheme, L"ms-");
+        }
+
+        return true;
     }
     catch (...)
     {

src/features.xml

@@ -186,4 +186,15 @@
         <alwaysDisabledReleaseTokens/>
     </feature>
 
+    <feature>
+        <name>Feature_DisableWebSourceIcons</name>
+        <description>Disables icon paths that make web requests</description>
+        <id>19075</id>
+        <stage>AlwaysDisabled</stage>
+        <alwaysEnabledBrandingTokens>
+            <brandingToken>Dev</brandingToken>
+            <brandingToken>Canary</brandingToken>
+        </alwaysEnabledBrandingTokens>
+    </feature>
+
 </featureStaging>