ash: Do not show the arrow in the tray bubbles when the tray isn't visible.

ash: Do not show the arrow in the tray bubbles when the tray isn't visible.

BUG=120140
TEST=none

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

[sadrul, 2012-03-28 05:38:47]

http://codereview.chromium.org/9873020/diff/1/ash/system/tray/system_tray.cc
File ash/system/tray/system_tray.cc (right):

http://codereview.chromium.org/9873020/diff/1/ash/system/tray/system_tray.cc#...
ash/system/tray/system_tray.cc:224:
internal::ShelfLayoutManager::AUTO_HIDE_SHOWN)) {
Is there a better way of determining if the tray/shelf is visible?

[sky, 2012-03-28 15:31:11]

LGTM. I filed a bug on this a while back (120140). You'll likely want to take
that bug and talk with Jenny.

http://codereview.chromium.org/9873020/diff/1/ash/system/tray/system_tray.cc
File ash/system/tray/system_tray.cc (right):

http://codereview.chromium.org/9873020/diff/1/ash/system/tray/system_tray.cc#...
ash/system/tray/system_tray.cc:224:
internal::ShelfLayoutManager::AUTO_HIDE_SHOWN)) {
On 2012/03/28 05:38:47, sadrul wrote:
> Is there a better way of determining if the tray/shelf is visible?

No. I'm not opposed to you adding a method is_visible that does this check.

[sadrul, 2012-03-28 18:24:49]

I have added ShelfLayoutManager::IsVisible, as suggested. Another look please?

[sky, 2012-03-28 20:23:53]

LGTM