Fix closure comilation error.

Fix closure comilation error.

As a quick fix, I'll revert the change of file_manager_private.js in
https://codereview.chromium.org/2580303002/.

BUG=none
TBR=nya@chromium.org
TEST=run third_party/closure_compiler/run_compiler

Review-Url: https://codereview.chromium.org/2625233004
Cr-Commit-Position: refs/heads/master@{#443214}
Committed: https://chromium.googlesource.com/chromium/src/+/71fc94ca2e7688389d7d7de63ebf67cc3284a30f

[fukino, 2017-01-12 11:44:02]

The CQ bit was checked by fukino@chromium.org

[commit-bot: I haz the power, 2017-01-12 11:44:14]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Shuhei Takahashi, 2017-01-12 12:41:00]

lgtm

Thanks Fukino-san for the prompt fix.

I'm wondering why this happened... I dumbly followed the command line printed by
the linter.

[commit-bot: I haz the power, 2017-01-12 12:49:41]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1484221442285710, "parent_rev":
"6d4cb822bf5bf1ceb4f53d602a9814b5811dff72", "commit_rev":
"71fc94ca2e7688389d7d7de63ebf67cc3284a30f"}

[commit-bot: I haz the power, 2017-01-12 12:50:08]

Description was changed from

==========
Fix closure comilation error.

As a quick fix, I'll revert the change of file_manager_private.js in
https://codereview.chromium.org/2580303002/.

BUG=none
TBR=nya@chromium.org
TEST=run third_party/closure_compiler/run_compiler
==========

to

==========
Fix closure comilation error.

As a quick fix, I'll revert the change of file_manager_private.js in
https://codereview.chromium.org/2580303002/.

BUG=none
TBR=nya@chromium.org
TEST=run third_party/closure_compiler/run_compiler

Review-Url: https://codereview.chromium.org/2625233004
Cr-Commit-Position: refs/heads/master@{#443214}
Committed:
https://chromium.googlesource.com/chromium/src/+/71fc94ca2e7688389d7d7de63ebf...
==========

[commit-bot: I haz the power, 2017-01-12 12:50:09]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/71fc94ca2e7688389d7d7de63ebf...

[fukino, 2017-01-12 12:54:03]

On 2017/01/12 12:41:00, Shuhei Takahashi wrote:
> lgtm
> 
> Thanks Fukino-san for the prompt fix.
> 
> I'm wondering why this happened... I dumbly followed the command line printed
by
> the linter.

No problem at all :)

Actually we don't use the linter these days.
Regarding this externs file, we still have some issues with generated one, so we
manually updates it.
We have a plan to update generator and file_manager to use it
(crbug.com/613096), but at this moment, I'd suggest you to add necessary
definitions to file_manager_private.js manually when you use new API in file
manager WebUI part.

[Dan Beam, 2017-01-12 20:00:52]

dbeam@chromium.org changed reviewers:
+ dbeam@chromium.org