From 2fbcbd1c7904e00eed4cf053a5777468b66d1d46 Mon Sep 17 00:00:00 2001
From: crisbeto <crisbeto@abv.bg>
Date: Sat, 16 Mar 2019 18:11:53 +0100
Subject: [PATCH] fix(slide-toggle): clear name from host node

Along the same lines as #15422 and #15368. Clears the static `name` attribute from the slide toggle's host node in order to prevent both the underlying input and the host from showing up in `document.getElementsByName` or `By.name`.
---
 src/lib/slide-toggle/slide-toggle.spec.ts | 6 ++++++
 src/lib/slide-toggle/slide-toggle.ts      | 1 +
 2 files changed, 7 insertions(+)

diff --git a/src/lib/slide-toggle/slide-toggle.spec.ts b/src/lib/slide-toggle/slide-toggle.spec.ts
index 4d6e881aed64..29e0f61fd014 100644
--- a/src/lib/slide-toggle/slide-toggle.spec.ts
+++ b/src/lib/slide-toggle/slide-toggle.spec.ts
@@ -1041,6 +1041,12 @@ describe('MatSlideToggle with forms', () => {
 
       expect(testComponent.isSubmitted).toBe(true);
     });
+
+    it('should clear static name attribute from the slide toggle host node', () => {
+      const hostNode = fixture.nativeElement.querySelector('.mat-slide-toggle');
+      expect(inputElement.getAttribute('name')).toBeTruthy();
+      expect(hostNode.hasAttribute('name')).toBe(false);
+    });
   });
 
   describe('with model and change event', () => {
diff --git a/src/lib/slide-toggle/slide-toggle.ts b/src/lib/slide-toggle/slide-toggle.ts
index df1fc7779e7a..6408b2c9e068 100644
--- a/src/lib/slide-toggle/slide-toggle.ts
+++ b/src/lib/slide-toggle/slide-toggle.ts
@@ -88,6 +88,7 @@ export const _MatSlideToggleMixinBase:
     '[id]': 'id',
     // Needs to be `-1` so it can still receive programmatic focus.
     '[attr.tabindex]': 'disabled ? null : -1',
+    '[attr.name]': 'null',
     '[class.mat-checked]': 'checked',
     '[class.mat-disabled]': 'disabled',
     '[class.mat-slide-toggle-label-before]': 'labelPosition == "before"',