Convert search_engines_page (and search_engine-adder) to be compatible with Polymer 0.8.

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 'cr-settings-search-engines-page' is the settings page
  * containing search engines settings.
  *
  * Example:
  *
  *    <core-animated-pages>
  *      <cr-settings-search-engines-page prefs="{{prefs}}">
  *      </cr-settings-search-engines-page>
  *      ... other pages ...
  *    </core-animated-pages>
  *
  * @group Chrome Settings Elements
  * @element cr-settings-search-engines-page
  */
-Polymer('cr-settings-search-engines-page', {
-  publish: {
+Polymer({
+  is: 'cr-settings-search-engines-page',
+
+  properties: {
     /**
      * Preferences state.
-     *
-     * @attribute prefs
-     * @type {CrSettingsPrefsElement}
-     * @default null
+     * @type {?CrSettingsPrefsElement}
      */
     prefs: null,

[Jeremy Klein, 2015/05/12 21:02:36]

Object, notify:true.

[Kyle Horimoto, 2015/05/12 21:41:52]

Done.

 
     /**
      * Route for the page.
-     *
-     * @attribute route
      * @type {string}
-     * @default ''
      */
-    route: '',
+    route: {
+      type: String,
+      value: ''
+    },
 
     /**
      * Whether the page is a subpage.
-     *
-     * @attribute subpage
      * @type {boolean}

[Jeremy Klein, 2015/05/12 21:02:36]

nit: Remove unnecessary @type on primitives.

[Kyle Horimoto, 2015/05/12 21:41:53]

Done.

-     * @default true
      */
-    subpage: true,
+    subpage: {

[Jeremy Klein, 2015/05/12 21:02:36]

Make this, PAGE_ID, and icon readOnly: true.

[Kyle Horimoto, 2015/05/12 21:41:53]

Done.

+      type: Boolean,
+      value: true
+    },
 
     /**
      * ID of the page.
-     *
-     * @attribute PAGE_ID
      * @const {string}
-     * @default 'search'
      */
-    PAGE_ID: 'search_engines',
+    PAGE_ID: {
+      type: String,
+      value: 'search_engines',
+      readOnly: true
+    },
 
     /**
      * Title for the page header and navigation menu.
-     *
-     * @attribute pageTitle
      * @type {string}
      */
-    pageTitle: loadTimeData.getString('searchEnginesPageTitle'),
+    pageTitle: {
+      type: String,
+      value: loadTimeData.getString('searchEnginesPageTitle')
+    },
 
     /**
      * Name of the 'core-icon' to be shown in the settings-page-header.
-     *
-     * @attribute icon
      * @type {string}
-     * @default 'search'
      */
-    icon: 'search',
+    icon: {
+      type: String,
+      value: 'search'
+    },
 
     /**
      * List of default search engines available.
-     *
-     * @attribute defaultSearchEngines
-     * @type {Array<!SearchEngine>}
-     * @default null
+     * @type {!Array<!SearchEngine>}
      */
-    defaultSearchEngines: null,
+    defaultSearchEngines: {
+      type: Array,
+      value: function() { return []; }
+    },
 
     /**
      * List of other search engines available.
-     *
-     * @attribute otherSearchEngines
-     * @type {Array<!SearchEngine>}
-     * @default null
+     * @type {!Array<!SearchEngine>}
      */
-    otherSearchEngines: null,
+    otherSearchEngines: {
+      type: Array,
+      value: function() { return []; }
+    },
 
     /**
      * GUID of the currently selected default search engine.
-     *
-     * @attribute defaultEngineGuid
      * @type {string}
-     * @default ''
      */
-    defaultEngineGuid: '',
+    defaultEngineGuid: {
+      type: String,
+      value: ''
+    },
   },
 
   /** @override */
-  created: function() {
-    this.searchEngines = [];
-  },
-
-  /** @override */
   domReady: function() {
     chrome.searchEnginesPrivate.onSearchEnginesChanged.addListener(
         this.updateSearchEngines_.bind(this));
     chrome.searchEnginesPrivate.getSearchEngines(
         this.updateSearchEngines_.bind(this));
   },
 
   /**
    * Persists the new default search engine back to Chrome. Called when the
    * user selects a new default in the search engines dropdown.
    */
   defaultEngineGuidChanged: function() {

[Jeremy Klein, 2015/05/12 21:02:36]

Well... Seems like half this file isn't used. Hooray copy-pasta!

[Kyle Horimoto, 2015/05/12 21:41:52]

Lame. Added a TODO in the HTML file for Oren to fix this again. I saw him
working on it a few weeks back so I know something was working, but I guess it
got lost.

     chrome.searchEnginesPrivate.setSelectedSearchEngine(this.defaultEngineGuid);
   },
 
 
   /**
    * Updates the lists of search engines based on the given |engines|.
    * @param {!Array<!SearchEngine>} engines All the search engines.
    * @private
    */
   updateSearchEngines_: function(engines) {
@@ skipping 10 matching lines @@
       } else if (engine.type ==
                  chrome.searchEnginesPrivate.SearchEngineType.OTHER) {
         otherEngines.push(engine);
       }
     }, this);
 
     this.defaultSearchEngines = defaultEngines;
     this.otherSearchEngines = otherEngines;
   }
 });