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

ash/system/user/tray_user.cc

Index: ash/system/user/tray_user.cc
diff --git a/ash/system/user/tray_user.cc b/ash/system/user/tray_user.cc
index e476254333c4c6e0ae373362d5ae29924b601679..78ca43b0b2e2b8d520ff8364b73ac719e9bb76a8 100644
--- a/ash/system/user/tray_user.cc
+++ b/ash/system/user/tray_user.cc
@@ -5,6 +5,7 @@
 #include "ash/system/user/tray_user.h"
 
 #include "ash/shell.h"
+#include "ash/system/tray/system_tray.h"

[msw, 2012/11/16 20:09:04]

nit: potentially not necessary; tray_user.h include system_tray_item.h

[bartfab (slow), 2012/11/19 17:15:31]

system_tray_item.h has a forward declaration of the SystemTray class only. This
file needs the actual definition.

[oshima, 2012/11/19 17:43:43]

looks like it's not used in a way need definition?

[bartfab (slow), 2012/11/19 18:00:31]

Actually, you are right. I can remove the include - it is a somewhat moot
exercise though as my next CL will add it straight back because it will be using
the system tray object.

On 2012/11/19 17:43:43, oshima wrote:

[oshima, 2012/11/19 21:14:17]

If this will be necessary in next CL, please add it in that CL.
This way, we can keep the code in clean state.

[bartfab (slow), 2012/11/19 22:32:25]

Yes, of course. I addressed your comment and removed the include. I just
mentioned that it will be used soon so that the unnecessary include would have
become useful soon anyway, even if it had not been spotted now.

On 2012/11/19 21:14:17, oshima wrote:

[oshima, 2012/11/19 22:53:12]

I understand, but we generally advise to add it when you need it, not when
you will need it. This is because many people are working on the same files and
not everyone would know why it's there, and such line may be deleted by
other people.

[bartfab (slow), 2012/11/19 22:57:55]

Absolutely. It was a mistake.

On 2012/11/19 22:53:12, oshima wrote:

 #include "ash/system/tray/system_tray_delegate.h"
 #include "ash/system/tray/tray_constants.h"
 #include "ash/system/tray/tray_item_view.h"
@@ -246,8 +247,9 @@ class UserView : public views::View,
 
 }  // namespace tray
 
-TrayUser::TrayUser()
-    : user_(NULL),
+TrayUser::TrayUser(SystemTray* system_tray)
+    : SystemTrayItem(system_tray),
+      user_(NULL),
       avatar_(NULL),
       label_(NULL) {
 }