aura: Uber tray should not show up on start screen in kiosk mode.

chrome/browser/chromeos/system/ash_system_tray_delegate.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/chromeos/system/ash_system_tray_delegate.h"
 
 #include "ash/shell.h"
 #include "ash/shell_window_ids.h"
 #include "ash/system/audio/audio_observer.h"
 #include "ash/system/brightness/brightness_observer.h"
@@ skipping 12 matching lines @@
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/chromeos/audio/audio_handler.h"
 #include "chrome/browser/chromeos/cros/cros_library.h"
 #include "chrome/browser/chromeos/cros/network_library.h"
 #include "chrome/browser/chromeos/dbus/dbus_thread_manager.h"
 #include "chrome/browser/chromeos/dbus/power_manager_client.h"
 #include "chrome/browser/chromeos/input_method/input_method_manager.h"
 #include "chrome/browser/chromeos/input_method/input_method_util.h"
 #include "chrome/browser/chromeos/input_method/input_method_whitelist.h"
 #include "chrome/browser/chromeos/input_method/xkeyboard.h"
+#include "chrome/browser/chromeos/kiosk_mode/kiosk_mode_helper.h"
 #include "chrome/browser/chromeos/login/base_login_display_host.h"
 #include "chrome/browser/chromeos/login/login_display_host.h"
 #include "chrome/browser/chromeos/login/user.h"
 #include "chrome/browser/chromeos/login/user_manager.h"
 #include "chrome/browser/chromeos/status/network_menu.h"
 #include "chrome/browser/chromeos/status/network_menu_icon.h"
 #include "chrome/browser/chromeos/system/timezone_settings.h"
 #include "chrome/browser/chromeos/system_key_event_listener.h"
 #include "chrome/browser/prefs/pref_service.h"
 #include "chrome/browser/profiles/profile_manager.h"
@@ skipping 85 matching lines @@
     AudioHandler* audiohandler = AudioHandler::GetInstance();
     if (audiohandler)
       audiohandler->RemoveVolumeObserver(this);
     DBusThreadManager::Get()->GetPowerManagerClient()->RemoveObserver(this);
     input_method::InputMethodManager::GetInstance()->RemoveObserver(this);
     system::TimezoneSettings::GetInstance()->RemoveObserver(this);
     if (SystemKeyEventListener::GetInstance())
       SystemKeyEventListener::GetInstance()->RemoveCapsLockObserver(this);
   }
 
+  virtual bool GetTrayVisibilityOnStartup() OVERRIDE {
+    return !chromeos::KioskModeHelper::IsKioskModeEnabled();

[sadrul, 2012/03/22 04:15:36]

Does this work correctly if you login (in kiosk mode), and the browser crashes?
(close all browsers, then go to 'Settings and Help' from the tray popup)

[varunjain, 2012/03/22 04:19:38]

Seems to work.

+  }
+
   // Overridden from ash::SystemTrayDelegate:
   virtual const std::string GetUserDisplayName() const OVERRIDE {
     return UserManager::Get()->GetLoggedInUser().GetDisplayName();
   }
 
   virtual const std::string GetUserEmail() const OVERRIDE {
     return UserManager::Get()->GetLoggedInUser().email();
   }
 
   virtual const SkBitmap& GetUserImage() const OVERRIDE {
@@ skipping 366 matching lines @@
   virtual void OnCellularDataPlanChanged(NetworkLibrary* crosnet) OVERRIDE {
     NotifyRefreshNetwork();
   }
 
   // content::NotificationObserver implementation.
   virtual void Observe(int type,
                        const content::NotificationSource& source,
                        const content::NotificationDetails& details) OVERRIDE {
     switch (type) {
       case chrome::NOTIFICATION_LOGIN_USER_CHANGED: {
         tray_->UpdateAfterLoginStatusChange(GetUserLoginStatus());

[sadrul, 2012/03/22 04:15:36]

Update visibility in SystemTray::UpdateAfterLoginStatusChange instead?

[varunjain, 2012/03/22 04:19:38]

Done.

+        // If tray is not initially visible, show it now that we have logged in.
+        if (!GetTrayVisibilityOnStartup())
+          tray_->SetVisible(true);
         break;
       }
       case chrome::NOTIFICATION_UPGRADE_RECOMMENDED: {
         ash::UpdateObserver* observer =
             ash::Shell::GetInstance()->tray()->update_observer();
         if (observer)
           observer->OnUpdateRecommended();
         break;
       }
       case chrome::NOTIFICATION_LOGIN_USER_IMAGE_CHANGED: {
@@ skipping 80 matching lines @@
   DISALLOW_COPY_AND_ASSIGN(SystemTrayDelegate);
 };
 
 }  // namespace
 
 ash::SystemTrayDelegate* CreateSystemTrayDelegate(ash::SystemTray* tray) {
   return new chromeos::SystemTrayDelegate(tray);
 }
 
 }  // namespace chromeos