Vectorize menu radio buttons.

Vectorize menu radio buttons.

BUG=561133
TEST=on cros, right click on shelf > shelf position

Committed: https://crrev.com/53cd62240b689e5fd8876580fff33a06c866928f
Cr-Commit-Position: refs/heads/master@{#371948}

[Evan Stade, 2016-01-26 19:40:20]

estade@chromium.org changed reviewers:
+ sadrul@chromium.org

[Evan Stade, 2016-01-26 19:40:29]

The CQ bit was checked by estade@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-01-26 19:41:58]

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

[commit-bot: I haz the power, 2016-01-26 20:38:13]

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

[commit-bot: I haz the power, 2016-01-26 20:38:13]

Dry run: This issue passed the CQ dry run.

[sadrul, 2016-01-26 21:59:30]

lgtm

https://codereview.chromium.org/1637973002/diff/1/ui/views/controls/menu/menu...
File ui/views/controls/menu/menu_image_util.h (right):

https://codereview.chromium.org/1637973002/diff/1/ui/views/controls/menu/menu...
ui/views/controls/menu/menu_image_util.h:21: // the radio option is active,
|hovered| describes the menu higlight/selection
"the radio option is active" would 'selected' be better than 'active' here?

|hovered| indicates highlight, right? How does it describe selection?

[Evan Stade, 2016-01-26 22:31:53]

https://codereview.chromium.org/1637973002/diff/1/ui/views/controls/menu/menu...
File ui/views/controls/menu/menu_image_util.h (right):

https://codereview.chromium.org/1637973002/diff/1/ui/views/controls/menu/menu...
ui/views/controls/menu/menu_image_util.h:21: // the radio option is active,
|hovered| describes the menu higlight/selection
On 2016/01/26 21:59:30, sadrul wrote:
> "the radio option is active" would 'selected' be better than 'active' here?
> 
> |hovered| indicates highlight, right? How does it describe selection?

I avoided the word selection because it's ambiguous here (it's used for both a
"checked" radio button and a mouse hover, depending on context).

[Evan Stade, 2016-01-28 00:06:43]

The CQ bit was checked by estade@chromium.org

[commit-bot: I haz the power, 2016-01-28 00:07:51]

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

[commit-bot: I haz the power, 2016-01-28 00:57:04]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-01-28 01:16:32]

Description was changed from

==========
Vectorize menu radio buttons.

BUG=561133
TEST=on cros, right click on shelf > shelf position
==========

to

==========
Vectorize menu radio buttons.

BUG=561133
TEST=on cros, right click on shelf > shelf position

Committed: https://crrev.com/53cd62240b689e5fd8876580fff33a06c866928f
Cr-Commit-Position: refs/heads/master@{#371948}
==========

[commit-bot: I haz the power, 2016-01-28 01:16:33]

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