Files App: Update the context menu when installed app is changed.

ui/file_manager/file_manager/foreground/js/task_controller.js

 // Copyright 2014 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.
 
 /**
  * @param {DialogType} dialogType
  * @param {!VolumeManagerWrapper} volumeManager
  * @param {!FileManagerUI} ui
  * @param {!MetadataModel} metadataModel
  * @param {!DirectoryModel} directoryModel
@@ skipping 80 matching lines @@
   this.tasks_ = null;
 
   ui.taskMenuButton.addEventListener(
       'select', this.onTaskItemClicked_.bind(this));
   this.selectionHandler_.addEventListener(
       FileSelectionHandler.EventType.CHANGE,
       this.onSelectionChanged_.bind(this));
   this.selectionHandler_.addEventListener(
       FileSelectionHandler.EventType.CHANGE_THROTTLED,
       this.onSelectionChangeThrottled_.bind(this));
+  chrome.fileManagerPrivate.onAppsUpdated.addListener(
+      this.onSelectionChangeThrottled_.bind(this));

[fukino, 2016/11/21 08:57:50]

Direct wiring to onSelectionChangeThrottled_ looks a bit strange to me.
How about extract a logic to update the context menu and dropdown menu, and call
it from onSelectionChangeThrottled_ and here?

[oka, 2016/11/21 09:14:47]

That's exactly what onSelectionChangeThrottled_ does. How about just renaming
onSelectionChangeThrottled_?

[fukino, 2016/11/21 09:27:29]

sgtm

 }
 
 /**
  * Cached the temporary disabled task item. Used inside
  * FileSelectionHandler.createTemporaryDisabledTaskItem_().
  * @type {Object}
  * @private
  */
 TaskController.cachedDisabledTaskItem_ = null;
 
@@ skipping 276 matching lines @@
   this.metadataModel_.get([entry], ['contentMimeType']).then(
       function(props) {
         FileTasks.create(
             this.volumeManager_, this.metadataModel_, this.directoryModel_,
             this.ui_, [entry], [props[0].contentMimeType || null])
             .then(function(tasks) {
               tasks.executeDefault();
             });
       }.bind(this));
 };