Implement message center on Windows

chrome/browser/notifications/message_center_notification_manager.cc

Index: chrome/browser/notifications/message_center_notification_manager.cc
diff --git a/chrome/browser/notifications/message_center_notification_manager.cc b/chrome/browser/notifications/message_center_notification_manager.cc
index fb1a6c934b76f0f7b42c020c6e821ea594d1808b..9ef6ee423641fbec96c78a856bcd942a12c78e4a 100644
--- a/chrome/browser/notifications/message_center_notification_manager.cc
+++ b/chrome/browser/notifications/message_center_notification_manager.cc
@@ -12,11 +12,16 @@
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/common/extensions/extension_set.h"
 #include "chrome/common/pref_names.h"
+#include "ui/message_center/message_center_tray.h"
 
 MessageCenterNotificationManager::MessageCenterNotificationManager(
     message_center::MessageCenter* message_center)
   : message_center_(message_center) {
   message_center_->SetDelegate(this);
+
+  // On Windows, the notification manager owns the tray icon and views.  Other
+  // platforms have global ownership and Create will return NULL.
+  tray_.reset(ui::MessageCenterTrayDelegate::CreateForPlatform());

[stevenjb, 2013/01/23 23:14:18]

I think this should use #if USE_ASH... relying on correct linkage for code
that's in the ui:: namespace but defined in src/ash is ugly at best. I would do
something like:

#if defined(USE_ASH)
tray_.reset(ash::Shell::GetInstance()->message_center_tray_delegate())
#elif defined(OS_WIN)
code for windows
#else
NOTIMPLEMENTED()
#endif

[dewittj, 2013/01/25 00:49:04]

Since on ChromeOS the Shell has the scoped pointer, I will just do nothing there
#if OS_CHROMEOS.

 }
 
 MessageCenterNotificationManager::~MessageCenterNotificationManager() {