MD Settings: announce search results

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

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 /**
  * @typedef {{about: boolean, settings: boolean}}
  */
 var MainPageVisibility;
 
 /**
@@ skipping 246 matching lines @@
         whenSearchDone.then(function(request) {
           resolve();
           if (!request.finished) {
             // Nothing to do here. A previous search request was canceled
             // because a new search request was issued before the first one
             // completed.
             return;
           }
 
           this.toolbarSpinnerActive = false;
           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));
   },
 });