fix(material/chips): remove tabindex

remove tabindex

fixes b/286286473

Author: DBowen33

src/material/chips/chip-option.html

@@ -3,7 +3,6 @@
 <span class="mdc-evolution-chip__cell mdc-evolution-chip__cell--primary">
   <button
     matChipAction
-    [tabIndex]="tabIndex"
     [_allowFocusWhenDisabled]="true"
     [attr.aria-selected]="ariaSelected"
     [attr.aria-label]="ariaLabel"

src/material/chips/chip-row.html

@@ -4,7 +4,6 @@
 
 <span class="mdc-evolution-chip__cell mdc-evolution-chip__cell--primary" role="gridcell"
     matChipAction
-    [tabIndex]="tabIndex"
     [disabled]="disabled"
     [attr.aria-label]="ariaLabel"
     [attr.aria-describedby]="_ariaDescriptionId">

src/material/chips/chip-row.ts

@@ -125,7 +125,6 @@ export class MatChipRow extends MatChip implements AfterViewInit {
       _document,
       animationMode,
       globalRippleOptions,
-      tabIndex,
     );
 
     this.role = 'row';

src/material/chips/chip.ts

@@ -36,7 +36,6 @@ import {
   afterNextRender,
   booleanAttribute,
   inject,
-  numberAttribute,
 } from '@angular/core';
 import {
   MAT_RIPPLE_GLOBAL_OPTIONS,
@@ -85,7 +84,6 @@ export interface MatChipEvent {
     '[class._mat-animation-noopable]': '_animationsDisabled',
     '[id]': 'id',
     '[attr.role]': 'role',
-    '[attr.tabindex]': '_getTabIndex()',
     '[attr.aria-label]': 'ariaLabel',
     '[attr.aria-hidden]': 'true',
     '(keydown)': '_handleKeydown($event)',
@@ -202,12 +200,6 @@ export class MatChip implements OnInit, AfterViewInit, AfterContentInit, DoCheck
   @Input({transform: booleanAttribute})
   disabled: boolean = false;
 
-  /** Tab index of the chip. */
-  @Input({
-    transform: (value: unknown) => (value == null ? undefined : numberAttribute(value)),
-  })
-  tabIndex: number = -1;
-
   /** Emitted when a chip is to be removed. */
   @Output() readonly removed: EventEmitter<MatChipEvent> = new EventEmitter<MatChipEvent>();
 
@@ -259,13 +251,9 @@ export class MatChip implements OnInit, AfterViewInit, AfterContentInit, DoCheck
     @Optional()
     @Inject(MAT_RIPPLE_GLOBAL_OPTIONS)
     private _globalRippleOptions?: RippleGlobalOptions,
-    @Attribute('tabindex') tabIndex?: string,
   ) {
     this._document = _document;
     this._animationsDisabled = animationMode === 'NoopAnimations';
-    if (tabIndex != null) {
-      this.tabIndex = parseInt(tabIndex) ?? -1;
-    }
     this._monitorFocus();
 
     this._rippleLoader?.configureRipple(this._elementRef.nativeElement, {
@@ -397,14 +385,6 @@ export class MatChip implements OnInit, AfterViewInit, AfterContentInit, DoCheck
     // Empty here, but is overwritten in child classes.
   }
 
-  /** Gets the tabindex of the chip. */
-  _getTabIndex() {
-    if (!this.role) {
-      return null;
-    }
-    return this.disabled ? -1 : this.tabIndex;
-  }
-
   /** Starts the focus monitoring process on the chip. */
   private _monitorFocus() {
     this._focusMonitor.monitor(this._elementRef, true).subscribe(origin => {