Add a disabled attribute to the cr-dropdown-menu.

Add a disabled attribute to the cr-dropdown-menu.

BUG=

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

[Jeremy Klein, 2015-03-04 00:43:51]

jlklein@chromium.org changed reviewers:
+ michaelpg@chromium.org

[michaelpg, 2015-03-06 03:41:44]

lgtm

I was just curious where the "disabled" attribute was actually published. It
comes from core-focusable:

https://github.com/Polymer/core-focusable/blob/master/core-focusable.js

which is mixin'd in here:

https://github.com/Polymer/paper-dropdown-menu/blob/master/paper-dropdown-men...

[michaelpg, 2015-03-06 03:44:10]

sorry, published the LGTM slightly prematurely -- couple comments.

https://codereview.chromium.org/977783002/diff/1/ui/webui/resources/cr_elemen...
File ui/webui/resources/cr_elements/cr_dropdown_menu/cr_dropdown_menu.js
(right):

https://codereview.chromium.org/977783002/diff/1/ui/webui/resources/cr_elemen...
ui/webui/resources/cr_elements/cr_dropdown_menu/cr_dropdown_menu.js:1: /*
Copyright 2015 The Chromium Authors. All rights reserved.
use // c-style-comments for copyright header in JS files

https://codereview.chromium.org/977783002/diff/1/ui/webui/resources/cr_elemen...
ui/webui/resources/cr_elements/cr_dropdown_menu/cr_dropdown_menu.js:7: *
'cr-dropdown-menu' is a Chrome-specific wrapper around paper-dropdown-menu.
I think we should use markdown-style backticks `` like other Polymer elements.
Thoughts?

[Jeremy Klein, 2015-03-06 06:33:32]

https://codereview.chromium.org/977783002/diff/1/ui/webui/resources/cr_elemen...
File ui/webui/resources/cr_elements/cr_dropdown_menu/cr_dropdown_menu.js
(right):

https://codereview.chromium.org/977783002/diff/1/ui/webui/resources/cr_elemen...
ui/webui/resources/cr_elements/cr_dropdown_menu/cr_dropdown_menu.js:1: /*
Copyright 2015 The Chromium Authors. All rights reserved.
On 2015/03/06 03:44:10, michaelpg wrote:
> use // c-style-comments for copyright header in JS files

Done.

https://codereview.chromium.org/977783002/diff/1/ui/webui/resources/cr_elemen...
ui/webui/resources/cr_elements/cr_dropdown_menu/cr_dropdown_menu.js:7: *
'cr-dropdown-menu' is a Chrome-specific wrapper around paper-dropdown-menu.
On 2015/03/06 03:44:10, michaelpg wrote:
> I think we should use markdown-style backticks `` like other Polymer elements.
> Thoughts?

Done.

[Jeremy Klein, 2015-03-06 06:33:37]

The CQ bit was checked by jlklein@chromium.org

[Jeremy Klein, 2015-03-06 06:33:51]

The CQ bit was unchecked by jlklein@chromium.org

[Jeremy Klein, 2015-03-06 06:34:59]

The CQ bit was checked by jlklein@chromium.org

[Jeremy Klein, 2015-03-06 06:35:00]

The patchset sent to the CQ was uploaded after l-g-t-m from
michaelpg@chromium.org
Link to the patchset: https://codereview.chromium.org/977783002/#ps20001 (title:
"Address a couple style comments")

[commit-bot: I haz the power, 2015-03-06 06:36:07]

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

[commit-bot: I haz the power, 2015-03-06 07:26:07]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-03-06 07:26:48]

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