Notify Files App when ARC++ app is installed/removed

chrome/browser/chromeos/extensions/file_manager/event_router.cc

Index: chrome/browser/chromeos/extensions/file_manager/event_router.cc
diff --git a/chrome/browser/chromeos/extensions/file_manager/event_router.cc b/chrome/browser/chromeos/extensions/file_manager/event_router.cc
index 2b78461c5a07ff58cbacdfa590ab0b40537a8eae..9eeba39b6fd10c38b8a100099905bd660a8e70ae 100644
--- a/chrome/browser/chromeos/extensions/file_manager/event_router.cc
+++ b/chrome/browser/chromeos/extensions/file_manager/event_router.cc
@@ -400,8 +400,19 @@ EventRouter::EventRouter(Profile* profile)
 EventRouter::~EventRouter() {

[hidehiko, 2016/11/25 15:38:00]

Optional: while you're here, how about s/{}/= default;/ here, too?

[oka, 2016/11/28 07:15:43]

Done.

 }
 
+void EventRouter::OnAppsUpdated() {
+  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
+  BroadcastEvent(profile_,

[hidehiko, 2016/11/25 17:10:44]

Even if this EventRouter is for non-primary user profile, this is invoked when
primary user's app-install update is observed.

It may be a problem, depending on JS side implementation. IMHO, at least it is
confusing. So could you avoid it by not-observing ArcServiceManager if
|profile_| is non ARC support case?

ArcSessionManager::IsAllowedForProfile() would help you to implement it.

[oka, 2016/11/28 07:15:43]

Done.

+                 extensions::events::FILE_MANAGER_PRIVATE_ON_APPS_UPDATED,
+                 file_manager_private::OnAppsUpdated::kEventName,
+                 file_manager_private::OnAppsUpdated::Create());
+}
+
 void EventRouter::Shutdown() {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
+
+  arc::ArcServiceManager::Get()->RemoveObserver(this);
+
   chromeos::system::TimezoneSettings::GetInstance()->RemoveObserver(this);
 
   DLOG_IF(WARNING, !file_watchers_.empty())
@@ -492,6 +503,8 @@ void EventRouter::ObserveEvents() {
   pref_change_registrar_->Add(prefs::kUse24HourClock, callback);
 
   chromeos::system::TimezoneSettings::GetInstance()->AddObserver(this);
+
+  arc::ArcServiceManager::Get()->AddObserver(this);
 }
 
 // File watch setup routines.