Removes tray keyboard icon from the status area.

ash/system/chromeos/virtual_keyboard/tray_keyboard_lock.cc

 // Copyright 2014 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 "ash/system/chromeos/virtual_keyboard/tray_keyboard_lock.h"
 
 #include "ash/shell.h"
 #include "ash/system/tray/system_tray.h"
 #include "ash/system/tray/tray_item_more.h"
 #include "ash/system/tray_accessibility.h"
@@ skipping 86 matching lines @@
 }
 
 void KeyboardLockDefaultView::OnAccessibilityModeChanged(
     ui::AccessibilityNotificationVisibility notify) {
   Update();
 }
 
 }  // namespace tray
 
 TrayKeyboardLock::TrayKeyboardLock(SystemTray* system_tray)
-    : TrayImageItem(system_tray,
-                    IDR_AURA_UBER_TRAY_VIRTUAL_KEYBOARD_SUPPRESSED),
-      virtual_keyboard_suppressed_(false) {
+    : SystemTrayItem(system_tray), virtual_keyboard_suppressed_(false) {
   Shell::GetInstance()->system_tray_notifier()->AddVirtualKeyboardObserver(
       this);
-  Shell::GetInstance()->system_tray_notifier()->AddAccessibilityObserver(this);
 }
 
 TrayKeyboardLock::~TrayKeyboardLock() {
   Shell::GetInstance()->system_tray_notifier()->RemoveVirtualKeyboardObserver(
       this);
-  Shell::GetInstance()->system_tray_notifier()->RemoveAccessibilityObserver(
-      this);
 }
 
 void TrayKeyboardLock::OnKeyboardSuppressionChanged(bool suppressed) {
   virtual_keyboard_suppressed_ = suppressed;

[sadrul, 2014/12/22 21:21:27]

Side question: instead of also adding KeyboardLockDefaultView as a VKObserver,
why not notify it from here?

[rsadam, 2014/12/22 21:26:35]

From what I understand, the DefaultView is only created when the user clicks on
the tray to bring it up. I felt that the current approach was cleaner than
checking whether a DefaultView was currently active and updating it. Is this
something I should change?

[sadrul, 2014/12/22 21:32:08]

Yes.
 if (default_view_)
   default_view_->UpdateSuppression(suppressed);

isn't that unclean. The reason I make this suggestion is that we follow this
pattern for a number of the other tray items (e.g. audio, bluetooth, network,
etc.).
Not necessarily (depends on the owner of this particular item; this is more of a
nit), and definitely not in this CL.

-  tray_view()->SetVisible(ShouldTrayBeVisible());
 }
 
 views::View* TrayKeyboardLock::CreateDefaultView(user::LoginStatus status) {
   return new tray::KeyboardLockDefaultView(this, virtual_keyboard_suppressed_);
 }
 
-void TrayKeyboardLock::OnAccessibilityModeChanged(
-    ui::AccessibilityNotificationVisibility notify) {
-  tray_view()->SetVisible(ShouldTrayBeVisible());
-}
-
-bool TrayKeyboardLock::GetInitialVisibility() {
-  return ShouldTrayBeVisible();
-}
-
-bool TrayKeyboardLock::ShouldTrayBeVisible() {
-  // Do not show if the keyboard is enabled.
-  return virtual_keyboard_suppressed_ && !keyboard::IsKeyboardEnabled();
-}
-
 }  // namespace ash