Stop using shell::GetInstance()->system_tray() in system tray items

ash/system/chromeos/network/tray_network.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 "ash/system/chromeos/network/tray_network.h"
 
 #include "ash/shell.h"
 #include "ash/system/chromeos/network/network_list_detailed_view_base.h"
 #include "ash/system/tray/system_tray.h"
 #include "ash/system/tray/system_tray_delegate.h"
@@ skipping 61 matching lines @@
 
   MessageMap& messages() { return messages_; }
   const MessageMap& messages() const { return messages_; }
 
  private:
   MessageMap messages_;
 };
 
 class NetworkTrayView : public TrayItemView {
  public:
-  NetworkTrayView(ColorTheme size, bool tray_icon)
-      : color_theme_(size), tray_icon_(tray_icon) {
+  NetworkTrayView(SystemTrayItem* tray, ColorTheme size, bool tray_icon)
+      : TrayItemView(tray), color_theme_(size), tray_icon_(tray_icon) {
     SetLayoutManager(
         new views::BoxLayout(views::BoxLayout::kHorizontal, 0, 0, 0));
 
     image_view_ = color_theme_ == DARK ?
         new FixedSizedImageView(0, kTrayPopupItemHeight) :
         new views::ImageView;
     AddChildView(image_view_);
 
     NetworkIconInfo info;
     Shell::GetInstance()->tray_delegate()->
@@ skipping 13 matching lines @@
  private:
   views::ImageView* image_view_;
   ColorTheme color_theme_;
   bool tray_icon_;
 
   DISALLOW_COPY_AND_ASSIGN(NetworkTrayView);
 };
 
 class NetworkDefaultView : public TrayItemMore {
  public:
-  NetworkDefaultView(SystemTrayItem* owner, bool show_more)
-      : TrayItemMore(owner, show_more) {
+  NetworkDefaultView(SystemTrayItem* tray, bool show_more)
+      : TrayItemMore(tray, show_more) {
     Update();
   }
 
   virtual ~NetworkDefaultView() {}
 
   void Update() {
     NetworkIconInfo info;
     Shell::GetInstance()->tray_delegate()->
         GetMostRelevantNetworkIcon(&info, true);
     SetImage(&info.image);
     SetLabel(info.description);
     SetAccessibleName(info.description);
   }
 
  private:
   DISALLOW_COPY_AND_ASSIGN(NetworkDefaultView);
 };
 
 class NetworkListDetailedView : public NetworkListDetailedViewBase {
  public:
-  NetworkListDetailedView(user::LoginStatus login, int header_string_id)
-      : NetworkListDetailedViewBase(login, header_string_id),
+  NetworkListDetailedView(SystemTrayItem* tray,
+                          user::LoginStatus login,
+                          int header_string_id)
+      : NetworkListDetailedViewBase(tray, login, header_string_id),
         airplane_(NULL),
         button_wifi_(NULL),
         button_mobile_(NULL),
         view_mobile_account_(NULL),
         setup_mobile_account_(NULL),
         other_wifi_(NULL),
         turn_on_wifi_(NULL),
         other_mobile_(NULL) {
   }
 
@@ skipping 203 matching lines @@
   views::View* setup_mobile_account_;
   TrayPopupLabelButton* other_wifi_;
   TrayPopupLabelButton* turn_on_wifi_;
   TrayPopupLabelButton* other_mobile_;
 
   DISALLOW_COPY_AND_ASSIGN(NetworkListDetailedView);
 };
 
 class NetworkWifiDetailedView : public NetworkDetailedView {
  public:
-  explicit NetworkWifiDetailedView(bool wifi_enabled) {
+  NetworkWifiDetailedView(SystemTrayItem* tray, bool wifi_enabled)
+     : NetworkDetailedView(tray) {
     SetLayoutManager(new views::BoxLayout(views::BoxLayout::kHorizontal,
                                           kTrayPopupPaddingHorizontal,
                                           10,
                                           kTrayPopupPaddingBetweenItems));
     ui::ResourceBundle& bundle = ui::ResourceBundle::GetSharedInstance();
     views::ImageView* image = new views::ImageView;
     const int image_id = wifi_enabled ?
         IDR_AURA_UBER_TRAY_WIFI_ENABLED : IDR_AURA_UBER_TRAY_WIFI_DISABLED;
     image->SetImage(bundle.GetImageNamed(image_id).ToImageSkia());
     AddChildView(image);
@@ skipping 123 matching lines @@
         new NetworkMessageView(tray_network(), iter->first, iter->second);
   }
 
   tray::NetworkMessageView* network_message_view_;
 
   DISALLOW_COPY_AND_ASSIGN(NetworkNotificationView);
 };
 
 }  // namespace tray
 
-TrayNetwork::TrayNetwork()
-    : tray_(NULL),
+TrayNetwork::TrayNetwork(SystemTray* system_tray)
+    : SystemTrayItem(system_tray),
+      tray_(NULL),
       default_(NULL),
       detailed_(NULL),
       notification_(NULL),
       messages_(new tray::NetworkMessages()),
       request_wifi_view_(false) {
 }
 
 TrayNetwork::~TrayNetwork() {
 }
 
 views::View* TrayNetwork::CreateTrayView(user::LoginStatus status) {
   CHECK(tray_ == NULL);
-  tray_ = new tray::NetworkTrayView(tray::LIGHT, true /*tray_icon*/);
+  tray_ = new tray::NetworkTrayView(this, tray::LIGHT, true /*tray_icon*/);
   return tray_;
 }
 
 views::View* TrayNetwork::CreateDefaultView(user::LoginStatus status) {
   CHECK(default_ == NULL);
   default_ =
       new tray::NetworkDefaultView(this, status != user::LOGGED_IN_LOCKED);
   return default_;
 }
 
 views::View* TrayNetwork::CreateDetailedView(user::LoginStatus status) {
   CHECK(detailed_ == NULL);
   // Clear any notifications when showing the detailed view.
   messages_->messages().clear();
   HideNotificationView();
   if (request_wifi_view_) {
     SystemTrayDelegate* delegate = Shell::GetInstance()->tray_delegate();
     // The Wi-Fi state is not toggled yet at this point.
-    detailed_ = new tray::NetworkWifiDetailedView(!delegate->GetWifiEnabled());
+    detailed_ = new tray::NetworkWifiDetailedView(this,
+                                                  !delegate->GetWifiEnabled());
     request_wifi_view_ = false;
   } else {
     detailed_ = new tray::NetworkListDetailedView(
-        status, IDS_ASH_STATUS_TRAY_NETWORK);
+        this, status, IDS_ASH_STATUS_TRAY_NETWORK);
     detailed_->Init();
   }
   return detailed_;
 }
 
 views::View* TrayNetwork::CreateNotificationView(user::LoginStatus status) {
   CHECK(notification_ == NULL);
   if (messages_->messages().empty())
     return NULL;  // Message has already been cleared.
   notification_ = new tray::NetworkNotificationView(this);
@@ skipping 67 matching lines @@
 
 void TrayNetwork::LinkClicked(MessageType message_type, int link_id) {
   tray::NetworkMessages::MessageMap::const_iterator iter =
       messages()->messages().find(message_type);
   if (iter != messages()->messages().end() && iter->second.delegate)
     iter->second.delegate->NotificationLinkClicked(link_id);
 }
 
 }  // namespace internal
 }  // namespace ash