fix: remove redundant sorter renderer (#538)

Fixes: vaadin/vaadin-grid#1841.
Cherry-pick: #515

vaadin-grid-flow-parent/vaadin-grid-flow-integration-tests/src/main/java/com/vaadin/flow/component/grid/it/SortingPage.java

@@ -23,6 +23,7 @@
 import com.vaadin.flow.component.grid.GridSortOrderBuilder;
 import com.vaadin.flow.component.html.Div;
 import com.vaadin.flow.component.html.NativeButton;
+import com.vaadin.flow.component.html.Span;
 import com.vaadin.flow.data.bean.Person;
 import com.vaadin.flow.router.Route;
 
@@ -69,10 +70,14 @@ private Grid createGrid(String gridId, String sortBtnId) {
         grid.setItems(new Person("B", 20), new Person("A", 30));
 
         Column<Person> nameColumn = grid.addColumn(Person::getFirstName)
-                .setHeader("Name");
+                .setHeader(new Span("Name"));
         Column<Person> ageColumn = grid.addColumn(Person::getAge)
                 .setHeader("Age");
 
+        // Needed to check that sorter is rendered in component header after
+        // adding new header row
+        grid.appendHeaderRow();
+
         List<GridSortOrder<Person>> sortByName = new GridSortOrderBuilder<Person>()
                 .thenAsc(nameColumn).build();
         grid.sort(sortByName);

vaadin-grid-flow-parent/vaadin-grid-flow/src/main/java/com/vaadin/flow/component/grid/AbstractColumn.java

@@ -175,10 +175,7 @@ protected void setFooterText(String text) {
     }
 
     protected void setHeaderComponent(Component component) {
-        /*
-         * Uses the special renderer to take care of the vaadin-grid-sorter.
-         */
-        setHeaderRenderer(new GridSorterComponentRenderer<>(this, component));
+        setHeaderRenderer(new ComponentRenderer<>(() -> component));
     }
 
     protected void setFooterComponent(Component component) {