Address @mdaigle's comments

Author: benrr101

src/Microsoft.Data.SqlClient/netcore/src/Microsoft/Data/SqlClient/SqlConnectionHelper.cs

@@ -156,9 +156,7 @@ override protected DbCommand CreateDbCommand()
         {
             using (TryEventScope.Create("<prov.DbConnectionHelper.CreateDbCommand|API> {0}", ObjectID))
             {
-                DbCommand command = null;
-                DbProviderFactory providerFactory = SqlConnectionFactory.ProviderFactory;
-                command = providerFactory.CreateCommand();
+                DbCommand command = SqlClientFactory.Instance.CreateCommand();
                 command.Connection = this;
                 return command;
             }

src/Microsoft.Data.SqlClient/netfx/src/Microsoft/Data/SqlClient/SqlConnectionHelper.cs

@@ -203,17 +203,16 @@ override protected DbCommand CreateDbCommand()
         {
             using (TryEventScope.Create("<prov.DbConnectionHelper.CreateDbCommand|API> {0}", ObjectID))
             {
-                DbProviderFactory providerFactory = SqlConnectionFactory.ProviderFactory;
-                DbCommand command = providerFactory.CreateCommand();
+                DbCommand command = SqlClientFactory.Instance.CreateCommand();
                 command.Connection = this;
                 return command;
             }
         }
 
         private static System.Security.CodeAccessPermission CreateExecutePermission()
         {
-            DBDataPermission p = (DBDataPermission)SqlConnectionFactory.ProviderFactory.CreatePermission(System.Security.Permissions.PermissionState.None);
-            p.Add(String.Empty, String.Empty, KeyRestrictionBehavior.AllowOnly);
+            DBDataPermission p = (DBDataPermission)SqlClientFactory.Instance.CreatePermission(System.Security.Permissions.PermissionState.None);
+            p.Add(string.Empty, string.Empty, KeyRestrictionBehavior.AllowOnly);
             return p;
         }
 

src/Microsoft.Data.SqlClient/src/Microsoft/Data/SqlClient/SqlConnectionFactory.cs

@@ -22,6 +22,7 @@
 
 namespace Microsoft.Data.SqlClient
 {
+    // @TODO: Facade pattern (interface, use interface, add constructor overloads for providing non-default interface, reseal)
     internal sealed class SqlConnectionFactory
     {
         #region Member Variables
@@ -67,8 +68,6 @@ private SqlConnectionFactory()
         #endregion
 
         #region Properties
-
-        internal static DbProviderFactory ProviderFactory => SqlClientFactory.Instance;
 
         internal static SqlConnectionFactory Instance { get; } = new SqlConnectionFactory();