Add ArcNotificationManager for new ARC notification

chrome/browser/chromeos/chrome_browser_main_chromeos.cc

Index: chrome/browser/chromeos/chrome_browser_main_chromeos.cc
diff --git a/chrome/browser/chromeos/chrome_browser_main_chromeos.cc b/chrome/browser/chromeos/chrome_browser_main_chromeos.cc
index 8c0d0ed9c1baf9a7703271d9edb7ba80c57acd9c..b614723648979cc97ad546b5eeedcbd88d577398 100644
--- a/chrome/browser/chromeos/chrome_browser_main_chromeos.cc
+++ b/chrome/browser/chromeos/chrome_browser_main_chromeos.cc
@@ -30,6 +30,7 @@
 #include "chrome/browser/chromeos/app_mode/kiosk_app_launch_error.h"
 #include "chrome/browser/chromeos/app_mode/kiosk_app_manager.h"
 #include "chrome/browser/chromeos/app_mode/kiosk_mode_idle_app_name_notification.h"
+#include "chrome/browser/chromeos/arc/notification/arc_notification_manager.h"
 #include "chrome/browser/chromeos/boot_times_recorder.h"
 #include "chrome/browser/chromeos/dbus/chrome_console_service_provider_delegate.h"
 #include "chrome/browser/chromeos/dbus/chrome_display_power_service_provider_delegate.h"
@@ -642,6 +643,13 @@ void ChromeBrowserMainPartsChromeos::PostProfileInit() {
   // available.
   idle_action_warning_observer_.reset(new IdleActionWarningObserver());
 
+  if (arc::ArcBridgeService::GetEnabled(
+          base::CommandLine::ForCurrentProcess())) {
+    DCHECK(arc_bridge_service_);
+    arc_notification_manager_.reset(
+        new ArcNotificationManager(arc_bridge_service_.get(), profile()));

[elijahtaylor1, 2015/12/09 06:41:26]

is the main reason ArcNotificationManager lives in chrome/browser/chromeos/arc
instead of components/arc because we need the profile for its initialization? 
Or is there a deeper integration to chrome/browser that might cause a
layering/dependency violation from components/ ?

We've been consolidating ARC related services in
components/arc/arc_service_manager (see
https://codereview.chromium.org/1496193002/), maybe we just need a post-profile
initialization there as well and we can have this live there too.

[yoshiki, 2015/12/09 17:11:09]

Currently notification needs Profile to determine a desktop to be shown on, as I
wrote in the previous comment. And, to extract an image, the current
ArcNotificationManager depends chrome::ImageDecoder, which decode images in
sandbox (renderer process). So it's difficult to move it to under component/arc.
That's good. I'll make ArcNotificationManager managed by ArcServiceManager.

+  }
+
   ChromeBrowserMainPartsLinux::PostProfileInit();
 }