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

Unified Diff: chrome/browser/resources/options/search_engine_manager.js

Issue 2939273002: DO NOT SUBMIT: what chrome/browser/resources/ could eventually look like with clang-format (Closed)
Patch Set: Created 3 years, 6 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/options/search_engine_manager.js
diff --git a/chrome/browser/resources/options/search_engine_manager.js b/chrome/browser/resources/options/search_engine_manager.js
index ce31ef6918163782cd2125798876f2c1a896a934..5b6ce06d998d4f725be601e597a35f3b7421887d 100644
--- a/chrome/browser/resources/options/search_engine_manager.js
+++ b/chrome/browser/resources/options/search_engine_manager.js
@@ -14,9 +14,10 @@ cr.define('options', function() {
*/
function SearchEngineManager() {
this.activeNavTab = null;
- Page.call(this, 'searchEngines',
- loadTimeData.getString('searchEngineManagerPageTabTitle'),
- 'search-engine-manager-page');
+ Page.call(
+ this, 'searchEngines',
+ loadTimeData.getString('searchEngineManagerPageTabTitle'),
+ 'search-engine-manager-page');
}
cr.addSingletonGetter(SearchEngineManager);
@@ -80,20 +81,20 @@ cr.define('options', function() {
updateSearchEngineList_: function(defaultEngines, otherEngines, keywords) {
this.defaultsList_.dataModel = new ArrayDataModel(defaultEngines);
- otherEngines = otherEngines.map(function(x) {
- return [x, x.name.toLocaleLowerCase()];
- }).sort(function(a, b) {
- return a[1].localeCompare(b[1]);
- }).map(function(x) {
- return x[0];
- });
+ otherEngines = otherEngines
+ .map(function(x) {
+ return [x, x.name.toLocaleLowerCase()];
+ })
+ .sort(function(a, b) {
+ return a[1].localeCompare(b[1]);
+ })
+ .map(function(x) {
+ return x[0];
+ });
var othersModel = new ArrayDataModel(otherEngines);
// Add a "new engine" row.
- othersModel.push({
- 'modelIndex': '-1',
- 'canBeEdited': true
- });
+ othersModel.push({'modelIndex': '-1', 'canBeEdited': true});
this.othersList_.dataModel = othersModel;
if (keywords.length > 0) {
@@ -106,12 +107,10 @@ cr.define('options', function() {
},
};
- SearchEngineManager.updateSearchEngineList = function(defaultEngines,
- otherEngines,
- keywords) {
- SearchEngineManager.getInstance().updateSearchEngineList_(defaultEngines,
- otherEngines,
- keywords);
+ SearchEngineManager.updateSearchEngineList = function(
+ defaultEngines, otherEngines, keywords) {
+ SearchEngineManager.getInstance().updateSearchEngineList_(
+ defaultEngines, otherEngines, keywords);
};
SearchEngineManager.validityCheckCallback = function(validity, modelIndex) {
@@ -125,9 +124,6 @@ cr.define('options', function() {
};
// Export
- return {
- SearchEngineManager: SearchEngineManager
- };
+ return {SearchEngineManager: SearchEngineManager};
});
-

Powered by Google App Engine
This is Rietveld 408576698