Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(461)

Side by Side Diff: chrome/browser/resources/options/search_engine_manager.js

Issue 543493002: Compile chrome://settings, part 2: reduce from 950 to 400 errors (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@F_settings
Patch Set: rebase? rebase! Created 6 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 cr.define('options', function() { 5 cr.define('options', function() {
6 /** @const */ var Page = cr.ui.pageManager.Page; 6 /** @const */ var Page = cr.ui.pageManager.Page;
7 /** @const */ var PageManager = cr.ui.pageManager.PageManager; 7 /** @const */ var PageManager = cr.ui.pageManager.PageManager;
8 /** @const */ var ArrayDataModel = cr.ui.ArrayDataModel; 8 /** @const */ var ArrayDataModel = cr.ui.ArrayDataModel;
9 9
10 /** 10 /**
11 * Encapsulated handling of search engine management page. 11 * Encapsulated handling of search engine management page.
12 * @constructor 12 * @constructor
13 * @extends {cr.ui.pageManager.Page}
13 */ 14 */
14 function SearchEngineManager() { 15 function SearchEngineManager() {
15 this.activeNavTab = null; 16 this.activeNavTab = null;
16 Page.call(this, 'searchEngines', 17 Page.call(this, 'searchEngines',
17 loadTimeData.getString('searchEngineManagerPageTabTitle'), 18 loadTimeData.getString('searchEngineManagerPageTabTitle'),
18 'search-engine-manager-page'); 19 'search-engine-manager-page');
19 } 20 }
20 21
21 cr.addSingletonGetter(SearchEngineManager); 22 cr.addSingletonGetter(SearchEngineManager);
22 23
(...skipping 31 matching lines...) Expand 10 before | Expand all | Expand 10 after
54 this.extensionList_ = $('extension-keyword-list'); 55 this.extensionList_ = $('extension-keyword-list');
55 this.setUpList_(this.extensionList_); 56 this.setUpList_(this.extensionList_);
56 57
57 $('search-engine-manager-confirm').onclick = function() { 58 $('search-engine-manager-confirm').onclick = function() {
58 PageManager.closeOverlay(); 59 PageManager.closeOverlay();
59 }; 60 };
60 }, 61 },
61 62
62 /** 63 /**
63 * Sets up the given list as a search engine list 64 * Sets up the given list as a search engine list
64 * @param {List} list The list to set up. 65 * @param {cr.ui.List} list The list to set up.
65 * @private 66 * @private
66 */ 67 */
67 setUpList_: function(list) { 68 setUpList_: function(list) {
68 options.search_engines.SearchEngineList.decorate(list); 69 options.search_engines.SearchEngineList.decorate(list);
69 list.autoExpands = true; 70 list.autoExpands = true;
70 }, 71 },
71 72
72 /** 73 /**
73 * Updates the search engine list with the given entries. 74 * Updates the search engine list with the given entries.
74 * @private 75 * @private
(...skipping 48 matching lines...) Expand 10 before | Expand all | Expand 10 after
123 validity, modelIndex); 124 validity, modelIndex);
124 }; 125 };
125 126
126 // Export 127 // Export
127 return { 128 return {
128 SearchEngineManager: SearchEngineManager 129 SearchEngineManager: SearchEngineManager
129 }; 130 };
130 131
131 }); 132 });
132 133
OLDNEW
« no previous file with comments | « chrome/browser/resources/options/preferences.js ('k') | chrome/browser/resources/options/search_engine_manager_engine_list.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698