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

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

Issue 426723004: options: Remove a bunch of useless: (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 4 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 | Annotate | Revision Log
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 // TODO(kochi): Generalize the notification as a component and put it 5 // TODO(kochi): Generalize the notification as a component and put it
6 // in js/cr/ui/notification.js . 6 // in js/cr/ui/notification.js .
7 7
8 cr.define('options', function() { 8 cr.define('options', function() {
9 /** @const */ var OptionsPage = options.OptionsPage; 9 /** @const */ var OptionsPage = options.OptionsPage;
10 /** @const */ var LanguageList = options.LanguageList; 10 /** @const */ var LanguageList = options.LanguageList;
(...skipping 138 matching lines...) Expand 10 before | Expand all | Expand 10 after
149 */ 149 */
150 languageCodeToInputMethodIdsMap_: {}, 150 languageCodeToInputMethodIdsMap_: {},
151 151
152 /** 152 /**
153 * The value that indicates if Translate feature is enabled or not. 153 * The value that indicates if Translate feature is enabled or not.
154 * @type {boolean} 154 * @type {boolean}
155 * @private 155 * @private
156 */ 156 */
157 enableTranslate_: false, 157 enableTranslate_: false,
158 158
159 /** 159 /** @override */
160 * Initializes LanguageOptions page.
161 * Calls base class implementation to start preference initialization.
162 */
163 initializePage: function() { 160 initializePage: function() {
164 OptionsPage.prototype.initializePage.call(this); 161 OptionsPage.prototype.initializePage.call(this);
165 162
166 var languageOptionsList = $('language-options-list'); 163 var languageOptionsList = $('language-options-list');
167 LanguageList.decorate(languageOptionsList); 164 LanguageList.decorate(languageOptionsList);
168 165
169 languageOptionsList.addEventListener('change', 166 languageOptionsList.addEventListener('change',
170 this.handleLanguageOptionsListChange_.bind(this)); 167 this.handleLanguageOptionsListChange_.bind(this));
171 languageOptionsList.addEventListener('save', 168 languageOptionsList.addEventListener('save',
172 this.handleLanguageOptionsListSave_.bind(this)); 169 this.handleLanguageOptionsListSave_.bind(this));
(...skipping 1185 matching lines...) Expand 10 before | Expand all | Expand 10 after
1358 1355
1359 LanguageOptions.onDictionaryDownloadFailure = function(languageCode) { 1356 LanguageOptions.onDictionaryDownloadFailure = function(languageCode) {
1360 LanguageOptions.getInstance().onDictionaryDownloadFailure_(languageCode); 1357 LanguageOptions.getInstance().onDictionaryDownloadFailure_(languageCode);
1361 }; 1358 };
1362 1359
1363 // Export 1360 // Export
1364 return { 1361 return {
1365 LanguageOptions: LanguageOptions 1362 LanguageOptions: LanguageOptions
1366 }; 1363 };
1367 }); 1364 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698