Add ArcNotificationManager for new ARC notification

chrome/browser/chromeos/chrome_browser_main_chromeos.cc

 // Copyright (c) 2013 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 "chrome/browser/chromeos/chrome_browser_main_chromeos.h"
 
 #include <string>
 #include <vector>
 
 #include "ash/ash_switches.h"
@@ skipping 129 matching lines @@
 #include "components/rlz/rlz_tracker.h"
 #endif
 
 // Exclude X11 dependents for ozone
 #if defined(USE_X11)
 #include "chrome/browser/chromeos/device_uma.h"
 #include "chrome/browser/chromeos/events/system_key_event_listener.h"
 #include "chrome/browser/chromeos/events/xinput_hierarchy_changed_event_listener.h"
 #endif
 
+#if defined(USE_ASH)
+#include "chrome/browser/ui/ash/multi_user/multi_user_util.h"
+#endif
+
 namespace chromeos {
 
 namespace {
 
 void ChromeOSVersionCallback(const std::string& version) {
   base::SetLinuxDistro(std::string("CrOS ") + version);
 }
 
 bool ShouldAutoLaunchKioskApp(const base::CommandLine& command_line) {
   KioskAppManager* app_manager = KioskAppManager::Get();
@@ skipping 471 matching lines @@
       new RendererFreezer(scoped_ptr<RendererFreezer::Delegate>(
           new FreezerCgroupProcessManager())));
 
   g_browser_process->platform_part()->InitializeAutomaticRebootManager();
   g_browser_process->platform_part()->InitializeDeviceDisablingManager();
 
   // This observer cannot be created earlier because it requires the shell to be
   // available.
   idle_action_warning_observer_.reset(new IdleActionWarningObserver());
 
+  // TODO(yoshiki): Make the ArcNotificationManager instance managed by
+  // ArcServiceManager, when it can manage a service under C/B/chromeos

[elijahtaylor1, 2015/12/16 02:34:37]

seems this comment is out of date now?

[Luis Héctor Chávez, 2015/12/17 17:43:09]

components/arc cannot have any dependency to chrome/, so this will never be
posible.

[yoshiki, 2015/12/17 19:15:13]

Yes, removed.

+  // (crbug.com/569562).
+  if (arc::ArcBridgeService::GetEnabled(
+          base::CommandLine::ForCurrentProcess())) {
+    DCHECK(arc_service_manager_);
+#if defined(USE_ASH)

[Luis Héctor Chávez, 2015/12/17 17:43:09]

Why not gate the whole chunk with USE_ASH instead of just this line?

[yoshiki, 2015/12/17 19:15:13]

multi_user_util.[h|cc] is in ash/ directory so I thought we needed this guard.
But I found these files are always complied on chromeos, so I removed it.

+    arc_service_manager_->PostProfileInit(
+        multi_user_util::GetAccountIdFromProfile(profile()));
+#endif  // defined(USE_ASH)
+  }
+
   ChromeBrowserMainPartsLinux::PostProfileInit();
 }
 
 void ChromeBrowserMainPartsChromeos::PreBrowserStart() {
   // -- This used to be in ChromeBrowserMainParts::PreMainMessageLoopRun()
   // -- just before MetricsService::LogNeedForCleanShutdown().
 
   // Start the external metrics service, which collects metrics from Chrome OS
   // and passes them to the browser process.
   external_metrics_ = new chromeos::ExternalMetrics;
@@ skipping 169 matching lines @@
   // Destroy DBus services immediately after threads are stopped.
   dbus_services_.reset();
 
   ChromeBrowserMainPartsLinux::PostDestroyThreads();
 
   // Destroy DeviceSettingsService after g_browser_process.
   DeviceSettingsService::Shutdown();
 }
 
 }  //  namespace chromeos