MD Settings: announce search results

chrome/browser/resources/settings/settings_main/settings_main.js

Index: chrome/browser/resources/settings/settings_main/settings_main.js
diff --git a/chrome/browser/resources/settings/settings_main/settings_main.js b/chrome/browser/resources/settings/settings_main/settings_main.js
index 22b4bd86fb2766ef7145bfac11d076b95fbf0639..402e0614864ccaae6410dfa9400c34dd70244320 100644
--- a/chrome/browser/resources/settings/settings_main/settings_main.js
+++ b/chrome/browser/resources/settings/settings_main/settings_main.js
@@ -267,6 +267,15 @@ Polymer({
           this.inSearchMode_ = !request.isSame('');

[dpapad, 2017/03/14 17:07:09]

Need to rebase this, after
https://codereview.chromium.org/2739323005/diff/120001/chrome/browser/resourc....

[Dan Beam, 2017/03/14 18:14:37]

Done.

           this.showNoResultsFound_ =
               this.inSearchMode_ && !request.didFindMatches();
+
+          if (this.inSearchMode_) {
+            Polymer.IronA11yAnnouncer.requestAvailability();
+            this.fire('iron-announce', {

[dpapad, 2017/03/14 17:07:09]

Do you also need { bubbles: true }, per the example at
https://www.webcomponents.org/element/PolymerElements/iron-a11y-announcer?

[Dan Beam, 2017/03/14 18:14:37]

no, it works without it

+              text: this.showNoResultsFound_ ?
+                  loadTimeData.getString('searchNoResults') :
+                  loadTimeData.getStringF('searchResults', query)
+            });
+          }
         }.bind(this));
       }.bind(this), 0);
     }.bind(this));