Correct indentation, JSDoc, etc... to comply with closure linter.

Correct indentation, JSDoc, etc... to comply with closure linter.

Errors reported by closure linter are fixed to keep accordance with style guide and prepare for closure compiler
Some errors are ignored.
- E:0007: We don't adopt the blank-line policy as it's not specified by style guide.
- E:0011,0002,0225: There are some false positives around bind().
- E:0121: Comma at end of object literal is valid in ES5, and not prohibited by style guide.

BUG=412662
TEST=find ui/file_manager -name *.js ! -path */*_scripts.js | xargs gjslint --strict --disable 0007,0011,0002,0121,0225

Committed: https://crrev.com/713ee20a4aa068ff1d9083d4fb115f77a6d2ff54
Cr-Commit-Position: refs/heads/master@{#295246}

[fukino, 2014-09-12 14:58:22]

fukino@chromium.org changed reviewers:
+ hirono@chromium.org, yoshiki@chromium.org

[fukino, 2014-09-12 14:59:03]

@yoshiki - PTAL at video_player and file_manager.
@hirono - PTAL at gallery and image_loader.
Thanks!

[yoshiki, 2014-09-16 00:49:09]

lgtm with nits. Thank you for doing hard work.

https://codereview.chromium.org/571453002/diff/40001/ui/file_manager/file_man...
File ui/file_manager/file_manager/background/js/background.js (right):

https://codereview.chromium.org/571453002/diff/40001/ui/file_manager/file_man...
ui/file_manager/file_manager/background/js/background.js:301: profileId,
function() {
nit: Isn't it better to use new line for function.

https://codereview.chromium.org/571453002/diff/40001/ui/file_manager/file_man...
File ui/file_manager/file_manager/foreground/js/file_manager.js (right):

https://codereview.chromium.org/571453002/diff/40001/ui/file_manager/file_man...
ui/file_manager/file_manager/foreground/js/file_manager.js:3761:
.catch(function(error) {
Isn't it incompatible with your other change:
ui/file_manager/file_manager/common/js/util.js:1124?

https://codereview.chromium.org/571453002/diff/40001/ui/file_manager/file_man...
File ui/file_manager/file_manager/foreground/js/file_transfer_controller.js
(right):

https://codereview.chromium.org/571453002/diff/40001/ui/file_manager/file_man...
ui/file_manager/file_manager/foreground/js/file_transfer_controller.js:40:
'change', function(event) {
nit: new line for function?

https://codereview.chromium.org/571453002/diff/40001/ui/file_manager/file_man...
ui/file_manager/file_manager/foreground/js/file_transfer_controller.js:396:
entry, 'thumbnail|filesystem', function(metadata) {
nit: new line for function?

[hirono, 2014-09-16 01:06:16]

On 2014/09/16 00:49:09, yoshiki wrote:
> lgtm with nits. Thank you for doing hard work.
> 
>
https://codereview.chromium.org/571453002/diff/40001/ui/file_manager/file_man...
> File ui/file_manager/file_manager/background/js/background.js (right):
> 
>
https://codereview.chromium.org/571453002/diff/40001/ui/file_manager/file_man...
> ui/file_manager/file_manager/background/js/background.js:301: profileId,
> function() {
> nit: Isn't it better to use new line for function.
> 
>
https://codereview.chromium.org/571453002/diff/40001/ui/file_manager/file_man...
> File ui/file_manager/file_manager/foreground/js/file_manager.js (right):
> 
>
https://codereview.chromium.org/571453002/diff/40001/ui/file_manager/file_man...
> ui/file_manager/file_manager/foreground/js/file_manager.js:3761:
> .catch(function(error) {
> Isn't it incompatible with your other change:
> ui/file_manager/file_manager/common/js/util.js:1124?
> 
>
https://codereview.chromium.org/571453002/diff/40001/ui/file_manager/file_man...
> File ui/file_manager/file_manager/foreground/js/file_transfer_controller.js
> (right):
> 
>
https://codereview.chromium.org/571453002/diff/40001/ui/file_manager/file_man...
> ui/file_manager/file_manager/foreground/js/file_transfer_controller.js:40:
> 'change', function(event) {
> nit: new line for function?
> 
>
https://codereview.chromium.org/571453002/diff/40001/ui/file_manager/file_man...
> ui/file_manager/file_manager/foreground/js/file_transfer_controller.js:396:
> entry, 'thumbnail|filesystem', function(metadata) {
> nit: new line for function?

lgtm after handling yoshiki-san's comment.
Thank you for the work!

[fukino, 2014-09-17 05:17:26]

Thank you for the review and comments!

https://codereview.chromium.org/571453002/diff/40001/ui/file_manager/file_man...
File ui/file_manager/file_manager/background/js/background.js (right):

https://codereview.chromium.org/571453002/diff/40001/ui/file_manager/file_man...
ui/file_manager/file_manager/background/js/background.js:301: profileId,
function() {
On 2014/09/16 00:49:08, yoshiki wrote:
> nit: Isn't it better to use new line for function.

Done.

https://codereview.chromium.org/571453002/diff/40001/ui/file_manager/file_man...
File ui/file_manager/file_manager/foreground/js/file_manager.js (right):

https://codereview.chromium.org/571453002/diff/40001/ui/file_manager/file_man...
ui/file_manager/file_manager/foreground/js/file_manager.js:3761:
.catch(function(error) {
On 2014/09/16 00:49:08, yoshiki wrote:
> Isn't it incompatible with your other change:
> ui/file_manager/file_manager/common/js/util.js:1124?

Done.
I thought it will beyond 80 columns when the promise chain is written in the
same line,
but I was wrong (It is 74 columns at max).
I adopt the same style as util.js:1124. Thanks.

https://codereview.chromium.org/571453002/diff/40001/ui/file_manager/file_man...
File ui/file_manager/file_manager/foreground/js/file_transfer_controller.js
(right):

https://codereview.chromium.org/571453002/diff/40001/ui/file_manager/file_man...
ui/file_manager/file_manager/foreground/js/file_transfer_controller.js:40:
'change', function(event) {
On 2014/09/16 00:49:08, yoshiki wrote:
> nit: new line for function?

Done.

https://codereview.chromium.org/571453002/diff/40001/ui/file_manager/file_man...
ui/file_manager/file_manager/foreground/js/file_transfer_controller.js:396:
entry, 'thumbnail|filesystem', function(metadata) {
On 2014/09/16 00:49:08, yoshiki wrote:
> nit: new line for function?

Done.

[fukino, 2014-09-17 05:17:38]

The CQ bit was checked by fukino@chromium.org

[commit-bot: I haz the power, 2014-09-17 05:18:24]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patchset/571453002/60001

[commit-bot: I haz the power, 2014-09-17 07:19:05]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-09-17 07:19:07]

Exceeded time limit waiting for builds to trigger.

[fukino, 2014-09-17 07:38:49]

The CQ bit was checked by fukino@chromium.org

[commit-bot: I haz the power, 2014-09-17 07:40:01]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patchset/571453002/60001

[commit-bot: I haz the power, 2014-09-17 07:49:32]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-09-17 07:49:34]

Try jobs failed on following builders:
  android_aosp on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_aosp/bu...)

[fukino, 2014-09-17 07:55:05]

The CQ bit was checked by fukino@chromium.org

[commit-bot: I haz the power, 2014-09-17 07:55:58]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patchset/571453002/80001

[commit-bot: I haz the power, 2014-09-17 08:53:28]

Committed patchset #5 (id:80001) as c71c3e32fdeb1defc45b7239242845379d6fe8ba

[commit-bot: I haz the power, 2014-09-17 08:54:12]

Patchset 5 (id:??) landed as
https://crrev.com/713ee20a4aa068ff1d9083d4fb115f77a6d2ff54
Cr-Commit-Position: refs/heads/master@{#295246}