Tray elements behave appropriately on the multiple signin screen (more like lock screen)

ash/system/bluetooth/tray_bluetooth.cc

Index: ash/system/bluetooth/tray_bluetooth.cc
diff --git a/ash/system/bluetooth/tray_bluetooth.cc b/ash/system/bluetooth/tray_bluetooth.cc
index 6f4509f8e8224a6fd50a79537cad687fa199d11b..829015ca213be0d207fbca5eef812e7a3fcf40f8 100644
--- a/ash/system/bluetooth/tray_bluetooth.cc
+++ b/ash/system/bluetooth/tray_bluetooth.cc
@@ -4,6 +4,7 @@
 
 #include "ash/system/bluetooth/tray_bluetooth.h"
 
+#include "ash/session/session_state_delegate.h"
 #include "ash/shell.h"
 #include "ash/system/tray/fixed_sized_scroll_view.h"
 #include "ash/system/tray/hover_highlight_view.h"
@@ -301,7 +302,11 @@ class BluetoothDetailedView : public TrayDetailsView,
 
     // Add bluetooth device requires a browser window, hide it for non logged in
     // user.
-    if (login_ == user::LOGGED_IN_NONE || login_ == user::LOGGED_IN_LOCKED)
+    if (login_ == user::LOGGED_IN_NONE || login_ == user::LOGGED_IN_LOCKED ||
+        ash::Shell::GetInstance()
+                ->session_state_delegate()
+                ->GetSessionState() ==
+            ash::SessionStateDelegate::SESSION_STATE_LOGIN_SECONDARY)

[oshima, 2014/07/31 18:28:31]

Am I correct that many checks are basically "if a user is primary, unlocked"? If
so, can you add a predicate method in Shell and use it instead of repeating this
everywhere?

[Roman Sorokin (ftl), 2014/07/31 18:51:32]

This method does exist (IsUserSessionBlocked) but if we use it we completely
ignore LoginStatus param from CreateDefaultView. This is a case of changing
LoginStatus
https://code.google.com/p/chromium/codesearch#chromium/src/ash/system/tray/sy....
And there is a test for it.

       return;

[oshima, 2014/07/31 18:28:31]

you need {} in this case

 
     ash::SystemTrayDelegate* delegate =