Revert "fix(material/slider): some screen readers announcing long decimal values (#20760)" (#20849)

This reverts commit 19f5f5c.

Reverting this for two reasons:
* the truthiness check is a bit too broad, preventing someone
  from setting the empty string or zero
* I'd prefer to have the input named `valuetext` rather than `aria-valuetext`
  since it's potentially confusing without supporting the `attr.`
  (I know we do this elsewhere, but I'd like to avoid more use of
  this pattern)

Author: jelbourn

src/material/slider/slider.spec.ts

@@ -507,27 +507,6 @@ describe('MatSlider', () => {
       expect(sliderInstance.value).toBe(0.3);
     });
 
-    it('should set the truncated value to the aria-valuetext', () => {
-      fixture.componentInstance.step = 0.1;
-      fixture.detectChanges();
-
-      dispatchSlideEventSequence(sliderNativeElement, 0, 0.333333);
-      fixture.detectChanges();
-
-      expect(sliderNativeElement.getAttribute('aria-valuetext')).toBe('33');
-    });
-
-    it('should be able to override the aria-valuetext', () => {
-      fixture.componentInstance.step = 0.1;
-      fixture.componentInstance.ariaValuetext = 'custom';
-      fixture.detectChanges();
-
-      dispatchSlideEventSequence(sliderNativeElement, 0, 0.333333);
-      fixture.detectChanges();
-
-      expect(sliderNativeElement.getAttribute('aria-valuetext')).toBe('custom');
-    });
-
   });
 
   describe('slider with auto ticks', () => {
@@ -1515,12 +1494,11 @@ class SliderWithMinAndMax {
 class SliderWithValue { }
 
 @Component({
-  template: `<mat-slider [step]="step" [aria-valuetext]="ariaValuetext"></mat-slider>`,
+  template: `<mat-slider [step]="step"></mat-slider>`,
   styles: [styles],
 })
 class SliderWithStep {
   step = 25;
-  ariaValuetext: string;
 }
 
 @Component({

src/material/slider/slider.ts

@@ -134,13 +134,6 @@ const _MatSliderMixinBase:
     '[attr.aria-valuemax]': 'max',
     '[attr.aria-valuemin]': 'min',
     '[attr.aria-valuenow]': 'value',
-
-    // NVDA and Jaws appear to announce the `aria-valuenow` by calculating its percentage based
-    // on its value between `aria-valuemin` and `aria-valuemax`. Due to how decimals are handled,
-    // it can cause the slider to read out a very long value like 0.20000068 if the current value
-    // is 0.2 with a min of 0 and max of 1. We work around the issue by setting `aria-valuetext`
-    // to the same value that we set on the slider's thumb which will be truncated.
-    '[attr.aria-valuetext]': 'ariaValuetext || displayValue',
     '[attr.aria-orientation]': 'vertical ? "vertical" : "horizontal"',
     '[class.mat-slider-disabled]': 'disabled',
     '[class.mat-slider-has-ticks]': 'tickInterval',
@@ -275,9 +268,6 @@ export class MatSlider extends _MatSliderMixinBase
    */
   @Input() displayWith: (value: number) => string | number;
 
-  /** Text corresponding to the slider's value. */
-  @Input('aria-valuetext') ariaValuetext: string;
-
   /** Whether the slider is vertical. */
   @Input()
   get vertical(): boolean { return this._vertical; }

tools/public_api_guard/material/slider.d.ts

@@ -5,7 +5,6 @@ export declare class MatSlider extends _MatSliderMixinBase implements ControlVal
     protected _document: Document;
     _isActive: boolean;
     _isSliding: boolean;
-    ariaValuetext: string;
     readonly change: EventEmitter<MatSliderChange>;
     get displayValue(): string | number;
     displayWith: (value: number) => string | number;
@@ -72,7 +71,7 @@ export declare class MatSlider extends _MatSliderMixinBase implements ControlVal
     static ngAcceptInputType_tickInterval: NumberInput;
     static ngAcceptInputType_value: NumberInput;
     static ngAcceptInputType_vertical: BooleanInput;
-    static ɵcmp: i0.ɵɵComponentDefWithMeta<MatSlider, "mat-slider", ["matSlider"], { "disabled": "disabled"; "color": "color"; "tabIndex": "tabIndex"; "invert": "invert"; "max": "max"; "min": "min"; "step": "step"; "thumbLabel": "thumbLabel"; "tickInterval": "tickInterval"; "value": "value"; "displayWith": "displayWith"; "ariaValuetext": "aria-valuetext"; "vertical": "vertical"; }, { "change": "change"; "input": "input"; "valueChange": "valueChange"; }, never, never>;
+    static ɵcmp: i0.ɵɵComponentDefWithMeta<MatSlider, "mat-slider", ["matSlider"], { "disabled": "disabled"; "color": "color"; "tabIndex": "tabIndex"; "invert": "invert"; "max": "max"; "min": "min"; "step": "step"; "thumbLabel": "thumbLabel"; "tickInterval": "tickInterval"; "value": "value"; "displayWith": "displayWith"; "vertical": "vertical"; }, { "change": "change"; "input": "input"; "valueChange": "valueChange"; }, never, never>;
     static ɵfac: i0.ɵɵFactoryDef<MatSlider, [null, null, null, { optional: true; }, { attribute: "tabindex"; }, null, null, { optional: true; }]>;
 }