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

Unified Diff: ui/file_manager/file_manager/background/js/background.js

Issue 571453002: Correct indentation, JSDoc, etc... to comply with closure linter. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Made it more strict. 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 side-by-side diff with in-line comments
Download patch
Index: ui/file_manager/file_manager/background/js/background.js
diff --git a/ui/file_manager/file_manager/background/js/background.js b/ui/file_manager/file_manager/background/js/background.js
index a38eca349bc195cf360ebe209a1cbb965324b7b0..975e2a8760307f42ea4d1e05bbb0cb017faea267 100644
--- a/ui/file_manager/file_manager/background/js/background.js
+++ b/ui/file_manager/file_manager/background/js/background.js
@@ -291,17 +291,16 @@ AppWindowWrapper.focusOnDesktop = function(appWindow, opt_profileId) {
if (opt_profileId) {
onFulfilled(opt_profileId);
} else {
- chrome.fileManagerPrivate.getProfiles(function(profiles,
- currentId,
- displayedId) {
- onFulfilled(currentId);
- });
+ chrome.fileManagerPrivate.getProfiles(
+ function(profiles, currentId, displayedId) {
+ onFulfilled(currentId);
+ });
}
}).then(function(profileId) {
appWindow.contentWindow.chrome.fileManagerPrivate.visitDesktop(
profileId, function() {
yoshiki 2014/09/16 00:49:08 nit: Isn't it better to use new line for function.
fukino 2014/09/17 05:17:26 Done.
- appWindow.focus();
- });
+ appWindow.focus();
+ });
});
};
@@ -805,12 +804,12 @@ audioPlayerInitializationQueue.run(function(callback) {
* @type {Object}
*/
var audioPlayerCreateOptions = Object.freeze({
- type: 'panel',
- hidden: true,
- minHeight: 44 + 73, // 44px: track, 73px: controller
- minWidth: 292,
- height: 44 + 73, // collapsed
- width: 292
+ type: 'panel',
+ hidden: true,
+ minHeight: 44 + 73, // 44px: track, 73px: controller
+ minWidth: 292,
+ height: 44 + 73, // collapsed
+ width: 292
});
audioPlayer = new SingletonAppWindowWrapper('audio_player.html',

Powered by Google App Engine
This is Rietveld 408576698