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

Side by Side Diff: chrome/browser/resources/extensions/extension_list.js

Issue 1137763002: Fix closure compile issue in extensions_list.js (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 7 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 <include src="extension_error.js"> 5 <include src="extension_error.js">
6 6
7 /////////////////////////////////////////////////////////////////////////////// 7 ///////////////////////////////////////////////////////////////////////////////
8 // ExtensionFocusRow: 8 // ExtensionFocusRow:
9 9
10 /** 10 /**
(...skipping 1097 matching lines...) Expand 10 before | Expand all | Expand 10 after
1108 1108
1109 // TODO(dbeam): why do we need to focus <extensionoptions> before and 1109 // TODO(dbeam): why do we need to focus <extensionoptions> before and
1110 // after its showing animation? Makes very little sense to me. 1110 // after its showing animation? Makes very little sense to me.
1111 overlay.setInitialFocus(); 1111 overlay.setInitialFocus();
1112 }, 1112 },
1113 1113
1114 /** 1114 /**
1115 * Hides the extension options overlay for the extension with id 1115 * Hides the extension options overlay for the extension with id
1116 * |extensionId|. If there is an overlay showing for a different extension, 1116 * |extensionId|. If there is an overlay showing for a different extension,
1117 * nothing happens. 1117 * nothing happens.
1118 * @param {string} extension id to hide. 1118 * @param {string} extensionId ID of the extension to hide.
1119 * @private 1119 * @private
1120 */ 1120 */
1121 hideEmbeddedExtensionOptions_: function(extensionId) { 1121 hideEmbeddedExtensionOptions_: function(extensionId) {
1122 if (!this.optionsShown_) 1122 if (!this.optionsShown_)
1123 return; 1123 return;
1124 1124
1125 var overlay = extensions.ExtensionOptionsOverlay.getInstance(); 1125 var overlay = extensions.ExtensionOptionsOverlay.getInstance();
1126 if (overlay.getExtensionId() == extensionId) 1126 if (overlay.getExtensionId() == extensionId)
1127 overlay.close(); 1127 overlay.close();
1128 }, 1128 },
(...skipping 26 matching lines...) Expand all
1155 this.createNode_(extension, nextExt ? $(nextExt.id) : null); 1155 this.createNode_(extension, nextExt ? $(nextExt.id) : null);
1156 } 1156 }
1157 } 1157 }
1158 }; 1158 };
1159 1159
1160 return { 1160 return {
1161 ExtensionList: ExtensionList, 1161 ExtensionList: ExtensionList,
1162 ExtensionListDelegate: ExtensionListDelegate 1162 ExtensionListDelegate: ExtensionListDelegate
1163 }; 1163 };
1164 }); 1164 });
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698