Add type signatures for enums in wallpaper API.

Add type signatures for enums in wallpaper API.

We use enums instead of string directly, but I forgot to add enum type signatures for closure compiler.

BUG=578935
Committed: https://crrev.com/07e06314d1e4cba78137fcea8bb374b168e1b632
Cr-Commit-Position: refs/heads/master@{#378174}

[ryoh, 2016-02-26 09:17:21]

ryoh@chromium.org changed reviewers:
+ fukino@chromium.org

[ryoh, 2016-02-26 09:17:21]

Sorry, I forgot to add type signatures for enums even though I use an enum value
instead of string directly.

Could you take a look?

[Dan Beam, 2016-02-26 17:05:39]

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

[Dan Beam, 2016-02-26 17:05:40]

lgtm

[ryoh, 2016-02-27 03:43:22]

The CQ bit was checked by ryoh@chromium.org

[ryoh, 2016-02-27 03:43:35]

On 2016/02/26 17:05:40, Dan Beam wrote:
> lgtm

Thank you!

[commit-bot: I haz the power, 2016-02-27 03:43:39]

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

[commit-bot: I haz the power, 2016-02-27 03:55:23]

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

[commit-bot: I haz the power, 2016-02-27 03:55:24]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[ryoh, 2016-02-29 05:50:00]

> fukino@.
I copied and pasted definitions from upstream externs file.
Could you take a look?
We need OWNER approval for commit.

[fukino, 2016-02-29 05:51:43]

lgtm with a nit.

https://codereview.chromium.org/1738103002/diff/20001/ui/file_manager/externs...
File ui/file_manager/externs/chrome_wallpaper.js (right):

https://codereview.chromium.org/1738103002/diff/20001/ui/file_manager/externs...
ui/file_manager/externs/chrome_wallpaper.js:10: * @const
nit: every line has a leading space. please remove them.

[ryoh, 2016-02-29 05:54:52]

https://codereview.chromium.org/1738103002/diff/20001/ui/file_manager/externs...
File ui/file_manager/externs/chrome_wallpaper.js (right):

https://codereview.chromium.org/1738103002/diff/20001/ui/file_manager/externs...
ui/file_manager/externs/chrome_wallpaper.js:10: * @const
On 2016/02/29 05:51:43, fukino wrote:
> nit: every line has a leading space. please remove them.

Done.

[ryoh, 2016-02-29 05:54:56]

The CQ bit was checked by ryoh@chromium.org

[ryoh, 2016-02-29 05:54:57]

The patchset sent to the CQ was uploaded after l-g-t-m from dbeam@chromium.org,
fukino@chromium.org
Link to the patchset: https://codereview.chromium.org/1738103002/#ps40001
(title: "remove leading spaces")

[commit-bot: I haz the power, 2016-02-29 05:55:06]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1738103002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1738103002/40001

[commit-bot: I haz the power, 2016-02-29 06:23:31]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-02-29 06:24:29]

Description was changed from

==========
Add type signatures for enums in wallpaper API.

We use enums instead of string directly, but I forgot to add enum type
signatures for closure compiler.

BUG=578935
==========

to

==========
Add type signatures for enums in wallpaper API.

We use enums instead of string directly, but I forgot to add enum type
signatures for closure compiler.

BUG=578935

Committed: https://crrev.com/07e06314d1e4cba78137fcea8bb374b168e1b632
Cr-Commit-Position: refs/heads/master@{#378174}
==========

[commit-bot: I haz the power, 2016-02-29 06:24:30]

Patchset 3 (id:??) landed as
https://crrev.com/07e06314d1e4cba78137fcea8bb374b168e1b632
Cr-Commit-Position: refs/heads/master@{#378174}