Revert "Add spoken feedback for ejecting a volume"

This reverts commit 24d7413c.

Reason for revert:

Findit (https://goo.gl/kROfz5) identified CL at revision 610526 as the
culprit for flakes in the build cycles as shown on:
https://findit-for-me.appspot.com/waterfall/flake/flake-culprit?key=ag9zfmZpbmRpdC1mb3ItbWVyQwsSDEZsYWtlQ3VscHJpdCIxY2hyb21pdW0vMjRkNzQxM2NkYmRhOGVhOTQ5NzBlYmE3NDhmZmQxYzExNDMzYzVmMww

Sample Failed Build: https://ci.chromium.org/buildbot/chromium.chromiumos/linux-chromeos-rel/16380

Sample Failed Step: viz_browser_tests

Sample Flaky Test: Providers/FilesAppBrowserTest.Test/providerEject

Original change's description:
> Add spoken feedback for ejecting a volume
> 
> Add a message to screen reader once a volume has been ejected, it uses
> |speakA11yMessage| function introduced on crrev.com/c/1347956.
> 
> --gtest_filter="Providers/FilesAppBrowserTest.Test/providerEject"
> 
> Test: browser_test
> Bug: 758638
> Change-Id: I16e20f828c05368a41046d71fcfbcf04cfb212f4
> Reviewed-on: https://chromium-review.googlesource.com/c/1348880
> Reviewed-by: Anand Mistry <amistry@chromium.org>
> Commit-Queue: Luciano Pacheco <lucmult@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#610526}

Change-Id: Ifbc16215312062cd1593f497d96f1bddde0b2745
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 758638, 851904
Reviewed-on: https://chromium-review.googlesource.com/c/1348737
Cr-Commit-Position: refs/heads/master@{#610540}

chrome/app/file_manager_strings.grdp

@@ -1271,9 +1271,6 @@
   <message name="IDS_FILE_BROWSER_OPEN_A11Y_PLURAL" desc="Text to be used by screen reader to indicate users that the opening multiple files is being executed.">
     Opening <ph name="NUMBER_FILES"> $1<ex>3</ex> files.</ph>
   </message>
-  <message name="IDS_FILE_BROWSER_A11Y_VOLUME_EJECT" desc="Text to be used by screen reader to indicate users that the eject volume action has finished.">
-    <ph name="VOLUME_NAME">$1<ex>file_name.rar</ex> has been ejected.</ph>
-  </message>
 
 <!-- Common for Audio player and Media player -->
   <message name="IDS_MEDIA_PLAYER_PLAY_BUTTON_LABEL" desc="Label for the Play button of media players (audio player / video player).">

chrome/app/file_manager_strings_grdp/IDS_FILE_BROWSER_A11Y_VOLUME_EJECT.png.sha1

-04111c75cd9cf7f0ef9310dbeff0c51ffbaf90f5
\ No newline at end of file

chrome/browser/chromeos/extensions/file_manager/private_api_strings.cc

@@ -831,7 +831,6 @@ ExtensionFunction::ResponseAction FileManagerPrivateGetStringsFunction::Run() {
   SET_STRING("SEE_MENU_FOR_ACTIONS", IDS_FILE_BROWSER_SEE_MENU_FOR_ACTIONS);
   SET_STRING("OPEN_A11Y", IDS_FILE_BROWSER_OPEN_A11Y);
   SET_STRING("OPEN_A11Y_PLURAL", IDS_FILE_BROWSER_OPEN_A11Y_PLURAL);
-  SET_STRING("A11Y_VOLUME_EJECT", IDS_FILE_BROWSER_A11Y_VOLUME_EJECT);
   SET_STRING("FILEMANAGER_APP_NAME", IDS_FILEMANAGER_APP_NAME);
 #undef SET_STRING
 

chrome/browser/chromeos/file_manager/file_manager_browsertest.cc

@@ -734,8 +734,7 @@ WRAPPED_INSTANTIATE_TEST_CASE_P(
     ::testing::Values(TestCase("requestMount"),
                       TestCase("requestMountMultipleMounts"),
                       TestCase("requestMountSourceDevice"),
-                      TestCase("requestMountSourceFile"),
-                      TestCase("providerEject")));
+                      TestCase("requestMountSourceFile")));
 
 WRAPPED_INSTANTIATE_TEST_CASE_P(
     GearMenu, /* gear_menu.js */

ui/file_manager/file_manager/foreground/js/file_manager_commands.js

@@ -530,10 +530,7 @@ CommandHandler.COMMANDS_['unmount'] = /** @type {Command} */ ({
       return;
     }
 
-    const label = volumeInfo.label || '';
-    fileManager.volumeManager.unmount(volumeInfo, () => {
-      const msg = strf('A11Y_VOLUME_EJECT', label);
-      fileManager.ui.speakA11yMessage(msg);
+    fileManager.volumeManager.unmount(volumeInfo, function() {
     }, errorCallback.bind(null, volumeInfo.volumeType));
   },
   /**

ui/file_manager/integration_tests/file_manager/providers.js

@@ -271,43 +271,4 @@ testcase.requestMountSourceFile = function() {
   requestMountNotInMenuInternal('manifest_source_file.json');
 };
 
-/**
- * Tests that pressing the eject button on a FSP adds a message to screen
- * reader.
- */
-testcase.providerEject = function() {
-  const manifest = 'manifest_source_file.json';
-  const steps = getSetupSteps(manifest).concat([
-    // Click to eject Test (1) provider/volume.
-    function() {
-      const ejectQuery =
-          ['#directory-tree [volume-type-for-testing="provided"] .root-eject'];
-      remoteCall.callRemoteTestUtil(
-          'fakeMouseClick', appId, ejectQuery, this.next);
-    },
-    // Wait a11y-msg to have some text.
-    function(result) {
-      chrome.test.assertTrue(result, 'click eject failed');
-      remoteCall.waitForElement(appId, '#a11y-msg:not(:empty)').then(this.next);
-    },
-    // Fetch A11y messages.
-    function() {
-      remoteCall.callRemoteTestUtil('getA11yAnnounces', appId, [])
-          .then(this.next);
-    },
-    // Check that opening the file was announced to screen reader.
-    function(a11yMessages) {
-      chrome.test.assertTrue(a11yMessages instanceof Array);
-      chrome.test.assertEq(1, a11yMessages.length);
-      chrome.test.assertEq('Test (1) has been ejected.', a11yMessages[0]);
-      this.next();
-    },
-    function() {
-      checkIfNoErrorsOccured(this.next);
-    }
-  ]);
-
-  StepsRunner.run(steps);
-};
-
 })();