From 1fc14a0c5eac061fd42dbd1c4ba2437aa9125c6f Mon Sep 17 00:00:00 2001
From: tinayuangao <tinagao@google.com>
Date: Mon, 4 Jun 2018 12:01:05 -0700
Subject: [PATCH] Revert "fix(tabs): Add role to mat-tab-nav-bar and
 mat-tab-link" (#11657)

Material nav-tabs intentionally behaves as a nav + anchor
---
 src/lib/tabs/tab-nav-bar/tab-nav-bar.html    |  3 ++-
 src/lib/tabs/tab-nav-bar/tab-nav-bar.spec.ts | 20 --------------------
 src/lib/tabs/tab-nav-bar/tab-nav-bar.ts      |  1 -
 3 files changed, 2 insertions(+), 22 deletions(-)

diff --git a/src/lib/tabs/tab-nav-bar/tab-nav-bar.html b/src/lib/tabs/tab-nav-bar/tab-nav-bar.html
index ccff5b6f8b9a..d842ff7060db 100644
--- a/src/lib/tabs/tab-nav-bar/tab-nav-bar.html
+++ b/src/lib/tabs/tab-nav-bar/tab-nav-bar.html
@@ -1,4 +1,5 @@
-<div class="mat-tab-links" (cdkObserveContent)="_alignInkBar()" role="tablist">
+<div class="mat-tab-links" (cdkObserveContent)="_alignInkBar()">
   <ng-content></ng-content>
   <mat-ink-bar></mat-ink-bar>
 </div>
+
diff --git a/src/lib/tabs/tab-nav-bar/tab-nav-bar.spec.ts b/src/lib/tabs/tab-nav-bar/tab-nav-bar.spec.ts
index 13b1cef645fc..b37e6a3fe71b 100644
--- a/src/lib/tabs/tab-nav-bar/tab-nav-bar.spec.ts
+++ b/src/lib/tabs/tab-nav-bar/tab-nav-bar.spec.ts
@@ -16,7 +16,6 @@ describe('MatTabNavBar', () => {
       imports: [MatTabsModule],
       declarations: [
         SimpleTabNavBarTestApp,
-        TabLink,
         TabLinkWithNgIf,
         TabLinkWithTabIndexBinding,
         TabLinkWithNativeTabindexAttr,
@@ -274,16 +273,6 @@ describe('MatTabNavBar', () => {
 
     expect(tabLink.tabIndex).toBe(3, 'Expected the tabIndex to be have been set to 3.');
   });
-
-  it('should set role on tablist and tab', () => {
-    const fixture = TestBed.createComponent(TabLink);
-    fixture.detectChanges();
-
-    const tabList = fixture.debugElement.query(By.css('.mat-tab-links'));
-    expect(tabList.nativeElement.getAttribute('role')).toEqual('tablist');
-    const tabLinkElement = tabList.query(By.directive(MatTabLink)).nativeElement;
-    expect(tabLinkElement.getAttribute('role')).toEqual('tab');
-  });
 });
 
 @Component({
@@ -312,15 +301,6 @@ class SimpleTabNavBarTestApp {
   activeIndex = 0;
 }
 
-@Component({
-  template: `
-    <nav mat-tab-nav-bar>
-      <a mat-tab-link role="willbeoverridden">Link</a>
-    </nav>
-  `
-})
-class TabLink {}
-
 @Component({
   template: `
     <nav mat-tab-nav-bar>
diff --git a/src/lib/tabs/tab-nav-bar/tab-nav-bar.ts b/src/lib/tabs/tab-nav-bar/tab-nav-bar.ts
index 665935dd9ea3..918b061370f6 100644
--- a/src/lib/tabs/tab-nav-bar/tab-nav-bar.ts
+++ b/src/lib/tabs/tab-nav-bar/tab-nav-bar.ts
@@ -172,7 +172,6 @@ export const _MatTabLinkMixinBase =
   inputs: ['disabled', 'disableRipple', 'tabIndex'],
   host: {
     'class': 'mat-tab-link',
-    'role': 'tab',
     '[attr.aria-disabled]': 'disabled.toString()',
     '[attr.tabIndex]': 'tabIndex',
     '[class.mat-tab-disabled]': 'disabled',