From edf5abed265993690492dd2530fa731259caacfb Mon Sep 17 00:00:00 2001
From: Ryan Nowak <nowakra@gmail.com>
Date: Mon, 1 Jul 2019 17:43:30 -0700
Subject: [PATCH] Remove unwanted EventCallbackFactory overload

---
 ...soft.AspNetCore.Components.netstandard2.0.cs |  2 --
 .../Components/src/EventCallbackFactory.cs      | 17 -----------------
 .../Components/test/EventCallbackFactoryTest.cs | 14 --------------
 3 files changed, 33 deletions(-)

diff --git a/src/Components/Components/ref/Microsoft.AspNetCore.Components.netstandard2.0.cs b/src/Components/Components/ref/Microsoft.AspNetCore.Components.netstandard2.0.cs
index aeb2ea8b73..99aa532c10 100644
--- a/src/Components/Components/ref/Microsoft.AspNetCore.Components.netstandard2.0.cs
+++ b/src/Components/Components/ref/Microsoft.AspNetCore.Components.netstandard2.0.cs
@@ -143,8 +143,6 @@ namespace Microsoft.AspNetCore.Components
         public Microsoft.AspNetCore.Components.EventCallback<T> Create<T>(object receiver, System.Action<T> callback) { throw null; }
         public Microsoft.AspNetCore.Components.EventCallback<T> Create<T>(object receiver, System.Func<System.Threading.Tasks.Task> callback) { throw null; }
         public Microsoft.AspNetCore.Components.EventCallback<T> Create<T>(object receiver, System.Func<T, System.Threading.Tasks.Task> callback) { throw null; }
-        [System.ComponentModel.EditorBrowsableAttribute(System.ComponentModel.EditorBrowsableState.Never)]
-        public string Create<T>(object receiver, string callback) { throw null; }
     }
     public static partial class EventCallbackFactoryBinderExtensions
     {
diff --git a/src/Components/Components/src/EventCallbackFactory.cs b/src/Components/Components/src/EventCallbackFactory.cs
index a93b897ab5..a920d2277d 100644
--- a/src/Components/Components/src/EventCallbackFactory.cs
+++ b/src/Components/Components/src/EventCallbackFactory.cs
@@ -98,23 +98,6 @@ namespace Microsoft.AspNetCore.Components
             return CreateCore(receiver, callback);
         }
 
-        /// <summary>
-        /// Returns the provided <paramref name="callback"/>. For internal framework use only.
-        /// </summary>
-        /// <param name="receiver"></param>
-        /// <param name="callback"></param>
-        /// <returns></returns>
-        [EditorBrowsable(EditorBrowsableState.Never)]
-        public string Create<T>(object receiver, string callback)
-        {
-            if (receiver == null)
-            {
-                throw new ArgumentNullException(nameof(receiver));
-            }
-
-            return callback;
-        }
-
         /// <summary>
         /// Returns the provided <paramref name="callback"/>. For internal framework use only.
         /// </summary>
diff --git a/src/Components/Components/test/EventCallbackFactoryTest.cs b/src/Components/Components/test/EventCallbackFactoryTest.cs
index e88e74badd..714c73c656 100644
--- a/src/Components/Components/test/EventCallbackFactoryTest.cs
+++ b/src/Components/Components/test/EventCallbackFactoryTest.cs
@@ -296,20 +296,6 @@ namespace Microsoft.AspNetCore.Components
             Assert.True(callback.RequiresExplicitReceiver);
         }
 
-        [Fact]
-        public void CreateT_String_ReturnsInput()
-        {
-            // Arrange
-            var component = new EventComponent();
-            var input = "some_js";
-
-            // Act
-            var callback = EventCallback.Factory.Create<UIMouseEventArgs>(component, input);
-
-            // Assert
-            Assert.Same(input, callback);
-        }
-
         [Fact]
         public void CreateT_EventCallback_ReturnsInput()
         {