Remove file-level 'use strict' from file_manager.

Remove file-level 'use strict' from file_manager.

BUG=406995
TEST=none

Committed: https://crrev.com/ed3332f7d0c397f17c0b85001ff40b170d00f33a
Cr-Commit-Position: refs/heads/master@{#301082}

[fukino, 2014-10-23 07:43:56]

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

[fukino, 2014-10-23 07:43:58]

yoshiki@, hirono@:
I'd like to remove file-level 'use strict's for the following reasons:
(1) They are warned by the compiler and prevent us from being fully type
checked.
(2) They are confusing. The actual strict mode depends on whether the script is
concatenated and what script is the first, and this lowers our scripts'
modularity.
(3) The compiler runs in 'ECMASCRIPT5_STRICT' mode. Most errors reported in
strict mode are also be checked by closure compiler without writing 'use
strict'.

The drawback is, when we run script without concatenation, scripts will be run
in non-strict mode.
When we have code which must be run in strict mode, I think we should enclose
them by anonymous function and expose global member explicitly to make sure they
are actually run in strict mode.

WDYT?

[hirono, 2014-10-23 12:26:28]

Sounds good to me. lgtm.

[yoshiki, 2014-10-24 10:59:30]

Thanks! LGTM

[fukino, 2014-10-24 11:26:23]

On 2014/10/24 10:59:30, yoshiki wrote:
> Thanks! LGTM

Thank y'all!

[fukino, 2014-10-24 11:26:35]

The CQ bit was checked by fukino@chromium.org

[commit-bot: I haz the power, 2014-10-24 11:27:30]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/676633002/1

[commit-bot: I haz the power, 2014-10-24 12:12:44]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2014-10-24 12:13:26]

Patchset 1 (id:??) landed as
https://crrev.com/ed3332f7d0c397f17c0b85001ff40b170d00f33a
Cr-Commit-Position: refs/heads/master@{#301082}