MD Settings: omnibox extensions list to use focus-row-behavior.

chrome/browser/resources/settings/search_engines_page/search_engines_page.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.
 
 /**
  * @fileoverview 'settings-search-engines-page' is the settings page
  * containing search engines settings.
  */
 Polymer({
   is: 'settings-search-engines-page',
@@ skipping 28 matching lines @@
       value: settings.Route.SEARCH_ENGINES,
     },
 
     /** @private {boolean} */
     showAddSearchEngineDialog_: Boolean,
 
     /** @private {boolean} */
     showExtensionsList_: {
       type: Boolean,
       computed: 'computeShowExtensionsList_(extensions)',
-    }
+    },
+
+    /** @private {Object}*/

[dpapad, 2017/05/04 16:35:32]

What is the type of this member variable? Can we do something better than just
Object (maybe Element or HTMLElement)?

Also, space missing after "}"

[scottchen, 2017/05/04 21:57:38]

Done.

+    OmniboxExtensionlastFocused_: Object,

[dpapad, 2017/05/04 16:35:32]

s/Omnibox/omnibox

[scottchen, 2017/05/04 21:57:38]

Done.

   },
 
   // Since the iron-list for extensions is enclosed in a dom-if, observe both
   // |extensions| and |showExtensionsList_|.
   observers: ['extensionsChanged_(extensions, showExtensionsList_)'],
 
   /** @override */
   ready: function() {
     settings.SearchEnginesBrowserProxyImpl.getInstance().
         getSearchEnginesList().then(this.enginesChanged_.bind(this));
@@ skipping 39 matching lines @@
         this.$.addSearchEngine.focus();
       }.bind(this));
     }.bind(this));
   },
 
   /** @private */
   computeShowExtensionsList_: function() {
     return this.extensions.length > 0;
   },
 });