Do not create display_change_listener for parent=NULL.

Do not create display_change_listener for parent=NULL.

The status_icon_win is not backed by any widget, so it is natural to pass |parent| to NULL and it seems to make sense that it won't have display_change_listener in that case.

BUG=181027
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=187356

[dewittj, 2013-03-08 17:57:43]

lgtm to fix 181027, but I don't really know anything about
display_change_listeners.

[dewittj, 2013-03-08 18:29:32]

On 2013/03/08 17:57:43, dewittj wrote:
> lgtm to fix 181027, but I don't really know anything about
> display_change_listeners.

If you could include code to revert crrev.com/12610010 in this CL I'd appreciate
it.

[Jun Mukai, 2013-03-08 23:50:22]

ben, can you review this?  This would be a fix for win aura.

[Ben Goodger (Google), 2013-03-11 16:10:07]

lgtm

https://codereview.chromium.org/12658003/diff/1/ui/views/controls/menu/menu_r...
File ui/views/controls/menu/menu_runner.cc (right):

https://codereview.chromium.org/12658003/diff/1/ui/views/controls/menu/menu_r...
ui/views/controls/menu/menu_runner.cc:287: if ((types & (IS_NESTED | FOR_DROP))
== 0 && parent != NULL) {
nit && parent

[Jun Mukai, 2013-03-11 16:42:28]

https://codereview.chromium.org/12658003/diff/1/ui/views/controls/menu/menu_r...
File ui/views/controls/menu/menu_runner.cc (right):

https://codereview.chromium.org/12658003/diff/1/ui/views/controls/menu/menu_r...
ui/views/controls/menu/menu_runner.cc:287: if ((types & (IS_NESTED | FOR_DROP))
== 0 && parent != NULL) {
On 2013/03/11 16:10:07, Ben Goodger (Google) wrote:
> nit && parent

Done.

[commit-bot: I haz the power, 2013-03-11 16:42:44]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mukai@chromium.org/12658003/8001

[commit-bot: I haz the power, 2013-03-11 20:12:38]

Change committed as 187356