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

Side by Side Diff: chrome/browser/resources/file_manager/js/metadata/function_parallel.js

Issue 12212187: [Cleanup] Files.app: Misc style fixes in Javascript code. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 10 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 /** 5 /**
6 * @class FunctionSequence to invoke steps in sequence 6 * @class FunctionSequence to invoke steps in sequence
7 * 7 *
8 * @param {string} name //TODO(JSDOC). 8 * @param {string} name //TODO(JSDOC).
9 * @param {Array.<function>} steps Array of functions to invoke in parallel. 9 * @param {Array.<function>} steps Array of functions to invoke in parallel.
10 * @param {Object} logger //TODO(JSDOC). 10 * @param {Object} logger //TODO(JSDOC).
(...skipping 15 matching lines...) Expand all
26 26
27 this.nextStep = this.nextStep_.bind(this); 27 this.nextStep = this.nextStep_.bind(this);
28 this.onError = this.onError_.bind(this); 28 this.onError = this.onError_.bind(this);
29 this.apply = this.start.bind(this); 29 this.apply = this.start.bind(this);
30 } 30 }
31 31
32 32
33 /** 33 /**
34 * Error handling function, which fires error callback. 34 * Error handling function, which fires error callback.
35 * 35 *
36 * @param {string} err Error message 36 * @param {string} err Error message.
37 * @private 37 * @private
38 */ 38 */
39 FunctionParallel.prototype.onError_ = function(err) { 39 FunctionParallel.prototype.onError_ = function(err) {
40 if (!this.failed_) { 40 if (!this.failed_) {
41 this.failed_ = true; 41 this.failed_ = true;
42 this.failureCallback_(err); 42 this.failureCallback_(err);
43 } 43 }
44 }; 44 };
45 45
46 /** 46 /**
47 * Advances to next step. This method should not be used externally. In external 47 * Advances to next step. This method should not be used externally. In external
48 * cases should be used nextStep function, which is defined in closure and thus 48 * cases should be used nextStep function, which is defined in closure and thus
49 * has access to internal variables of functionsequence. 49 * has access to internal variables of functionsequence.
50 * 50 *
51 * @private 51 * @private
52 */ 52 */
53 FunctionParallel.prototype.nextStep_ = function() { 53 FunctionParallel.prototype.nextStep_ = function() {
54 if (--this.remaining == 0 && !this.failed_) { 54 if (--this.remaining == 0 && !this.failed_) {
55 this.callback_(); 55 this.callback_();
56 } 56 }
57 }; 57 };
58 58
59 /** 59 /**
60 * This function should be called only once on start, so start all the children 60 * This function should be called only once on start, so start all the children
61 * at once 61 * at once
62 * @param {...} var_args //TODO(JSDOC). 62 * @param {...} var_args //TODO(JSDOC).
63 */ 63 */
64 FunctionParallel.prototype.start = function(var_args) { 64 FunctionParallel.prototype.start = function(var_args) {
65 this.logger.vlog('Starting [' + this.steps_.length + '] parallel tasks with ' 65 this.logger.vlog('Starting [' + this.steps_.length + '] parallel tasks ' +
66 + arguments.length + ' argument(s)'); 66 'with ' + arguments.length + ' argument(s)');
67 if (this.logger.verbose) { 67 if (this.logger.verbose) {
68 for (var j = 0; j < arguments.length; j++) { 68 for (var j = 0; j < arguments.length; j++) {
69 this.logger.vlog(arguments[j]); 69 this.logger.vlog(arguments[j]);
70 } 70 }
71 } 71 }
72 for (var i = 0; i < this.steps_.length; i++) { 72 for (var i = 0; i < this.steps_.length; i++) {
73 this.logger.vlog('Attempting to start step [' + this.steps_[i].name + ']'); 73 this.logger.vlog('Attempting to start step [' + this.steps_[i].name + ']');
74 try { 74 try {
75 this.steps_[i].apply(this, arguments); 75 this.steps_[i].apply(this, arguments);
76 } catch (e) { 76 } catch (e) {
77 this.onError(e.toString()); 77 this.onError(e.toString());
78 } 78 }
79 } 79 }
80 }; 80 };
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698