Remove use of `String.Split` in Environment taghelper

Fixes #3617

src/Microsoft.AspNet.Mvc.TagHelpers/EnvironmentTagHelper.cs

@@ -2,9 +2,9 @@
 // Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.
 
 using System;
-using System.Collections.Generic;
 using Microsoft.AspNet.Hosting;
 using Microsoft.AspNet.Razor.TagHelpers;
+using Microsoft.Extensions.Primitives;
 
 namespace Microsoft.AspNet.Mvc.TagHelpers
 {
@@ -14,7 +14,7 @@ namespace Microsoft.AspNet.Mvc.TagHelpers
     /// </summary>
     public class EnvironmentTagHelper : TagHelper
     {
-        private static readonly char[] NameSeparator = new[] { ',' };
+        private static readonly char NameSeparator = ',';
 
         /// <summary>
         /// Creates a new <see cref="EnvironmentTagHelper"/>.
@@ -26,13 +26,7 @@ namespace Microsoft.AspNet.Mvc.TagHelpers
         }
 
         /// <inheritdoc />
-        public override int Order
+        public override int Order => -1000;
-        {
-            get
-            {
-                return -1000;
-            }
-        }
 
         /// <summary>
         /// A comma separated list of environment names in which the content should be rendered.
@@ -74,36 +68,28 @@ namespace Microsoft.AspNet.Mvc.TagHelpers
                 return;
             }
 
-            var values = Names.Split(NameSeparator, StringSplitOptions.RemoveEmptyEntries);
+            var tokenizer = new StringTokenizer(Names, NameSeparator);
-            var environments = new List<string>();
+            var hasEnvironments = false;
-
+            foreach (var item in tokenizer)
-            for (var i = 0; i < values.Length; i++)
             {
-                var trimmedValue = values[i].Trim();
+                var environment = item.Trim();
-
+                if (environment.HasValue && environment.Length > 0)
-                if (trimmedValue.Length > 0)
                 {
-                    environments.Add(trimmedValue);
+                    hasEnvironments = true;
+                    if (environment.Equals(currentEnvironmentName, StringComparison.OrdinalIgnoreCase))
+                    {
+                        // Matching environment name found, do nothing
+                        return;
+                    }
                 }
             }
 
-            if (environments.Count == 0)
+            if (hasEnvironments)
             {
-                // Names contains only commas or empty entries, do nothing
+                // This instance had at least one non-empty environment specified but none of these
-                return;
+                // environments matched the current environment. Suppress the output in this case.
+                output.SuppressOutput();
             }
-
-            for (var i = 0; i < environments.Count; i++)
-            {
-                if (string.Equals(environments[i], currentEnvironmentName, StringComparison.OrdinalIgnoreCase))
-                {
-                    // Matching environment name found, do nothing
-                    return;
-                }
-            }
-
-            // No matching environment name found, suppress all output
-            output.SuppressOutput();
         }
     }
 }

src/Microsoft.AspNet.Mvc.TagHelpers/project.json

@@ -17,6 +17,7 @@
       "version": "1.0.0-*",
       "type": "build"
     },
+    "Microsoft.Extensions.Primitives": "1.0.0-*",
     "Microsoft.Extensions.PropertyHelper.Sources": {
       "version": "1.0.0-*",
       "type": "build"